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.march...@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