blob: 62320f93e903edeceab7c7e621b750647d0ca75a [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;
Bobby Powersf75a8df2012-03-05 15:08:09 -080022use File::Basename;
Sam Ravnborg77124012008-06-15 21:41:09 +020023
24my ($dir, $arch, @files) = @ARGV;
25
26my $ret = 0;
27my $line;
28my $lineno = 0;
29my $filename;
30
31foreach my $file (@files) {
32 $filename = $file;
Stephen Hemmingerdbbe33e2010-02-22 15:17:24 -080033
34 open(my $fh, '<', $filename)
35 or die "$filename: $!\n";
Sam Ravnborg77124012008-06-15 21:41:09 +020036 $lineno = 0;
Stephen Hemmingerdbbe33e2010-02-22 15:17:24 -080037 while ($line = <$fh>) {
Sam Ravnborg77124012008-06-15 21:41:09 +020038 $lineno++;
Sam Ravnborg483b4122008-12-30 11:34:58 +010039 &check_include();
40 &check_asm_types();
41 &check_sizetypes();
Amerigo Wang67b7ebe2009-06-04 22:12:01 -040042 &check_declarations();
Sam Ravnborg7e3fa562009-01-30 23:56:42 +010043 # Dropped for now. Too much noise &check_config();
Sam Ravnborg77124012008-06-15 21:41:09 +020044 }
Stephen Hemmingerdbbe33e2010-02-22 15:17:24 -080045 close $fh;
Sam Ravnborg77124012008-06-15 21:41:09 +020046}
47exit $ret;
48
49sub check_include
50{
51 if ($line =~ m/^\s*#\s*include\s+<((asm|linux).*)>/) {
52 my $inc = $1;
53 my $found;
54 $found = stat($dir . "/" . $inc);
55 if (!$found) {
56 $inc =~ s#asm/#asm-$arch/#;
57 $found = stat($dir . "/" . $inc);
58 }
59 if (!$found) {
60 printf STDERR "$filename:$lineno: included file '$inc' is not exported\n";
61 $ret = 1;
62 }
63 }
64}
Mike Frysinger46b8af52008-12-27 02:43:36 -050065
Amerigo Wang67b7ebe2009-06-04 22:12:01 -040066sub check_declarations
Mike Frysinger46b8af52008-12-27 02:43:36 -050067{
Paul Bollea7e1d982014-01-23 15:54:08 -080068 # soundcard.h is what it is
69 if ($line =~ m/^void seqbuf_dump\(void\);/) {
70 return;
71 }
72 if ($line =~ m/^(\s*extern|unsigned|char|short|int|long|void)\b/) {
Amerigo Wang67b7ebe2009-06-04 22:12:01 -040073 printf STDERR "$filename:$lineno: " .
akpm@linux-foundation.orgd52784e2010-11-30 13:52:14 -080074 "userspace cannot reference function or " .
75 "variable defined in the kernel\n";
Mike Frysinger46b8af52008-12-27 02:43:36 -050076 }
77}
Sam Ravnborg7e557a22008-12-27 19:52:20 +010078
79sub check_config
80{
Robert P. J. Day1581c1c2009-05-12 13:43:36 -070081 if ($line =~ m/[^a-zA-Z0-9_]+CONFIG_([a-zA-Z0-9_]+)[^a-zA-Z0-9_]/) {
Sam Ravnborg7e557a22008-12-27 19:52:20 +010082 printf STDERR "$filename:$lineno: leaks CONFIG_$1 to userspace where it is not valid\n";
83 }
84}
85
Sam Ravnborg483b4122008-12-30 11:34:58 +010086my $linux_asm_types;
Stephen Hemmingerdbbe33e2010-02-22 15:17:24 -080087sub check_asm_types
Sam Ravnborg483b4122008-12-30 11:34:58 +010088{
Sam Ravnborgb67ff8c2008-12-31 09:32:30 +010089 if ($filename =~ /types.h|int-l64.h|int-ll64.h/o) {
90 return;
91 }
Sam Ravnborg483b4122008-12-30 11:34:58 +010092 if ($lineno == 1) {
93 $linux_asm_types = 0;
94 } elsif ($linux_asm_types >= 1) {
95 return;
96 }
97 if ($line =~ m/^\s*#\s*include\s+<asm\/types.h>/) {
98 $linux_asm_types = 1;
99 printf STDERR "$filename:$lineno: " .
100 "include of <linux/types.h> is preferred over <asm/types.h>\n"
101 # Warn until headers are all fixed
102 #$ret = 1;
103 }
104}
105
106my $linux_types;
Bobby Powersf75a8df2012-03-05 15:08:09 -0800107my %import_stack = ();
108sub check_include_typesh
109{
110 my $path = $_[0];
111 my $import_path;
112
113 my $fh;
114 my @file_paths = ($path, $dir . "/" . $path, dirname($filename) . "/" . $path);
115 for my $possible ( @file_paths ) {
116 if (not $import_stack{$possible} and open($fh, '<', $possible)) {
117 $import_path = $possible;
118 $import_stack{$import_path} = 1;
119 last;
120 }
121 }
122 if (eof $fh) {
123 return;
124 }
125
126 my $line;
127 while ($line = <$fh>) {
128 if ($line =~ m/^\s*#\s*include\s+<linux\/types.h>/) {
129 $linux_types = 1;
130 last;
131 }
132 if (my $included = ($line =~ /^\s*#\s*include\s+[<"](\S+)[>"]/)[0]) {
133 check_include_typesh($included);
134 }
135 }
136 close $fh;
137 delete $import_stack{$import_path};
138}
139
Sam Ravnborg483b4122008-12-30 11:34:58 +0100140sub check_sizetypes
141{
Sam Ravnborgb67ff8c2008-12-31 09:32:30 +0100142 if ($filename =~ /types.h|int-l64.h|int-ll64.h/o) {
143 return;
144 }
Sam Ravnborg483b4122008-12-30 11:34:58 +0100145 if ($lineno == 1) {
146 $linux_types = 0;
147 } elsif ($linux_types >= 1) {
148 return;
149 }
150 if ($line =~ m/^\s*#\s*include\s+<linux\/types.h>/) {
151 $linux_types = 1;
152 return;
153 }
Bobby Powersf75a8df2012-03-05 15:08:09 -0800154 if (my $included = ($line =~ /^\s*#\s*include\s+[<"](\S+)[>"]/)[0]) {
155 check_include_typesh($included);
156 }
Sam Ravnborg483b4122008-12-30 11:34:58 +0100157 if ($line =~ m/__[us](8|16|32|64)\b/) {
158 printf STDERR "$filename:$lineno: " .
159 "found __[us]{8,16,32,64} type " .
160 "without #include <linux/types.h>\n";
161 $linux_types = 2;
162 # Warn until headers are all fixed
163 #$ret = 1;
164 }
165}