kbuild: uapi: Drop types.h check from headers_check.pl

Since commit d6fc9fcbaa65 ("kbuild: compile-test exported headers to
ensure they are self-contained"), the UAPI headers are compile tested
standalone with the compiler. This compile testing will catch a missing
include of types.h, making the types.h checking in headers_check.pl
originally added in commit 483b41218fa9 ("kbuild: add checks for include
of linux/types in userspace headers") obsolete.

Signed-off-by: Nathan Chancellor <nathan@kernel.org>
Reviewed-by: Thomas Weißschuh <thomas.weissschuh@linutronix.de>
Link: https://patch.msgid.link/20251023-headers-pl-drop-check-sizes-v1-1-18bd21cf8f5e@kernel.org
Signed-off-by: Nicolas Schier <nsc@kernel.org>

diff --git a/usr/include/headers_check.pl b/usr/include/headers_check.pl
index 1fbc8785f96e0b8f779b1236aafa19273c46d641..af5a513eaa00bb348ccd6ffe6ccbfc973f048834 100755 (executable)
--- a/usr/include/headers_check.pl
+++ b/usr/include/headers_check.pl
@@ -39,7 +39,6 @@ foreach my $file (@files) {
                $lineno++;
                &check_include();
                &check_asm_types();
-               &check_sizetypes();
                &check_declarations();
                # Dropped for now. Too much noise &check_config();
        }
@@ -103,65 +102,3 @@ sub check_asm_types
                $ret = 1;
        }
 }
-
-my $linux_types;
-my %import_stack = ();
-sub check_include_typesh
-{
-       my $path = $_[0];
-       my $import_path;
-
-       my $fh;
-       my @file_paths = ($path, $dir . "/" .  $path, dirname($filename) . "/" . $path);
-       for my $possible ( @file_paths ) {
-           if (not $import_stack{$possible} and open($fh, '<', $possible)) {
-               $import_path = $possible;
-               $import_stack{$import_path} = 1;
-               last;
-           }
-       }
-       if (eof $fh) {
-           return;
-       }
-
-       my $line;
-       while ($line = <$fh>) {
-               if ($line =~ m/^\s*#\s*include\s+<linux\/types.h>/) {
-                       $linux_types = 1;
-                       last;
-               }
-               if (my $included = ($line =~ /^\s*#\s*include\s+[<"](\S+)[>"]/)[0]) {
-                       check_include_typesh($included);
-               }
-       }
-       close $fh;
-       delete $import_stack{$import_path};
-}
-
-sub check_sizetypes
-{
-       if ($filename =~ /types.h|int-l64.h|int-ll64.h/o) {
-               return;
-       }
-       if ($lineno == 1) {
-               $linux_types = 0;
-       } elsif ($linux_types >= 1) {
-               return;
-       }
-       if ($line =~ m/^\s*#\s*include\s+<linux\/types.h>/) {
-               $linux_types = 1;
-               return;
-       }
-       if (my $included = ($line =~ /^\s*#\s*include\s+[<"](\S+)[>"]/)[0]) {
-               check_include_typesh($included);
-       }
-       # strip single-line comments, as types may be referenced within them
-       $line =~ s@/\*.*?\*/@@;
-       if ($line =~ m/__[us](8|16|32|64)\b/) {
-               printf STDERR "$filename:$lineno: " .
-                             "found __[us]{8,16,32,64} type " .
-                             "without #include <linux/types.h>\n";
-               $linux_types = 2;
-               $ret = 1;
-       }
-}