DPDK patches and discussions
 help / color / mirror / Atom feed
From: "McDaniel, Timothy" <timothy.mcdaniel@intel.com>
To: Thomas Monjalon <thomas@monjalon.net>, "dev@dpdk.org" <dev@dpdk.org>
Cc: "ncopa@alpinelinux.org" <ncopa@alpinelinux.org>,
	Andrew Boyer <aboyer@pensando.io>,
	Andrew Rybchenko <andrew.rybchenko@oktetlabs.ru>,
	"David Marchand" <david.marchand@redhat.com>,
	"Burakov, Anatoly" <anatoly.burakov@intel.com>,
	Stephen Hemminger <sthemmin@microsoft.com>,
	"Long Li" <longli@microsoft.com>,
	Jay Zhou <jianjay.zhou@huawei.com>,
	Rasesh Mody <rmody@marvell.com>,
	Shahed Shaikh <shshaikh@marvell.com>,
	Maxime Coquelin <maxime.coquelin@redhat.com>,
	"Xia, Chenbo" <chenbo.xia@intel.com>,
	"Wang, Xiao W" <xiao.w.wang@intel.com>
Subject: Re: [dpdk-dev] [PATCH v7 06/17] drivers: replace page size definitions with function
Date: Fri, 19 Mar 2021 15:47:55 +0000	[thread overview]
Message-ID: <SN6PR11MB3103DF59F47751E8DFC5C9849E689@SN6PR11MB3103.namprd11.prod.outlook.com> (raw)
In-Reply-To: <20210319145730.3555384-7-thomas@monjalon.net>



> -----Original Message-----
> From: Thomas Monjalon <thomas@monjalon.net>
> Sent: Friday, March 19, 2021 9:57 AM
> To: dev@dpdk.org
> Cc: ncopa@alpinelinux.org; Andrew Boyer <aboyer@pensando.io>; Andrew
> Rybchenko <andrew.rybchenko@oktetlabs.ru>; David Marchand
> <david.marchand@redhat.com>; Burakov, Anatoly
> <anatoly.burakov@intel.com>; Stephen Hemminger
> <sthemmin@microsoft.com>; Long Li <longli@microsoft.com>; Jay Zhou
> <jianjay.zhou@huawei.com>; McDaniel, Timothy
> <timothy.mcdaniel@intel.com>; Rasesh Mody <rmody@marvell.com>; Shahed
> Shaikh <shshaikh@marvell.com>; Maxime Coquelin
> <maxime.coquelin@redhat.com>; Xia, Chenbo <chenbo.xia@intel.com>; Wang,
> Xiao W <xiao.w.wang@intel.com>
> Subject: [PATCH v7 06/17] drivers: replace page size definitions with function
> 
> The page size is often retrieved from the macro PAGE_SIZE.
> If PAGE_SIZE is not defined, it is either using hard coded default,
> or getting the system value from the UNIX-only function sysconf().
> 
> Such definitions are replaced with the generic function
> rte_mem_page_size() defined for each supported OS.
> 
> Removing PAGE_SIZE definitions will fix dlb drivers for musl libc,
> because #ifdef checks were missing, causing redefinition errors.
> 
> Signed-off-by: Thomas Monjalon <thomas@monjalon.net>
> Acked-by: Andrew Boyer <aboyer@pensando.io>
> Acked-by: Andrew Rybchenko <andrew.rybchenko@oktetlabs.ru>
> Acked-by: David Marchand <david.marchand@redhat.com>
> ---
>  drivers/bus/pci/linux/pci_vfio.c        |  9 ++-------
>  drivers/bus/vmbus/linux/vmbus_uio.c     |  4 ++--
>  drivers/bus/vmbus/private.h             |  5 +----
>  drivers/bus/vmbus/rte_vmbus_reg.h       |  2 +-
>  drivers/bus/vmbus/vmbus_common_uio.c    |  4 ++--
>  drivers/crypto/virtio/virtio_pci.h      |  3 ++-
>  drivers/event/dlb/dlb.c                 |  7 +++++--
>  drivers/event/dlb/dlb_priv.h            |  2 --
>  drivers/event/dlb/pf/base/dlb_osdep.h   |  2 +-
>  drivers/event/dlb/pf/dlb_main.h         |  5 +----
>  drivers/event/dlb/pf/dlb_pf.c           | 10 +++++----
>  drivers/event/dlb2/dlb2_priv.h          |  2 --
>  drivers/event/dlb2/pf/base/dlb2_osdep.h |  2 +-
>  drivers/event/dlb2/pf/dlb2_main.h       |  5 +----
>  drivers/event/dlb2/pf/dlb2_pf.c         |  8 ++++----
>  drivers/net/bnx2x/ecore_fw_defs.h       | 18 +++++++++++------
>  drivers/net/ionic/ionic_lif.c           | 27 ++++++++++++++-----------
>  drivers/net/ionic/ionic_main.c          |  3 ++-
>  drivers/net/ionic/ionic_osdep.h         |  2 +-
>  drivers/net/netvsc/hn_rndis.c           | 12 +++++------
>  drivers/net/netvsc/hn_rxtx.c            | 10 +++++----
>  drivers/net/netvsc/hn_var.h             |  9 +++------
>  drivers/net/virtio/virtio.h             |  2 +-
>  drivers/net/virtio/virtio_ethdev.c      |  6 +++---
>  drivers/net/virtio/virtio_ethdev.h      |  4 ----
>  drivers/vdpa/ifc/ifcvf_vdpa.c           | 17 +++++++---------
>  26 files changed, 85 insertions(+), 95 deletions(-)
> 
> diff --git a/drivers/bus/pci/linux/pci_vfio.c b/drivers/bus/pci/linux/pci_vfio.c
> index e3f7b6abeb..9d689d6fd9 100644
> --- a/drivers/bus/pci/linux/pci_vfio.c
> +++ b/drivers/bus/pci/linux/pci_vfio.c
> @@ -38,11 +38,6 @@
> 
>  #ifdef VFIO_PRESENT
> 
> -#ifndef PAGE_SIZE
> -#define PAGE_SIZE   (sysconf(_SC_PAGESIZE))
> -#endif
> -#define PAGE_MASK   (~(PAGE_SIZE - 1))
> -
>  static struct rte_tailq_elem rte_vfio_tailq = {
>  	.name = "VFIO_RESOURCE_LIST",
>  };
> @@ -507,8 +502,8 @@ pci_vfio_mmap_bar(int vfio_dev_fd, struct
> mapped_pci_resource *vfio_res,
>  		 */
>  		uint32_t table_start = msix_table->offset;
>  		uint32_t table_end = table_start + msix_table->size;
> -		table_end = RTE_ALIGN(table_end, PAGE_SIZE);
> -		table_start = RTE_ALIGN_FLOOR(table_start, PAGE_SIZE);
> +		table_end = RTE_ALIGN(table_end, rte_mem_page_size());
> +		table_start = RTE_ALIGN_FLOOR(table_start,
> rte_mem_page_size());
> 
>  		/* If page-aligned start of MSI-X table is less than the
>  		 * actual MSI-X table start address, reassign to the actual
> diff --git a/drivers/bus/vmbus/linux/vmbus_uio.c
> b/drivers/bus/vmbus/linux/vmbus_uio.c
> index 5dc0c47de6..b52ca5bf1d 100644
> --- a/drivers/bus/vmbus/linux/vmbus_uio.c
> +++ b/drivers/bus/vmbus/linux/vmbus_uio.c
> @@ -154,7 +154,7 @@ vmbus_uio_map_resource_by_index(struct
> rte_vmbus_device *dev, int idx,
>  		vmbus_map_addr = vmbus_find_max_end_va();
> 
>  	/* offset is special in uio it indicates which resource */
> -	offset = idx * PAGE_SIZE;
> +	offset = idx * rte_mem_page_size();
> 
>  	mapaddr = vmbus_map_resource(vmbus_map_addr, fd, offset, size,
> flags);
>  	close(fd);
> @@ -224,7 +224,7 @@ static int vmbus_uio_map_subchan(const struct
> rte_vmbus_device *dev,
>  	}
>  	file_size = sb.st_size;
> 
> -	if (file_size == 0 || (file_size & (PAGE_SIZE - 1))) {
> +	if (file_size == 0 || (file_size & (rte_mem_page_size() - 1))) {
>  		VMBUS_LOG(ERR, "incorrect size %s: %zu",
>  			  ring_path, file_size);
> 
> diff --git a/drivers/bus/vmbus/private.h b/drivers/bus/vmbus/private.h
> index f19b14e4a6..528d60a42f 100644
> --- a/drivers/bus/vmbus/private.h
> +++ b/drivers/bus/vmbus/private.h
> @@ -9,13 +9,10 @@
>  #include <stdbool.h>
>  #include <sys/uio.h>
>  #include <rte_log.h>
> +#include <rte_eal_paging.h>
>  #include <rte_vmbus_reg.h>
>  #include <rte_bus_vmbus.h>
> 
> -#ifndef PAGE_SIZE
> -#define PAGE_SIZE	4096
> -#endif
> -
>  extern struct rte_vmbus_bus rte_vmbus_bus;
> 
>  extern int vmbus_logtype_bus;
> diff --git a/drivers/bus/vmbus/rte_vmbus_reg.h
> b/drivers/bus/vmbus/rte_vmbus_reg.h
> index f5a0693dcb..8562672979 100644
> --- a/drivers/bus/vmbus/rte_vmbus_reg.h
> +++ b/drivers/bus/vmbus/rte_vmbus_reg.h
> @@ -100,7 +100,7 @@ struct vmbus_bufring {
>  		uint32_t value;
>  	} feature_bits;
> 
> -	/* Pad it to PAGE_SIZE so that data starts on page boundary */
> +	/* Pad it to rte_mem_page_size() so that data starts on page boundary
> */
>  	uint8_t	reserved2[4028];
> 
>  	/*
> diff --git a/drivers/bus/vmbus/vmbus_common_uio.c
> b/drivers/bus/vmbus/vmbus_common_uio.c
> index a689bf11b3..8582e32c1d 100644
> --- a/drivers/bus/vmbus/vmbus_common_uio.c
> +++ b/drivers/bus/vmbus/vmbus_common_uio.c
> @@ -63,7 +63,7 @@ vmbus_uio_map_secondary(struct rte_vmbus_device
> *dev)
> 
>  	for (i = 0; i != uio_res->nb_maps; i++) {
>  		void *mapaddr;
> -		off_t offset = i * PAGE_SIZE;
> +		off_t offset = i * rte_mem_page_size();
> 
>  		mapaddr = vmbus_map_resource(uio_res->maps[i].addr,
>  					     fd, offset,
> @@ -175,7 +175,7 @@ vmbus_uio_map_resource(struct rte_vmbus_device
> *dev)
>  	}
> 
>  	dev->int_page = (uint32_t *)((char *)uio_res-
> >maps[HV_INT_PAGE_MAP].addr
> -				     + (PAGE_SIZE >> 1));
> +				     + (rte_mem_page_size() >> 1));
>  	dev->monitor_page = uio_res->maps[HV_MON_PAGE_MAP].addr;
>  	return 0;
>  }
> diff --git a/drivers/crypto/virtio/virtio_pci.h b/drivers/crypto/virtio/virtio_pci.h
> index d9a214dfd0..0a7ea1bb64 100644
> --- a/drivers/crypto/virtio/virtio_pci.h
> +++ b/drivers/crypto/virtio/virtio_pci.h
> @@ -7,6 +7,7 @@
> 
>  #include <stdint.h>
> 
> +#include <rte_eal_paging.h>
>  #include <rte_pci.h>
>  #include <rte_bus_pci.h>
>  #include <rte_cryptodev.h>
> @@ -67,7 +68,7 @@ struct virtqueue;
>   *
>   * Note the sizeof(struct vring_desc) is 16 bytes.
>   */
> -#define VIRTIO_MAX_INDIRECT ((int) (PAGE_SIZE / 16))
> +#define VIRTIO_MAX_INDIRECT ((int) (rte_mem_page_size() / 16))
> 
>  /* Do we get callbacks when the ring is completely used, even if we've
>   * suppressed them?
> diff --git a/drivers/event/dlb/dlb.c b/drivers/event/dlb/dlb.c
> index 8b26d1d2d2..1174ab3d2b 100644
> --- a/drivers/event/dlb/dlb.c
> +++ b/drivers/event/dlb/dlb.c
> @@ -23,6 +23,7 @@
>  #include <rte_io.h>
>  #include <rte_kvargs.h>
>  #include <rte_log.h>
> +#include <rte_eal_paging.h>
>  #include <rte_malloc.h>
>  #include <rte_mbuf.h>
>  #include <rte_power_intrinsics.h>
> @@ -991,7 +992,8 @@ dlb_hw_create_ldb_port(struct dlb_eventdev *dlb,
>  		goto error_exit;
>  	}
> 
> -	qm_port->pp_mmio_base = DLB_LDB_PP_BASE + PAGE_SIZE *
> qm_port_id;
> +	qm_port->pp_mmio_base = DLB_LDB_PP_BASE +
> +			rte_mem_page_size() * qm_port_id;
>  	qm_port->id = qm_port_id;
> 
>  	/* The credit window is one high water mark of QEs */
> @@ -1181,7 +1183,8 @@ dlb_hw_create_dir_port(struct dlb_eventdev *dlb,
>  		goto error_exit;
>  	}
> 
> -	qm_port->pp_mmio_base = DLB_DIR_PP_BASE + PAGE_SIZE *
> qm_port_id;
> +	qm_port->pp_mmio_base = DLB_DIR_PP_BASE +
> +			rte_mem_page_size() * qm_port_id;
>  	qm_port->id = qm_port_id;
> 
>  	/* The credit window is one high water mark of QEs */
> diff --git a/drivers/event/dlb/dlb_priv.h b/drivers/event/dlb/dlb_priv.h
> index 272e17482f..ca4d6a84cf 100644
> --- a/drivers/event/dlb/dlb_priv.h
> +++ b/drivers/event/dlb/dlb_priv.h
> @@ -76,8 +76,6 @@
> 
>  #define PP_BASE(is_dir) ((is_dir) ? DLB_DIR_PP_BASE : DLB_LDB_PP_BASE)
> 
> -#define PAGE_SIZE (sysconf(_SC_PAGESIZE))
> -
>  #define DLB_NUM_QES_PER_CACHE_LINE 4
> 
>  #define DLB_MAX_ENQUEUE_DEPTH 64
> diff --git a/drivers/event/dlb/pf/base/dlb_osdep.h
> b/drivers/event/dlb/pf/base/dlb_osdep.h
> index 0c119b759a..dee20660ed 100644
> --- a/drivers/event/dlb/pf/base/dlb_osdep.h
> +++ b/drivers/event/dlb/pf/base/dlb_osdep.h
> @@ -92,7 +92,7 @@ static inline void *os_map_producer_port(struct dlb_hw
> *hw,
> 
> 
>  	pp_dma_base = (uintptr_t)hw->func_kva + DLB_PP_BASE(is_ldb);
> -	addr = (pp_dma_base + (PAGE_SIZE * port_id));
> +	addr = (pp_dma_base + (rte_mem_page_size() * port_id));
> 
>  	return (void *)(uintptr_t)addr;
> 
> diff --git a/drivers/event/dlb/pf/dlb_main.h b/drivers/event/dlb/pf/dlb_main.h
> index 22e215223d..e66ba228d1 100644
> --- a/drivers/event/dlb/pf/dlb_main.h
> +++ b/drivers/event/dlb/pf/dlb_main.h
> @@ -10,10 +10,7 @@
>  #include <rte_spinlock.h>
>  #include <rte_pci.h>
>  #include <rte_bus_pci.h>
> -
> -#ifndef PAGE_SIZE
> -#define PAGE_SIZE (sysconf(_SC_PAGESIZE))
> -#endif
> +#include <rte_eal_paging.h>
> 
>  #include "base/dlb_hw_types.h"
>  #include "../dlb_user.h"
> diff --git a/drivers/event/dlb/pf/dlb_pf.c b/drivers/event/dlb/pf/dlb_pf.c
> index 3aeef6f91d..2f7e30abbd 100644
> --- a/drivers/event/dlb/pf/dlb_pf.c
> +++ b/drivers/event/dlb/pf/dlb_pf.c
> @@ -303,7 +303,7 @@ dlb_pf_ldb_port_create(struct dlb_hw_dev *handle,
>  	alloc_sz = RTE_CACHE_LINE_ROUNDUP(alloc_sz);
> 
>  	port_base = dlb_alloc_coherent_aligned(&mz, &pc_dma_base,
> -					       alloc_sz, PAGE_SIZE);
> +					       alloc_sz, rte_mem_page_size());
>  	if (port_base == NULL)
>  		return -ENOMEM;
> 
> @@ -328,7 +328,8 @@ dlb_pf_ldb_port_create(struct dlb_hw_dev *handle,
> 
>  	pp_dma_base = (uintptr_t)dlb_dev->hw.func_kva + PP_BASE(is_dir);
>  	dlb_port[response.id][DLB_LDB].pp_addr =
> -		(void *)(uintptr_t)(pp_dma_base + (PAGE_SIZE * response.id));
> +			(void *)(uintptr_t)(pp_dma_base +
> +			(rte_mem_page_size() * response.id));
> 
>  	dlb_port[response.id][DLB_LDB].cq_base =
>  		(void *)(uintptr_t)(port_base + (2 * RTE_CACHE_LINE_SIZE));
> @@ -381,7 +382,7 @@ dlb_pf_dir_port_create(struct dlb_hw_dev *handle,
>  	alloc_sz = RTE_CACHE_LINE_ROUNDUP(alloc_sz);
> 
>  	port_base = dlb_alloc_coherent_aligned(&mz, &pc_dma_base,
> -					       alloc_sz, PAGE_SIZE);
> +					       alloc_sz, rte_mem_page_size());
>  	if (port_base == NULL)
>  		return -ENOMEM;
> 
> @@ -406,7 +407,8 @@ dlb_pf_dir_port_create(struct dlb_hw_dev *handle,
> 
>  	pp_dma_base = (uintptr_t)dlb_dev->hw.func_kva + PP_BASE(is_dir);
>  	dlb_port[response.id][DLB_DIR].pp_addr =
> -		(void *)(uintptr_t)(pp_dma_base + (PAGE_SIZE * response.id));
> +			(void *)(uintptr_t)(pp_dma_base +
> +			(rte_mem_page_size() * response.id));
> 
>  	dlb_port[response.id][DLB_DIR].cq_base =
>  		(void *)(uintptr_t)(port_base + (2 * RTE_CACHE_LINE_SIZE));
> diff --git a/drivers/event/dlb2/dlb2_priv.h b/drivers/event/dlb2/dlb2_priv.h
> index b73cf3ff14..eb1a932399 100644
> --- a/drivers/event/dlb2/dlb2_priv.h
> +++ b/drivers/event/dlb2/dlb2_priv.h
> @@ -78,8 +78,6 @@
>  				    DLB2_LDB_CQ_MAX_SIZE)
>  #define PP_BASE(is_dir) ((is_dir) ? DLB2_DIR_PP_BASE : DLB2_LDB_PP_BASE)
> 
> -#define PAGE_SIZE (sysconf(_SC_PAGESIZE))
> -
>  #define DLB2_NUM_QES_PER_CACHE_LINE 4
> 
>  #define DLB2_MAX_ENQUEUE_DEPTH 64
> diff --git a/drivers/event/dlb2/pf/base/dlb2_osdep.h
> b/drivers/event/dlb2/pf/base/dlb2_osdep.h
> index c4c34eba5f..aa101a49a3 100644
> --- a/drivers/event/dlb2/pf/base/dlb2_osdep.h
> +++ b/drivers/event/dlb2/pf/base/dlb2_osdep.h
> @@ -89,7 +89,7 @@ static inline void *os_map_producer_port(struct dlb2_hw
> *hw,
>  	uint64_t pp_dma_base;
> 
>  	pp_dma_base = (uintptr_t)hw->func_kva + DLB2_PP_BASE(is_ldb);
> -	addr = (pp_dma_base + (PAGE_SIZE * port_id));
> +	addr = (pp_dma_base + (rte_mem_page_size() * port_id));
> 
>  	return (void *)(uintptr_t)addr;
>  }
> diff --git a/drivers/event/dlb2/pf/dlb2_main.h
> b/drivers/event/dlb2/pf/dlb2_main.h
> index f3bee71fb6..9eeda482a3 100644
> --- a/drivers/event/dlb2/pf/dlb2_main.h
> +++ b/drivers/event/dlb2/pf/dlb2_main.h
> @@ -10,10 +10,7 @@
>  #include <rte_spinlock.h>
>  #include <rte_pci.h>
>  #include <rte_bus_pci.h>
> -
> -#ifndef PAGE_SIZE
> -#define PAGE_SIZE (sysconf(_SC_PAGESIZE))
> -#endif
> +#include <rte_eal_paging.h>
> 
>  #include "base/dlb2_hw_types.h"
>  #include "../dlb2_user.h"
> diff --git a/drivers/event/dlb2/pf/dlb2_pf.c b/drivers/event/dlb2/pf/dlb2_pf.c
> index 1a7d8fc294..1142da5b25 100644
> --- a/drivers/event/dlb2/pf/dlb2_pf.c
> +++ b/drivers/event/dlb2/pf/dlb2_pf.c
> @@ -284,7 +284,7 @@ dlb2_pf_ldb_port_create(struct dlb2_hw_dev *handle,
>  	alloc_sz = RTE_CACHE_LINE_ROUNDUP(alloc_sz);
> 
>  	port_base = dlb2_alloc_coherent_aligned(&mz, &cq_base, alloc_sz,
> -						PAGE_SIZE);
> +						rte_mem_page_size());
>  	if (port_base == NULL)
>  		return -ENOMEM;
> 
> @@ -307,7 +307,7 @@ dlb2_pf_ldb_port_create(struct dlb2_hw_dev *handle,
> 
>  	pp_base = (uintptr_t)dlb2_dev->hw.func_kva + PP_BASE(is_dir);
>  	dlb2_port[response.id][DLB2_LDB_PORT].pp_addr =
> -		(void *)(pp_base + (PAGE_SIZE * response.id));
> +		(void *)(pp_base + (rte_mem_page_size() * response.id));
> 
>  	dlb2_port[response.id][DLB2_LDB_PORT].cq_base = (void
> *)(port_base);
>  	memset(&port_memory, 0, sizeof(port_memory));
> @@ -359,7 +359,7 @@ dlb2_pf_dir_port_create(struct dlb2_hw_dev *handle,
>  	alloc_sz = RTE_CACHE_LINE_ROUNDUP(alloc_sz);
> 
>  	port_base = dlb2_alloc_coherent_aligned(&mz, &cq_base, alloc_sz,
> -						PAGE_SIZE);
> +						rte_mem_page_size());
>  	if (port_base == NULL)
>  		return -ENOMEM;
> 
> @@ -382,7 +382,7 @@ dlb2_pf_dir_port_create(struct dlb2_hw_dev *handle,
> 
>  	pp_base = (uintptr_t)dlb2_dev->hw.func_kva + PP_BASE(is_dir);
>  	dlb2_port[response.id][DLB2_DIR_PORT].pp_addr =
> -		(void *)(pp_base + (PAGE_SIZE * response.id));
> +		(void *)(pp_base + (rte_mem_page_size() * response.id));
> 
>  	dlb2_port[response.id][DLB2_DIR_PORT].cq_base =
>  		(void *)(port_base);
> diff --git a/drivers/net/bnx2x/ecore_fw_defs.h
> b/drivers/net/bnx2x/ecore_fw_defs.h
> index 5397a701aa..93bca8ad33 100644
> --- a/drivers/net/bnx2x/ecore_fw_defs.h
> +++ b/drivers/net/bnx2x/ecore_fw_defs.h
> @@ -13,6 +13,8 @@
>  #ifndef ECORE_FW_DEFS_H
>  #define ECORE_FW_DEFS_H
> 
> +#include <rte_eal_paging.h>
> +
>  #define CSTORM_ASSERT_LIST_INDEX_OFFSET	(IRO[152].base)
>  #define CSTORM_ASSERT_LIST_OFFSET(assertListEntry) \
>  	(IRO[151].base + ((assertListEntry) * IRO[151].m1))
> @@ -252,7 +254,8 @@
>  #define X_ETH_LOCAL_RING_SIZE 13
>  #define FIRST_BD_IN_PKT	0
>  #define PARSE_BD_INDEX 1
> -#define NUM_OF_ETH_BDS_IN_PAGE
> ((PAGE_SIZE)/(STRUCT_SIZE(eth_tx_bd)/8))
> +#define NUM_OF_ETH_BDS_IN_PAGE \
> +	(rte_mem_page_size() / (STRUCT_SIZE(eth_tx_bd) / 8))
>  #define U_ETH_NUM_OF_SGES_TO_FETCH 8
>  #define U_ETH_MAX_SGES_FOR_PACKET 3
> 
> @@ -265,11 +268,14 @@
>  #define IP_HEADER_ALIGNMENT_PADDING 2
> 
>  #define U_ETH_SGES_PER_PAGE_INVERSE_MASK \
> -	(0xFFFF - ((PAGE_SIZE/((STRUCT_SIZE(eth_rx_sge))/8))-1))
> +	(0xFFFF - ((rte_mem_page_size() / ((STRUCT_SIZE(eth_rx_sge)) / 8)) - 1))
> 
> -#define TU_ETH_CQES_PER_PAGE (PAGE_SIZE/(STRUCT_SIZE(eth_rx_cqe)/8))
> -#define U_ETH_BDS_PER_PAGE (PAGE_SIZE/(STRUCT_SIZE(eth_rx_bd)/8))
> -#define U_ETH_SGES_PER_PAGE (PAGE_SIZE/(STRUCT_SIZE(eth_rx_sge)/8))
> +#define TU_ETH_CQES_PER_PAGE \
> +	(rte_mem_page_size() / (STRUCT_SIZE(eth_rx_cqe) / 8))
> +#define U_ETH_BDS_PER_PAGE \
> +	(rte_mem_page_size() / (STRUCT_SIZE(eth_rx_bd) / 8))
> +#define U_ETH_SGES_PER_PAGE \
> +	(rte_mem_page_size() / (STRUCT_SIZE(eth_rx_sge) / 8))
> 
>  #define U_ETH_BDS_PER_PAGE_MASK	(U_ETH_BDS_PER_PAGE-1)
>  #define U_ETH_CQE_PER_PAGE_MASK	(TU_ETH_CQES_PER_PAGE-1)
> @@ -396,7 +402,7 @@
> 
>  /* Event Ring definitions */
>  #define C_ERES_PER_PAGE \
> -	(PAGE_SIZE / BITS_TO_BYTES(STRUCT_SIZE(event_ring_elem)))
> +	(rte_mem_page_size() /
> BITS_TO_BYTES(STRUCT_SIZE(event_ring_elem)))
>  #define C_ERE_PER_PAGE_MASK (C_ERES_PER_PAGE - 1)
> 
>  /* number of statistic command */
> diff --git a/drivers/net/ionic/ionic_lif.c b/drivers/net/ionic/ionic_lif.c
> index cd220abee2..431eda777b 100644
> --- a/drivers/net/ionic/ionic_lif.c
> +++ b/drivers/net/ionic/ionic_lif.c
> @@ -612,18 +612,18 @@ ionic_qcq_alloc(struct ionic_lif *lif,
>  	cq_size = num_descs * cq_desc_size;
>  	sg_size = num_descs * sg_desc_size;
> 
> -	total_size = RTE_ALIGN(q_size, PAGE_SIZE) +
> -		RTE_ALIGN(cq_size, PAGE_SIZE);
> +	total_size = RTE_ALIGN(q_size, rte_mem_page_size()) +
> +			RTE_ALIGN(cq_size, rte_mem_page_size());
>  	/*
>  	 * Note: aligning q_size/cq_size is not enough due to cq_base address
>  	 * aligning as q_base could be not aligned to the page.
> -	 * Adding PAGE_SIZE.
> +	 * Adding rte_mem_page_size().
>  	 */
> -	total_size += PAGE_SIZE;
> +	total_size += rte_mem_page_size();
> 
>  	if (flags & IONIC_QCQ_F_SG) {
> -		total_size += RTE_ALIGN(sg_size, PAGE_SIZE);
> -		total_size += PAGE_SIZE;
> +		total_size += RTE_ALIGN(sg_size, rte_mem_page_size());
> +		total_size += rte_mem_page_size();
>  	}
> 
>  	new = rte_zmalloc("ionic", struct_size, 0);
> @@ -636,7 +636,7 @@ ionic_qcq_alloc(struct ionic_lif *lif,
> 
>  	new->q.info = rte_calloc_socket("ionic",
>  				num_descs, sizeof(void *),
> -				PAGE_SIZE, socket_id);
> +				rte_mem_page_size(), socket_id);
>  	if (!new->q.info) {
>  		IONIC_PRINT(ERR, "Cannot allocate queue info");
>  		err = -ENOMEM;
> @@ -673,13 +673,16 @@ ionic_qcq_alloc(struct ionic_lif *lif,
>  	q_base = new->base;
>  	q_base_pa = new->base_pa;
> 
> -	cq_base = (void *)RTE_ALIGN((uintptr_t)q_base + q_size, PAGE_SIZE);
> -	cq_base_pa = RTE_ALIGN(q_base_pa + q_size, PAGE_SIZE);
> +	cq_base = (void *)RTE_ALIGN((uintptr_t)q_base + q_size,
> +			rte_mem_page_size());
> +	cq_base_pa = RTE_ALIGN(q_base_pa + q_size,
> +			rte_mem_page_size());
> 
>  	if (flags & IONIC_QCQ_F_SG) {
>  		sg_base = (void *)RTE_ALIGN((uintptr_t)cq_base + cq_size,
> -			PAGE_SIZE);
> -		sg_base_pa = RTE_ALIGN(cq_base_pa + cq_size, PAGE_SIZE);
> +				rte_mem_page_size());
> +		sg_base_pa = RTE_ALIGN(cq_base_pa + cq_size,
> +				rte_mem_page_size());
>  		ionic_q_sg_map(&new->q, sg_base, sg_base_pa);
>  	}
> 
> @@ -1002,7 +1005,7 @@ ionic_lif_alloc(struct ionic_lif *lif)
> 
>  	IONIC_PRINT(DEBUG, "Allocating Lif Info");
> 
> -	lif->info_sz = RTE_ALIGN(sizeof(*lif->info), PAGE_SIZE);
> +	lif->info_sz = RTE_ALIGN(sizeof(*lif->info), rte_mem_page_size());
> 
>  	lif->info_z = rte_eth_dma_zone_reserve(lif->eth_dev,
>  		"lif_info", 0 /* queue_idx*/,
> diff --git a/drivers/net/ionic/ionic_main.c b/drivers/net/ionic/ionic_main.c
> index 9aa7b2e96c..7301f53342 100644
> --- a/drivers/net/ionic/ionic_main.c
> +++ b/drivers/net/ionic/ionic_main.c
> @@ -456,7 +456,8 @@ ionic_port_init(struct ionic_adapter *adapter)
>  	if (idev->port_info)
>  		return 0;
> 
> -	idev->port_info_sz = RTE_ALIGN(sizeof(*idev->port_info), PAGE_SIZE);
> +	idev->port_info_sz = RTE_ALIGN(sizeof(*idev->port_info),
> +			rte_mem_page_size());
> 
>  	snprintf(z_name, sizeof(z_name), "%s_port_%s_info",
>  		IONIC_DRV_NAME, adapter->name);
> diff --git a/drivers/net/ionic/ionic_osdep.h b/drivers/net/ionic/ionic_osdep.h
> index a6575c36b5..89ed106d11 100644
> --- a/drivers/net/ionic/ionic_osdep.h
> +++ b/drivers/net/ionic/ionic_osdep.h
> @@ -17,6 +17,7 @@
>  #include <rte_byteorder.h>
>  #include <rte_io.h>
>  #include <rte_memory.h>
> +#include <rte_eal_paging.h>
> 
>  #include "ionic_logs.h"
> 
> @@ -25,7 +26,6 @@
> 
>  #ifndef PAGE_SHIFT
>  #define PAGE_SHIFT      12
> -#define PAGE_SIZE       (1 << PAGE_SHIFT)
>  #endif
> 
>  #define __iomem
> diff --git a/drivers/net/netvsc/hn_rndis.c b/drivers/net/netvsc/hn_rndis.c
> index e317539de7..e3f7e636d7 100644
> --- a/drivers/net/netvsc/hn_rndis.c
> +++ b/drivers/net/netvsc/hn_rndis.c
> @@ -67,7 +67,7 @@ hn_rndis_rid(struct hn_data *hv)
> 
>  static void *hn_rndis_alloc(size_t size)
>  {
> -	return rte_zmalloc("RNDIS", size, PAGE_SIZE);
> +	return rte_zmalloc("RNDIS", size, rte_mem_page_size());
>  }
> 
>  #ifdef RTE_LIBRTE_NETVSC_DEBUG_DUMP
> @@ -265,17 +265,17 @@ static int hn_nvs_send_rndis_ctrl(struct
> vmbus_channel *chan,
>  		return -EINVAL;
>  	}
> 
> -	if (unlikely(reqlen > PAGE_SIZE)) {
> +	if (unlikely(reqlen > rte_mem_page_size())) {
>  		PMD_DRV_LOG(ERR, "RNDIS request %u greater than page
> size",
>  			    reqlen);
>  		return -EINVAL;
>  	}
> 
> -	sg.page = addr / PAGE_SIZE;
> +	sg.page = addr / rte_mem_page_size();
>  	sg.ofs  = addr & PAGE_MASK;
>  	sg.len  = reqlen;
> 
> -	if (sg.ofs + reqlen >  PAGE_SIZE) {
> +	if (sg.ofs + reqlen >  rte_mem_page_size()) {
>  		PMD_DRV_LOG(ERR, "RNDIS request crosses page boundary");
>  		return -EINVAL;
>  	}
> @@ -479,7 +479,7 @@ hn_rndis_query(struct hn_data *hv, uint32_t oid,
>  		return -ENOMEM;
> 
>  	comp_len = sizeof(*comp) + odlen;
> -	comp = rte_zmalloc("QUERY", comp_len, PAGE_SIZE);
> +	comp = rte_zmalloc("QUERY", comp_len, rte_mem_page_size());
>  	if (!comp) {
>  		error = -ENOMEM;
>  		goto done;
> @@ -736,7 +736,7 @@ hn_rndis_set(struct hn_data *hv, uint32_t oid, const
> void *data, uint32_t dlen)
>  	int error;
> 
>  	reqlen = sizeof(*req) + dlen;
> -	req = rte_zmalloc("RNDIS_SET", reqlen, PAGE_SIZE);
> +	req = rte_zmalloc("RNDIS_SET", reqlen, rte_mem_page_size());
>  	if (!req)
>  		return -ENOMEM;
> 
> diff --git a/drivers/net/netvsc/hn_rxtx.c b/drivers/net/netvsc/hn_rxtx.c
> index 0f4ef0100b..c6bf7cc132 100644
> --- a/drivers/net/netvsc/hn_rxtx.c
> +++ b/drivers/net/netvsc/hn_rxtx.c
> @@ -1387,7 +1387,8 @@ static unsigned int hn_get_slots(const struct rte_mbuf
> *m)
>  		unsigned int size = rte_pktmbuf_data_len(m);
>  		unsigned int offs = rte_mbuf_data_iova(m) & PAGE_MASK;
> 
> -		slots += (offs + size + PAGE_SIZE - 1) / PAGE_SIZE;
> +		slots += (offs + size + rte_mem_page_size() - 1) /
> +				rte_mem_page_size();
>  		m = m->next;
>  	}
> 
> @@ -1402,12 +1403,13 @@ static unsigned int hn_fill_sg(struct vmbus_gpa *sg,
> 
>  	while (m) {
>  		rte_iova_t addr = rte_mbuf_data_iova(m);
> -		unsigned int page = addr / PAGE_SIZE;
> +		unsigned int page = addr / rte_mem_page_size();
>  		unsigned int offset = addr & PAGE_MASK;
>  		unsigned int len = rte_pktmbuf_data_len(m);
> 
>  		while (len > 0) {
> -			unsigned int bytes = RTE_MIN(len, PAGE_SIZE - offset);
> +			unsigned int bytes = RTE_MIN(len,
> +					rte_mem_page_size() - offset);
> 
>  			sg[segs].page = page;
>  			sg[segs].ofs = offset;
> @@ -1450,7 +1452,7 @@ static int hn_xmit_sg(struct hn_tx_queue *txq,
>  	addr = txq->tx_rndis_iova +
>  		((char *)txd->rndis_pkt - (char *)txq->tx_rndis);
> 
> -	sg[0].page = addr / PAGE_SIZE;
> +	sg[0].page = addr / rte_mem_page_size();
>  	sg[0].ofs = addr & PAGE_MASK;
>  	sg[0].len = RNDIS_PACKET_MSG_OFFSET_ABS(hn_rndis_pktlen(txd-
> >rndis_pkt));
>  	segs = 1;
> diff --git a/drivers/net/netvsc/hn_var.h b/drivers/net/netvsc/hn_var.h
> index b7405ca726..43642408bc 100644
> --- a/drivers/net/netvsc/hn_var.h
> +++ b/drivers/net/netvsc/hn_var.h
> @@ -6,6 +6,8 @@
>   * All rights reserved.
>   */
> 
> +#include <rte_eal_paging.h>
> +
>  /*
>   * Tunable ethdev params
>   */
> @@ -28,13 +30,8 @@
> 
>  #define HN_RX_EXTMBUF_ENABLE	0
> 
> -/* Buffers need to be aligned */
> -#ifndef PAGE_SIZE
> -#define PAGE_SIZE 4096
> -#endif
> -
>  #ifndef PAGE_MASK
> -#define PAGE_MASK (PAGE_SIZE - 1)
> +#define PAGE_MASK (rte_mem_page_size() - 1)
>  #endif
> 
>  struct hn_data;
> diff --git a/drivers/net/virtio/virtio.h b/drivers/net/virtio/virtio.h
> index 21d54904e7..2c987d19ab 100644
> --- a/drivers/net/virtio/virtio.h
> +++ b/drivers/net/virtio/virtio.h
> @@ -98,7 +98,7 @@
>   *
>   * Note the sizeof(struct vring_desc) is 16 bytes.
>   */
> -#define VIRTIO_MAX_INDIRECT ((int)(PAGE_SIZE / 16))
> +#define VIRTIO_MAX_INDIRECT ((int)(rte_mem_page_size() / 16))
> 
>  /*
>   * Maximum number of virtqueues per device.
> diff --git a/drivers/net/virtio/virtio_ethdev.c b/drivers/net/virtio/virtio_ethdev.c
> index 333a5243a9..289b240c4e 100644
> --- a/drivers/net/virtio/virtio_ethdev.c
> +++ b/drivers/net/virtio/virtio_ethdev.c
> @@ -21,8 +21,8 @@
>  #include <rte_errno.h>
>  #include <rte_cpuflags.h>
>  #include <rte_vect.h>
> -
>  #include <rte_memory.h>
> +#include <rte_eal_paging.h>
>  #include <rte_eal.h>
>  #include <rte_dev.h>
>  #include <rte_cycles.h>
> @@ -469,7 +469,7 @@ virtio_init_queue(struct rte_eth_dev *dev, uint16_t
> queue_idx)
>  		sz_hdr_mz = vq_size * sizeof(struct virtio_tx_region);
>  	} else if (queue_type == VTNET_CQ) {
>  		/* Allocate a page for control vq command, data and status */
> -		sz_hdr_mz = PAGE_SIZE;
> +		sz_hdr_mz = rte_mem_page_size();
>  	}
> 
>  	vq = rte_zmalloc_socket(vq_name, size, RTE_CACHE_LINE_SIZE,
> @@ -568,7 +568,7 @@ virtio_init_queue(struct rte_eth_dev *dev, uint16_t
> queue_idx)
>  		cvq->mz = mz;
>  		cvq->virtio_net_hdr_mz = hdr_mz;
>  		cvq->virtio_net_hdr_mem = hdr_mz->iova;
> -		memset(cvq->virtio_net_hdr_mz->addr, 0, PAGE_SIZE);
> +		memset(cvq->virtio_net_hdr_mz->addr, 0,
> rte_mem_page_size());
> 
>  		hw->cvq = cvq;
>  	}
> diff --git a/drivers/net/virtio/virtio_ethdev.h b/drivers/net/virtio/virtio_ethdev.h
> index 873327d989..5a501e7890 100644
> --- a/drivers/net/virtio/virtio_ethdev.h
> +++ b/drivers/net/virtio/virtio_ethdev.h
> @@ -11,10 +11,6 @@
> 
>  #include "virtio.h"
> 
> -#ifndef PAGE_SIZE
> -#define PAGE_SIZE 4096
> -#endif
> -
>  #define VIRTIO_MAX_RX_QUEUES 128U
>  #define VIRTIO_MAX_TX_QUEUES 128U
>  #define VIRTIO_MAX_MAC_ADDRS 64
> diff --git a/drivers/vdpa/ifc/ifcvf_vdpa.c b/drivers/vdpa/ifc/ifcvf_vdpa.c
> index 6a1b44bc77..39237aecf7 100644
> --- a/drivers/vdpa/ifc/ifcvf_vdpa.c
> +++ b/drivers/vdpa/ifc/ifcvf_vdpa.c
> @@ -11,6 +11,7 @@
>  #include <linux/virtio_net.h>
>  #include <stdbool.h>
> 
> +#include <rte_eal_paging.h>
>  #include <rte_malloc.h>
>  #include <rte_memory.h>
>  #include <rte_bus_pci.h>
> @@ -30,10 +31,6 @@ RTE_LOG_REGISTER(ifcvf_vdpa_logtype,
> pmd.net.ifcvf_vdpa, NOTICE);
>  	rte_log(RTE_LOG_ ## level, ifcvf_vdpa_logtype, \
>  		"IFCVF %s(): " fmt "\n", __func__, ##args)
> 
> -#ifndef PAGE_SIZE
> -#define PAGE_SIZE 4096
> -#endif
> -
>  #define IFCVF_USED_RING_LEN(size) \
>  	((size) * sizeof(struct vring_used_elem) + sizeof(uint16_t) * 3)
> 
> @@ -600,11 +597,11 @@ m_ifcvf_start(struct ifcvf_internal *internal)
>  	for (i = 0; i < nr_vring; i++) {
>  		rte_vhost_get_vhost_vring(vid, i, &vq);
> 
> -		size = RTE_ALIGN_CEIL(vring_size(vq.size, PAGE_SIZE),
> -				PAGE_SIZE);
> -		vring_buf = rte_zmalloc("ifcvf", size, PAGE_SIZE);
> +		size = RTE_ALIGN_CEIL(vring_size(vq.size,
> rte_mem_page_size()),
> +				rte_mem_page_size());
> +		vring_buf = rte_zmalloc("ifcvf", size, rte_mem_page_size());
>  		vring_init(&internal->m_vring[i], vq.size, vring_buf,
> -				PAGE_SIZE);
> +				rte_mem_page_size());
> 
>  		ret = rte_vfio_container_dma_map(internal-
> >vfio_container_fd,
>  			(uint64_t)(uintptr_t)vring_buf, m_vring_iova, size);
> @@ -686,8 +683,8 @@ m_ifcvf_stop(struct ifcvf_internal *internal)
>  		len = IFCVF_USED_RING_LEN(vq.size);
>  		rte_vhost_log_used_vring(vid, i, 0, len);
> 
> -		size = RTE_ALIGN_CEIL(vring_size(vq.size, PAGE_SIZE),
> -				PAGE_SIZE);
> +		size = RTE_ALIGN_CEIL(vring_size(vq.size,
> rte_mem_page_size()),
> +				rte_mem_page_size());
>  		rte_vfio_container_dma_unmap(internal->vfio_container_fd,
>  			(uint64_t)(uintptr_t)internal->m_vring[i].desc,
>  			m_vring_iova, size);
> --
> 2.30.1

ack

  reply	other threads:[~2021-03-19 15:48 UTC|newest]

Thread overview: 140+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-03-13 17:06 [dpdk-dev] [PATCH v3 00/15] Build fixes for musl libc Natanael Copa
2019-03-13 17:06 ` [dpdk-dev] [PATCH v3 01/15] app/testpmd: replace uint with unsigned int Natanael Copa
2019-03-26 16:59   ` Iremonger, Bernard
2019-03-26 16:59     ` Iremonger, Bernard
2019-03-30 22:17   ` Thomas Monjalon
2019-03-30 22:17     ` Thomas Monjalon
2019-07-04  9:20     ` Iremonger, Bernard
2019-03-13 17:06 ` [dpdk-dev] [PATCH v3 02/15] net/cxgbe: " Natanael Copa
2019-03-13 17:06 ` [dpdk-dev] [PATCH v3 03/15] bus/pci: add fallback for out[lwb]_p for non GNU libc Natanael Copa
2019-03-30 22:22   ` Thomas Monjalon
2019-03-30 22:22     ` Thomas Monjalon
2019-07-04 12:16     ` Natanael Copa
2019-07-04 13:20       ` Thomas Monjalon
2019-07-04 15:30         ` Natanael Copa
2019-03-13 17:06 ` [dpdk-dev] [PATCH v3 04/15] bus/pci: factor out various ifdefs in pci_uio_ioport_{read, write} Natanael Copa
2019-03-30 22:25   ` Thomas Monjalon
2019-03-30 22:25     ` Thomas Monjalon
2019-03-13 17:06 ` [dpdk-dev] [PATCH v3 05/15] bus/fslmc: fix compile error with musl libc Natanael Copa
2019-03-13 17:06 ` [dpdk-dev] [PATCH v3 06/15] bus/fslmc: remove unused include of error.h Natanael Copa
2019-03-13 17:06 ` [dpdk-dev] [PATCH v3 07/15] net/nfp: build fix for musl libc Natanael Copa
2019-03-21  9:48   ` Alejandro Lucero
2019-03-21  9:48     ` Alejandro Lucero
2019-03-13 17:06 ` [dpdk-dev] [PATCH v3 08/15] app/test: include fcntl.h due to use of O_RDONLY Natanael Copa
2019-03-13 17:06 ` [dpdk-dev] [PATCH v3 09/15] app/test: fix setting of -D_GNU_SOURCE with meson Natanael Copa
2019-03-13 17:06 ` [dpdk-dev] [PATCH v3 10/15] bus/dpaa: use warn(3) instead of error(3) to improve portability Natanael Copa
2019-03-30 22:33   ` Thomas Monjalon
2019-03-30 22:33     ` Thomas Monjalon
2020-07-10  5:13   ` Hemant Agrawal
2021-03-19  9:56     ` Thomas Monjalon
2019-03-13 17:06 ` [dpdk-dev] [PATCH v3 11/15] bus/dpaa: fix warning: "__WORDSIZE" is not defined, evaluates to 0 Natanael Copa
2019-03-30 22:36   ` Thomas Monjalon
2019-03-30 22:36     ` Thomas Monjalon
2020-07-10  5:10   ` Hemant Agrawal
2019-03-13 17:06 ` [dpdk-dev] [PATCH v3 12/15] crypto/dpaa2_sec: build fix for musl libc Natanael Copa
2020-07-10  5:11   ` Hemant Agrawal
2019-03-13 17:06 ` [dpdk-dev] [PATCH v3 13/15] crypto/dpaa2_sec: simplify pr_{debug, err, warn} macros Natanael Copa
2019-03-30 22:39   ` Thomas Monjalon
2019-03-30 22:39     ` Thomas Monjalon
2020-07-10  5:09   ` Hemant Agrawal
2020-07-10  5:11   ` Hemant Agrawal
2019-03-13 17:06 ` [dpdk-dev] [PATCH v3 14/15] net/netvsc: fix compile warning for fcntl.h include Natanael Copa
2019-03-13 17:06 ` [dpdk-dev] [PATCH v3 15/15] eal/linux: simplify debug message in sigbus_handler Natanael Copa
2019-03-14 17:09 ` [dpdk-dev] [PATCH v3 00/15] Build fixes for musl libc Sirvys, Andrius
2019-03-14 17:09   ` Sirvys, Andrius
2019-03-30 23:57 ` Thomas Monjalon
2019-03-30 23:57   ` Thomas Monjalon
2019-07-05  8:03   ` David Marchand
2020-11-05 21:17   ` [dpdk-dev] " Natanael Copa
2020-11-05 21:17     ` [dpdk-dev] [PATCH v4 1/8] app/testpmd: fix uint build error with " Natanael Copa
2020-11-06  9:22       ` [dpdk-dev] [PATCH v4 1/8] app/testpmd: fix uint build error withmusl libc Morten Brørup
2020-11-05 21:17     ` [dpdk-dev] [PATCH v4 2/8] net/cxgbe: fix uint build error with musl libc Natanael Copa
2020-11-06  9:24       ` [dpdk-dev] [PATCH v4 2/8] net/cxgbe: fix uint build error with musllibc Morten Brørup
2020-11-05 21:17     ` [dpdk-dev] [PATCH v4 3/8] bus/pci: add fallback for out* for non GNU libc Natanael Copa
2020-11-05 21:17     ` [dpdk-dev] [PATCH v4 4/8] bus/dpaa: use warn instead of error to improve portability Natanael Copa
2020-11-10 14:41       ` Fabrizio D'Angelo
2020-11-05 21:17     ` [dpdk-dev] [PATCH v4 5/8] bus/dpaa: fix detection of 64 bit arch Natanael Copa
2020-11-05 21:17     ` [dpdk-dev] [PATCH v4 6/8] common/dpaax: build fix for musl libc Natanael Copa
2020-11-05 21:17     ` [dpdk-dev] [PATCH v4 7/8] common/dpaax: simplify pr debug/err/warn macros Natanael Copa
2020-11-05 21:17     ` [dpdk-dev] [PATCH v4 8/8] eal: add missing include to fix build with musl libc Natanael Copa
2020-11-06 10:11     ` [dpdk-dev] Build fixes for " Thomas Monjalon
2020-11-06 10:44     ` David Marchand
2021-02-18 16:13       ` Thomas Monjalon
2021-02-20  8:51         ` Natanael Copa
2021-02-25  9:41           ` Thomas Monjalon
2021-02-25 18:22 ` [dpdk-dev] [PATCH v5 00/17] Alpine/musl build support Thomas Monjalon
2021-02-25 18:22   ` [dpdk-dev] [PATCH v5 01/17] eal: fix comment of OS-specific header files Thomas Monjalon
2021-02-25 18:22   ` [dpdk-dev] [PATCH v5 02/17] buildtools: fix build with busybox Thomas Monjalon
2021-02-26  9:11     ` Bruce Richardson
2021-02-25 18:22   ` [dpdk-dev] [PATCH v5 03/17] build: detect execinfo library on Linux Thomas Monjalon
2021-02-26  9:08     ` Bruce Richardson
2021-02-25 18:22   ` [dpdk-dev] [PATCH v5 04/17] build: define _GNU_SOURCE globally Thomas Monjalon
2021-02-26  9:08     ` Bruce Richardson
2021-02-26  9:40       ` Thomas Monjalon
2021-02-26  9:46         ` Bruce Richardson
2021-02-26 10:04           ` Thomas Monjalon
2021-02-28 12:53             ` Thomas Monjalon
2021-02-25 18:22   ` [dpdk-dev] [PATCH v5 05/17] eal/linux: fix build with musl Thomas Monjalon
2021-02-25 18:22   ` [dpdk-dev] [PATCH v5 06/17] drivers: fix header includes for musl Thomas Monjalon
2021-02-25 18:22   ` [dpdk-dev] [PATCH v5 07/17] drivers: replace page size definitions with function Thomas Monjalon
2021-02-25 18:22   ` [dpdk-dev] [PATCH v5 08/17] bus/pci: support I/O port operations with musl Thomas Monjalon
2021-02-25 18:22   ` [dpdk-dev] [PATCH v5 09/17] bus/dpaa: fix 64-bit arch detection Thomas Monjalon
2021-02-25 18:22   ` [dpdk-dev] [PATCH v5 10/17] bus/dpaa: fix build with musl Thomas Monjalon
2021-02-25 18:22   ` [dpdk-dev] [PATCH v5 11/17] common/dpaax/caamflib: " Thomas Monjalon
2021-02-25 18:22   ` [dpdk-dev] [PATCH v5 12/17] common/dpaax/caamflib: simplify log macros Thomas Monjalon
2021-02-25 18:22   ` [dpdk-dev] [PATCH v5 13/17] net/cxgbe: remove use of uint type Thomas Monjalon
2021-02-25 18:22   ` [dpdk-dev] [PATCH v5 14/17] net/igc: " Thomas Monjalon
2021-02-26  0:49     ` Wang, Haiyue
2021-02-25 18:22   ` [dpdk-dev] [PATCH v5 15/17] vdpa/mlx5: replace pthread functions unavailable in musl Thomas Monjalon
2021-03-01  7:30     ` Matan Azrad
2021-02-25 18:22   ` [dpdk-dev] [PATCH v5 16/17] app/testpmd: fix build with musl Thomas Monjalon
2021-02-25 18:22   ` [dpdk-dev] [PATCH v5 17/17] examples/bbdev: fix header include for musl Thomas Monjalon
2021-02-28 12:53 ` [dpdk-dev] [PATCH v6 00/17] Alpine/musl build support Thomas Monjalon
2021-02-28 12:53   ` [dpdk-dev] [PATCH v6 01/17] eal: fix comment of OS-specific header files Thomas Monjalon
2021-02-28 12:53   ` [dpdk-dev] [PATCH v6 02/17] buildtools: fix build with busybox Thomas Monjalon
2021-02-28 12:53   ` [dpdk-dev] [PATCH v6 03/17] build: detect execinfo library on Linux Thomas Monjalon
2021-02-28 12:53   ` [dpdk-dev] [PATCH v6 04/17] build: remove redundant _GNU_SOURCE definitions Thomas Monjalon
2021-02-28 12:53   ` [dpdk-dev] [PATCH v6 05/17] eal: fix build with musl Thomas Monjalon
2021-02-28 12:53   ` [dpdk-dev] [PATCH v6 06/17] drivers: fix header includes for musl Thomas Monjalon
2021-02-28 12:53   ` [dpdk-dev] [PATCH v6 07/17] drivers: replace page size definitions with function Thomas Monjalon
2021-03-01  9:41     ` Bruce Richardson
2021-03-01  9:53       ` Thomas Monjalon
2021-03-01 10:09         ` Bruce Richardson
2021-03-01 14:22     ` Andrew Boyer
2021-02-28 12:53   ` [dpdk-dev] [PATCH v6 08/17] bus/pci: support I/O port operations with musl Thomas Monjalon
2021-02-28 12:53   ` [dpdk-dev] [PATCH v6 09/17] bus/dpaa: fix 64-bit arch detection Thomas Monjalon
2021-02-28 12:53   ` [dpdk-dev] [PATCH v6 10/17] bus/dpaa: fix build with musl Thomas Monjalon
2021-02-28 12:53   ` [dpdk-dev] [PATCH v6 11/17] common/dpaax/caamflib: " Thomas Monjalon
2021-02-28 12:53   ` [dpdk-dev] [PATCH v6 12/17] common/dpaax/caamflib: simplify log macros Thomas Monjalon
2021-02-28 12:53   ` [dpdk-dev] [PATCH v6 13/17] net/cxgbe: remove use of uint type Thomas Monjalon
2021-02-28 12:53   ` [dpdk-dev] [PATCH v6 14/17] net/igc: " Thomas Monjalon
2021-02-28 12:53   ` [dpdk-dev] [PATCH v6 15/17] vdpa/mlx5: replace pthread functions unavailable in musl Thomas Monjalon
2021-02-28 12:53   ` [dpdk-dev] [PATCH v6 16/17] app/testpmd: fix build with musl Thomas Monjalon
2021-02-28 12:53   ` [dpdk-dev] [PATCH v6 17/17] examples/bbdev: fix header include for musl Thomas Monjalon
2021-03-17 22:42   ` [dpdk-dev] [PATCH v6 00/17] Alpine/musl build support Thomas Monjalon
2021-03-18  8:12     ` Andrew Rybchenko
2021-03-18  8:56     ` David Marchand
2021-03-19 14:57 ` [dpdk-dev] [PATCH v7 " Thomas Monjalon
2021-03-19 14:57   ` [dpdk-dev] [PATCH v7 01/17] eal: fix comment of OS-specific header files Thomas Monjalon
2021-03-19 14:57   ` [dpdk-dev] [PATCH v7 02/17] buildtools: fix build with busybox Thomas Monjalon
2021-03-22  8:52     ` Kinsella, Ray
2021-03-22  8:55       ` Thomas Monjalon
2021-03-22  9:01         ` Kinsella, Ray
2021-03-19 14:57   ` [dpdk-dev] [PATCH v7 03/17] build: detect execinfo library on Linux Thomas Monjalon
2021-03-19 14:57   ` [dpdk-dev] [PATCH v7 04/17] build: remove redundant _GNU_SOURCE definitions Thomas Monjalon
2021-03-19 14:57   ` [dpdk-dev] [PATCH v7 05/17] eal: fix build with musl Thomas Monjalon
2021-03-19 14:57   ` [dpdk-dev] [PATCH v7 06/17] drivers: replace page size definitions with function Thomas Monjalon
2021-03-19 15:47     ` McDaniel, Timothy [this message]
2021-03-19 14:57   ` [dpdk-dev] [PATCH v7 07/17] common/dpaax/caamflib: fix build with musl Thomas Monjalon
2021-03-19 14:57   ` [dpdk-dev] [PATCH v7 08/17] common/dpaax/caamflib: simplify log macros Thomas Monjalon
2021-03-19 14:57   ` [dpdk-dev] [PATCH v7 09/17] bus/dpaa: fix 64-bit arch detection Thomas Monjalon
2021-03-19 14:57   ` [dpdk-dev] [PATCH v7 10/17] bus/dpaa: fix build with musl Thomas Monjalon
2021-03-19 14:57   ` [dpdk-dev] [PATCH v7 11/17] bus/pci: support I/O port operations " Thomas Monjalon
2021-03-19 14:57   ` [dpdk-dev] [PATCH v7 12/17] net/cxgbe: remove use of uint type Thomas Monjalon
2021-03-19 14:57   ` [dpdk-dev] [PATCH v7 13/17] net/igc: " Thomas Monjalon
2021-03-19 14:57   ` [dpdk-dev] [PATCH v7 14/17] vdpa/mlx5: replace pthread functions unavailable in musl Thomas Monjalon
2021-03-19 14:57   ` [dpdk-dev] [PATCH v7 15/17] event/dlb: fix header includes for musl Thomas Monjalon
2021-03-19 15:47     ` McDaniel, Timothy
2021-03-19 14:57   ` [dpdk-dev] [PATCH v7 16/17] app/testpmd: fix build with musl Thomas Monjalon
2021-03-19 14:57   ` [dpdk-dev] [PATCH v7 17/17] examples/bbdev: fix header include for musl Thomas Monjalon
2021-03-23 10:02   ` [dpdk-dev] [PATCH v7 00/17] Alpine/musl build support Thomas Monjalon

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=SN6PR11MB3103DF59F47751E8DFC5C9849E689@SN6PR11MB3103.namprd11.prod.outlook.com \
    --to=timothy.mcdaniel@intel.com \
    --cc=aboyer@pensando.io \
    --cc=anatoly.burakov@intel.com \
    --cc=andrew.rybchenko@oktetlabs.ru \
    --cc=chenbo.xia@intel.com \
    --cc=david.marchand@redhat.com \
    --cc=dev@dpdk.org \
    --cc=jianjay.zhou@huawei.com \
    --cc=longli@microsoft.com \
    --cc=maxime.coquelin@redhat.com \
    --cc=ncopa@alpinelinux.org \
    --cc=rmody@marvell.com \
    --cc=shshaikh@marvell.com \
    --cc=sthemmin@microsoft.com \
    --cc=thomas@monjalon.net \
    --cc=xiao.w.wang@intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).