diff options
| author | Thomas Weißschuh <thomas.weissschuh@linutronix.de> | 2025-12-30 08:09:29 +0100 |
|---|---|---|
| committer | Nathan Chancellor <nathan@kernel.org> | 2025-12-30 13:06:03 -0700 |
| commit | 1b5e068d598e16b3a4ee3fa632108ea393d4e3f1 (patch) | |
| tree | 1162a890b7a4bb1795e431460570a4bd9bcda8c8 /usr/include | |
| parent | f8e05c10631e665ccabc8410beec6058d2a07c57 (diff) | |
kbuild: uapi: Drop check_config()
headers_install.sh already tests for CONFIG symbols. It does so in a
more comprehensive way, by only checking for references in code and
also running even if CONFIG_UAPI_HEADER_TEST is disabled.
And check_config() has been disabled any case.
Drop the pointless check.
Signed-off-by: Thomas Weißschuh <thomas.weissschuh@linutronix.de>
Reviewed-by: Nicolas Schier <nsc@kernel.org>
Link: https://patch.msgid.link/20251230-uapi-check_config-v1-1-00ecec109f39@linutronix.de
Signed-off-by: Nathan Chancellor <nathan@kernel.org>
Diffstat (limited to 'usr/include')
| -rwxr-xr-x | usr/include/headers_check.pl | 8 |
1 files changed, 0 insertions, 8 deletions
diff --git a/usr/include/headers_check.pl b/usr/include/headers_check.pl index af5a513eaa00..6cd6eb652c8d 100755 --- a/usr/include/headers_check.pl +++ b/usr/include/headers_check.pl @@ -40,7 +40,6 @@ foreach my $file (@files) { &check_include(); &check_asm_types(); &check_declarations(); - # Dropped for now. Too much noise &check_config(); } close $fh; } @@ -77,13 +76,6 @@ sub check_declarations } } -sub check_config -{ - if ($line =~ m/[^a-zA-Z0-9_]+CONFIG_([a-zA-Z0-9_]+)[^a-zA-Z0-9_]/) { - printf STDERR "$filename:$lineno: leaks CONFIG_$1 to userspace where it is not valid\n"; - } -} - my $linux_asm_types; sub check_asm_types { |