[PATCH v3 4/4] buildtools/chkincs: check driver specific headers
David Marchand
david.marchand at redhat.com
Mon Oct 21 18:04:47 CEST 2024
Only driver headers exported by libraries were checked.
Add driver specific headers to the checked headers list.
This reveals a small issue with the vmbus driver header as some driver
headers rely on it.
Fixes: b232b2aa212b ("buildtools/chkincs: check SDK headers for C++ compatibility")
Fixes: 84aaf06d817c ("bus/vmbus: make driver-only headers private")
Signed-off-by: David Marchand <david.marchand at redhat.com>
---
buildtools/chkincs/meson.build | 3 +++
drivers/meson.build | 1 +
2 files changed, 4 insertions(+)
diff --git a/buildtools/chkincs/meson.build b/buildtools/chkincs/meson.build
index 787d70272b..ed65105599 100644
--- a/buildtools/chkincs/meson.build
+++ b/buildtools/chkincs/meson.build
@@ -19,6 +19,9 @@ sources += gen_c_files.process(dpdk_chkinc_headers)
# some driver SDK headers depend on these two buses, which are mandatory in build
# so we always include them in deps list
deps = [get_variable('shared_rte_bus_vdev'), get_variable('shared_rte_bus_pci')]
+if dpdk_conf.has('RTE_BUS_VMBUS')
+ deps += get_variable('shared_rte_bus_vmbus')
+endif
# add the rest of the libs to the dependencies
foreach l:dpdk_libs_enabled
deps += get_variable('shared_rte_' + l)
diff --git a/drivers/meson.build b/drivers/meson.build
index 2733306698..5270160c56 100644
--- a/drivers/meson.build
+++ b/drivers/meson.build
@@ -220,6 +220,7 @@ foreach subpath:subdirs
if get_option('enable_driver_sdk')
install_headers(driver_sdk_headers)
endif
+ dpdk_chkinc_headers += driver_sdk_headers
if headers.length() > 0
dpdk_includes += include_directories(drv_path)
--
2.46.2
More information about the dev
mailing list