> -----Original Message----- > From: McDaniel, Timothy <timothy.mcdan...@intel.com> > Sent: Friday, September 11, 2020 3:26 PM > To: Richardson, Bruce <bruce.richard...@intel.com>; Ray Kinsella > <m...@ashroe.eu>; Neil Horman <nhor...@tuxdriver.com> > Cc: dev@dpdk.org; Carrillo, Erik G <erik.g.carri...@intel.com>; Eads, Gage > <gage.e...@intel.com>; Van Haaren, Harry <harry.van.haa...@intel.com>; > jer...@marvell.com > Subject: [PATCH 01/22] event/dlb2: add meson build infrastructure > > Adds the meson build infrastructure, which includes > compile-time constants in rte_config.h. DLB2 is > only supported on Linux X86 platforms at this time. > > Signed-off-by: Timothy McDaniel <timothy.mcdan...@intel.com> > --- > config/rte_config.h | 7 +++++++ > drivers/event/dlb2/meson.build | 7 +++++++ > drivers/event/dlb2/rte_pmd_dlb2_event_version.map | 3 +++ > drivers/event/meson.build | 4 ++++ > 4 files changed, 21 insertions(+) > create mode 100644 drivers/event/dlb2/meson.build > create mode 100644 drivers/event/dlb2/rte_pmd_dlb2_event_version.map > > diff --git a/config/rte_config.h b/config/rte_config.h > index 0bae630..fd1b3c3 100644 > --- a/config/rte_config.h > +++ b/config/rte_config.h > @@ -131,4 +131,11 @@ > /* QEDE PMD defines */ > #define RTE_LIBRTE_QEDE_FW "" > > +/* DLB2 defines */ > +#define RTE_LIBRTE_PMD_DLB2_POLL_INTERVAL 1000 > +#define RTE_LIBRTE_PMD_DLB2_UMWAIT_CTL_STATE 0 > +#undef RTE_LIBRTE_PMD_DLB2_QUELL_STATS > +#define RTE_LIBRTE_PMD_DLB2_SW_CREDIT_QUANTA 32 > +#define RTE_PMD_DLB2_DEFAULT_DEPTH_THRESH 256 > + > #endif /* _RTE_CONFIG_H_ */ > diff --git a/drivers/event/dlb2/meson.build b/drivers/event/dlb2/meson.build > new file mode 100644 > index 0000000..d4fd39f > --- /dev/null > +++ b/drivers/event/dlb2/meson.build > @@ -0,0 +1,7 @@ > +# SPDX-License-Identifier: BSD-3-Clause > +# Copyright(c) 2019-2020 Intel Corporation > + > +sources = files( > +) > + > +deps += ['mbuf', 'mempool', 'ring', 'bus_vdev', 'pci', 'bus_pci'] I don't believe the code in this series depends on bus_vdev, is that right? With that resolved (if need be): Reviewed-by: Gage Eads <gage.e...@intel.com> Thanks, Gage
