blob: db1dd7a549f2b9e1259148c39175ad6ee6df3d02 [file] [log] [blame]
Jeremy Huntwork15a2ee72008-10-29 14:20:13 -07001#!/usr/bin/perl -w
Sam Ravnborg77124012008-06-15 21:41:09 +02002#
3# headers_check.pl execute a number of trivial consistency checks
4#
Amerigo Wang67b7ebe2009-06-04 22:12:01 -04005# Usage: headers_check.pl dir arch [files...]
Sam Ravnborg77124012008-06-15 21:41:09 +02006# dir: dir to look for included files
7# arch: architecture
8# files: list of files to check
9#
10# The script reads the supplied files line by line and:
11#
12# 1) for each include statement it checks if the
13# included file actually exists.
14# Only include files located in asm* and linux* are checked.
15# The rest are assumed to be system include files.
16#
Mike Frysinger46b8af52008-12-27 02:43:36 -050017# 2) It is checked that prototypes does not use "extern"
18#
Sam Ravnborg7e557a22008-12-27 19:52:20 +010019# 3) Check for leaked CONFIG_ symbols
Sam Ravnborg77124012008-06-15 21:41:09 +020020
21use strict;
Sam Ravnborg77124012008-06-15 21:41:09 +020022
23my ($dir, $arch, @files) = @ARGV;
24
25my $ret = 0;
26my $line;
27my $lineno = 0;
28my $filename;
29
30foreach my $file (@files) {
Jeremy Huntwork15a2ee72008-10-29 14:20:13 -070031 local *FH;
Sam Ravnborg77124012008-06-15 21:41:09 +020032 $filename = $file;
Jeremy Huntwork15a2ee72008-10-29 14:20:13 -070033 open(FH, "<$filename") or die "$filename: $!\n";
Sam Ravnborg77124012008-06-15 21:41:09 +020034 $lineno = 0;
Jeremy Huntwork15a2ee72008-10-29 14:20:13 -070035 while ($line = <FH>) {
Sam Ravnborg77124012008-06-15 21:41:09 +020036 $lineno++;
Sam Ravnborg483b4122008-12-30 11:34:58 +010037 &check_include();
38 &check_asm_types();
39 &check_sizetypes();
Amerigo Wang67b7ebe2009-06-04 22:12:01 -040040 &check_declarations();
Sam Ravnborg7e3fa562009-01-30 23:56:42 +010041 # Dropped for now. Too much noise &check_config();
Sam Ravnborg77124012008-06-15 21:41:09 +020042 }
Jeremy Huntwork15a2ee72008-10-29 14:20:13 -070043 close FH;
Sam Ravnborg77124012008-06-15 21:41:09 +020044}
45exit $ret;
46
47sub check_include
48{
49 if ($line =~ m/^\s*#\s*include\s+<((asm|linux).*)>/) {
50 my $inc = $1;
51 my $found;
52 $found = stat($dir . "/" . $inc);
53 if (!$found) {
54 $inc =~ s#asm/#asm-$arch/#;
55 $found = stat($dir . "/" . $inc);
56 }
57 if (!$found) {
58 printf STDERR "$filename:$lineno: included file '$inc' is not exported\n";
59 $ret = 1;
60 }
61 }
62}
Mike Frysinger46b8af52008-12-27 02:43:36 -050063
Amerigo Wang67b7ebe2009-06-04 22:12:01 -040064sub check_declarations
Mike Frysinger46b8af52008-12-27 02:43:36 -050065{
Amerigo Wang67b7ebe2009-06-04 22:12:01 -040066 if ($line =~m/^\s*extern\b/) {
67 printf STDERR "$filename:$lineno: " .
68 "userspace cannot call function or variable " .
69 "defined in the kernel\n";
Mike Frysinger46b8af52008-12-27 02:43:36 -050070 }
71}
Sam Ravnborg7e557a22008-12-27 19:52:20 +010072
73sub check_config
74{
Robert P. J. Day1581c1c2009-05-12 13:43:36 -070075 if ($line =~ m/[^a-zA-Z0-9_]+CONFIG_([a-zA-Z0-9_]+)[^a-zA-Z0-9_]/) {
Sam Ravnborg7e557a22008-12-27 19:52:20 +010076 printf STDERR "$filename:$lineno: leaks CONFIG_$1 to userspace where it is not valid\n";
77 }
78}
79
Sam Ravnborg483b4122008-12-30 11:34:58 +010080my $linux_asm_types;
81sub check_asm_types()
82{
Sam Ravnborgb67ff8c2008-12-31 09:32:30 +010083 if ($filename =~ /types.h|int-l64.h|int-ll64.h/o) {
84 return;
85 }
Sam Ravnborg483b4122008-12-30 11:34:58 +010086 if ($lineno == 1) {
87 $linux_asm_types = 0;
88 } elsif ($linux_asm_types >= 1) {
89 return;
90 }
91 if ($line =~ m/^\s*#\s*include\s+<asm\/types.h>/) {
92 $linux_asm_types = 1;
93 printf STDERR "$filename:$lineno: " .
94 "include of <linux/types.h> is preferred over <asm/types.h>\n"
95 # Warn until headers are all fixed
96 #$ret = 1;
97 }
98}
99
100my $linux_types;
101sub check_sizetypes
102{
Sam Ravnborgb67ff8c2008-12-31 09:32:30 +0100103 if ($filename =~ /types.h|int-l64.h|int-ll64.h/o) {
104 return;
105 }
Sam Ravnborg483b4122008-12-30 11:34:58 +0100106 if ($lineno == 1) {
107 $linux_types = 0;
108 } elsif ($linux_types >= 1) {
109 return;
110 }
111 if ($line =~ m/^\s*#\s*include\s+<linux\/types.h>/) {
112 $linux_types = 1;
113 return;
114 }
115 if ($line =~ m/__[us](8|16|32|64)\b/) {
116 printf STDERR "$filename:$lineno: " .
117 "found __[us]{8,16,32,64} type " .
118 "without #include <linux/types.h>\n";
119 $linux_types = 2;
120 # Warn until headers are all fixed
121 #$ret = 1;
122 }
123}
124