On 3/12/2021 10:20 PM, Tyler Retzlaff wrote:
Introduce a meson option enable_driver_sdk when true installs internal
driver headers for ethdev. this allows drivers that do not depend on
stable api/abi to be built external to the dpdk source tree.
Signed-off-by: Tyler Retzlaff <roret...@linux.microsoft.com>
---
lib/librte_ethdev/meson.build | 6 ++++++
lib/meson.build | 4 ++++
meson_options.txt | 2 ++
3 files changed, 12 insertions(+)
diff --git a/lib/librte_ethdev/meson.build b/lib/librte_ethdev/meson.build
index c37b2e377..4353fa6b7 100644
--- a/lib/librte_ethdev/meson.build
+++ b/lib/librte_ethdev/meson.build
@@ -20,8 +20,14 @@ headers = files('rte_ethdev.h',
'rte_mtr_driver.h',
'rte_tm.h',
'rte_tm_driver.h')
+
indirect_headers += files(
'rte_ethdev_core.h',
'rte_eth_ctrl.h')
+driver_sdk_headers += files(
+ 'ethdev_driver.h',
+ 'ethdev_pci.h',
+ 'ethdev_vdev.h')
+
deps += ['net', 'kvargs', 'meter', 'telemetry']
diff --git a/lib/meson.build b/lib/meson.build
index 7712aa497..992ebdf63 100644
--- a/lib/meson.build
+++ b/lib/meson.build
@@ -67,6 +67,7 @@ foreach l:libraries
sources = []
headers = []
indirect_headers = [] # public headers not directly included by apps
+ driver_sdk_headers = [] # public headers included by drivers
includes = []
cflags = default_cflags
objs = [] # other object files to link against, used e.g. for
@@ -105,6 +106,9 @@ foreach l:libraries
dpdk_conf.set('RTE_LIB_' + name.to_upper(), 1)
install_headers(headers)
install_headers(indirect_headers)
+ if get_option('enable_driver_sdk')
+ install_headers(driver_sdk_headers)
+ endif
dpdk_chkinc_headers += headers
libname = 'rte_' + name
diff --git a/meson_options.txt b/meson_options.txt
index 6eff62e47..857874a19 100644
--- a/meson_options.txt
+++ b/meson_options.txt
@@ -8,6 +8,8 @@ option('drivers_install_subdir', type: 'string', value:
'dpdk/pmds-<VERSION>',
description: 'Subdirectory of libdir where to install PMDs. Defaults to
using a versioned subdirectory.')
option('enable_docs', type: 'boolean', value: false,
description: 'build documentation')
+option('enable_driver_sdk', type: 'boolean', value: false,
+ description: 'install internal driver plugin headers')
option('enable_kmods', type: 'boolean', value: false,
description: 'build kernel modules')
option('examples', type: 'string', value: '',
+Ed, who was looking way to install 'ark_ext.h' for Ark PMD.
Ed,
Can you please review the patch from your perspective?