Unverified Commit 1b5e068d authored by Thomas Weißschuh's avatar Thomas Weißschuh Committed by Nathan Chancellor
Browse files

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: default avatarThomas Weißschuh <thomas.weissschuh@linutronix.de>
Reviewed-by: default avatarNicolas Schier <nsc@kernel.org>
Link: https://patch.msgid.link/20251230-uapi-check_config-v1-1-00ecec109f39@linutronix.de


Signed-off-by: default avatarNathan Chancellor <nathan@kernel.org>
parent f8e05c10

usr/include/headers_check.pl

+0 −8
Original line number Diff line number Diff line
@@ -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

{