[dpdk-dev] [PATCH v6 07/17] drivers: replace page size definitions with function
Andrew Boyer
aboyer at pensando.io
Mon Mar 1 15:22:05 CET 2021
> On Feb 28, 2021, at 7:53 AM, Thomas Monjalon <thomas at monjalon.net> wrote:
>
> 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 at monjalon.net>
> ---
> 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 | 25 ++++++++++++++-----------
> 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, 84 insertions(+), 94 deletions(-)
>
OK for net/ionic
Acked-by: Andrew Boyer <aboyer at pensando.io>
More information about the dev
mailing list