buildtools/chkincs: check headers without internal API

Checking all headers with internal symbols may let some incorrect calls
leak in normal applications headers.
Track the driver headers in a new list and check it with
ALLOW_INTERNAL_API.

At the same time, include driver headers from bus drivers.

Fixes: b232b2a ("buildtools/chkincs: check SDK headers for C++ compatibility")

Signed-off-by: David Marchand <[email protected]>

buildtools/chkincs/meson.build

@@ -12,7 +12,6 @@ gen_c_files = generator(gen_c_file_for_header,
         arguments: ['@INPUT@', '@OUTPUT@'])
 
 cflags = machine_args
-cflags += '-DALLOW_INTERNAL_API'
 
 sources = files('main.c')
 sources += gen_c_files.process(dpdk_chkinc_headers)
@@ -39,6 +38,19 @@ executable('chkincs-exp', sources,
         dependencies: deps,
         install: false)
 
+cflags += '-DALLOW_INTERNAL_API'
+
+if dpdk_chkinc_driver_headers.length() != 0
+    sources_driver = files('main.c')
+    sources_driver += gen_c_files.process(dpdk_chkinc_driver_headers)
+
+    executable('chkincs-driver', sources_driver,
+            c_args: cflags,
+            include_directories: includes,
+            dependencies: deps,
+            install: false)
+endif
+
 # run tests for c++ builds also
 if not add_languages('cpp', required: false)
     subdir_done()
@@ -59,7 +71,7 @@ gen_cpp_files = generator(gen_c_file_for_header,
         arguments: ['@INPUT@', '@OUTPUT@'])
 
 cpp_sources = files('main.cpp')
-cpp_sources += gen_cpp_files.process(dpdk_chkinc_headers)
+cpp_sources += gen_cpp_files.process(dpdk_chkinc_headers + dpdk_chkinc_driver_headers)
 
 executable('chkincs-cpp', cpp_sources,
         cpp_args: ['-include', 'rte_config.h', cflags],

drivers/meson.build

@@ -218,6 +218,7 @@ foreach subpath:subdirs
         if get_option('enable_driver_sdk')
             install_headers(driver_sdk_headers)
         endif
+        dpdk_chkinc_driver_headers += driver_sdk_headers
 
         if headers.length() > 0
             dpdk_includes += include_directories(drv_path)

lib/meson.build

@@ -240,7 +240,7 @@ foreach l:libraries
         install_headers(driver_sdk_headers)
     endif
     dpdk_chkinc_headers += headers
-    dpdk_chkinc_headers += driver_sdk_headers
+    dpdk_chkinc_driver_headers += driver_sdk_headers
 
     libname = 'rte_' + name
     includes += include_directories(l)

meson.build

@@ -39,6 +39,7 @@ dpdk_static_libraries = []
 dpdk_shared_lib_deps = []
 dpdk_static_lib_deps = []
 dpdk_chkinc_headers = []
+dpdk_chkinc_driver_headers = []
 dpdk_driver_classes = []
 dpdk_drivers = []
 dpdk_extra_ldflags = []