Linux Audio

Check our new training course

Loading...
v5.4
  1#!/usr/bin/env perl
  2# SPDX-License-Identifier: GPL-2.0
  3#
  4# headers_check.pl execute a number of trivial consistency checks
  5#
  6# Usage: headers_check.pl dir arch [files...]
  7# dir:   dir to look for included files
  8# arch:  architecture
  9# files: list of files to check
 10#
 11# The script reads the supplied files line by line and:
 12#
 13# 1) for each include statement it checks if the
 14#    included file actually exists.
 15#    Only include files located in asm* and linux* are checked.
 16#    The rest are assumed to be system include files.
 17#
 18# 2) It is checked that prototypes does not use "extern"
 19#
 20# 3) Check for leaked CONFIG_ symbols
 21
 22use warnings;
 23use strict;
 24use File::Basename;
 25
 26my ($dir, $arch, @files) = @ARGV;
 27
 28my $ret = 0;
 29my $line;
 30my $lineno = 0;
 31my $filename;
 32
 33foreach my $file (@files) {
 34	$filename = $file;
 35
 36	open(my $fh, '<', $filename)
 37		or die "$filename: $!\n";
 38	$lineno = 0;
 39	while ($line = <$fh>) {
 40		$lineno++;
 41		&check_include();
 42		&check_asm_types();
 43		&check_sizetypes();
 44		&check_declarations();
 45		# Dropped for now. Too much noise &check_config();
 46	}
 47	close $fh;
 48}
 49exit $ret;
 50
 51sub check_include
 52{
 53	if ($line =~ m/^\s*#\s*include\s+<((asm|linux).*)>/) {
 54		my $inc = $1;
 55		my $found;
 56		$found = stat($dir . "/" . $inc);
 57		if (!$found) {
 58			$inc =~ s#asm/#asm-$arch/#;
 59			$found = stat($dir . "/" . $inc);
 60		}
 61		if (!$found) {
 62			printf STDERR "$filename:$lineno: included file '$inc' is not exported\n";
 63			$ret = 1;
 64		}
 65	}
 66}
 67
 68sub check_declarations
 69{
 70	# soundcard.h is what it is
 71	if ($line =~ m/^void seqbuf_dump\(void\);/) {
 72		return;
 73	}
 74	# drm headers are being C++ friendly
 75	if ($line =~ m/^extern "C"/) {
 76		return;
 77	}
 78	if ($line =~ m/^(\s*extern|unsigned|char|short|int|long|void)\b/) {
 79		printf STDERR "$filename:$lineno: " .
 80			      "userspace cannot reference function or " .
 81			      "variable defined in the kernel\n";
 82	}
 83}
 84
 85sub check_config
 86{
 87	if ($line =~ m/[^a-zA-Z0-9_]+CONFIG_([a-zA-Z0-9_]+)[^a-zA-Z0-9_]/) {
 88		printf STDERR "$filename:$lineno: leaks CONFIG_$1 to userspace where it is not valid\n";
 89	}
 90}
 91
 92my $linux_asm_types;
 93sub check_asm_types
 94{
 95	if ($filename =~ /types.h|int-l64.h|int-ll64.h/o) {
 96		return;
 97	}
 98	if ($lineno == 1) {
 99		$linux_asm_types = 0;
100	} elsif ($linux_asm_types >= 1) {
101		return;
102	}
103	if ($line =~ m/^\s*#\s*include\s+<asm\/types.h>/) {
104		$linux_asm_types = 1;
105		printf STDERR "$filename:$lineno: " .
106		"include of <linux/types.h> is preferred over <asm/types.h>\n"
107		# Warn until headers are all fixed
108		#$ret = 1;
109	}
110}
111
112my $linux_types;
113my %import_stack = ();
114sub check_include_typesh
115{
116	my $path = $_[0];
117	my $import_path;
118
119	my $fh;
120	my @file_paths = ($path, $dir . "/" .  $path, dirname($filename) . "/" . $path);
121	for my $possible ( @file_paths ) {
122	    if (not $import_stack{$possible} and open($fh, '<', $possible)) {
123		$import_path = $possible;
124		$import_stack{$import_path} = 1;
125		last;
126	    }
127	}
128	if (eof $fh) {
129	    return;
130	}
131
132	my $line;
133	while ($line = <$fh>) {
134		if ($line =~ m/^\s*#\s*include\s+<linux\/types.h>/) {
135			$linux_types = 1;
136			last;
137		}
138		if (my $included = ($line =~ /^\s*#\s*include\s+[<"](\S+)[>"]/)[0]) {
139			check_include_typesh($included);
140		}
141	}
142	close $fh;
143	delete $import_stack{$import_path};
144}
145
146sub check_sizetypes
147{
148	if ($filename =~ /types.h|int-l64.h|int-ll64.h/o) {
149		return;
150	}
151	if ($lineno == 1) {
152		$linux_types = 0;
153	} elsif ($linux_types >= 1) {
154		return;
155	}
156	if ($line =~ m/^\s*#\s*include\s+<linux\/types.h>/) {
157		$linux_types = 1;
158		return;
159	}
160	if (my $included = ($line =~ /^\s*#\s*include\s+[<"](\S+)[>"]/)[0]) {
161		check_include_typesh($included);
162	}
163	if ($line =~ m/__[us](8|16|32|64)\b/) {
164		printf STDERR "$filename:$lineno: " .
165		              "found __[us]{8,16,32,64} type " .
166		              "without #include <linux/types.h>\n";
167		$linux_types = 2;
168		# Warn until headers are all fixed
169		#$ret = 1;
170	}
171}
v3.15
  1#!/usr/bin/perl -w
 
  2#
  3# headers_check.pl execute a number of trivial consistency checks
  4#
  5# Usage: headers_check.pl dir arch [files...]
  6# 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#
 17# 2) It is checked that prototypes does not use "extern"
 18#
 19# 3) Check for leaked CONFIG_ symbols
 20
 
 21use strict;
 22use File::Basename;
 23
 24my ($dir, $arch, @files) = @ARGV;
 25
 26my $ret = 0;
 27my $line;
 28my $lineno = 0;
 29my $filename;
 30
 31foreach my $file (@files) {
 32	$filename = $file;
 33
 34	open(my $fh, '<', $filename)
 35		or die "$filename: $!\n";
 36	$lineno = 0;
 37	while ($line = <$fh>) {
 38		$lineno++;
 39		&check_include();
 40		&check_asm_types();
 41		&check_sizetypes();
 42		&check_declarations();
 43		# Dropped for now. Too much noise &check_config();
 44	}
 45	close $fh;
 46}
 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}
 65
 66sub check_declarations
 67{
 68	# 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/) {
 73		printf STDERR "$filename:$lineno: " .
 74			      "userspace cannot reference function or " .
 75			      "variable defined in the kernel\n";
 76	}
 77}
 78
 79sub check_config
 80{
 81	if ($line =~ m/[^a-zA-Z0-9_]+CONFIG_([a-zA-Z0-9_]+)[^a-zA-Z0-9_]/) {
 82		printf STDERR "$filename:$lineno: leaks CONFIG_$1 to userspace where it is not valid\n";
 83	}
 84}
 85
 86my $linux_asm_types;
 87sub check_asm_types
 88{
 89	if ($filename =~ /types.h|int-l64.h|int-ll64.h/o) {
 90		return;
 91	}
 92	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;
107my %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
140sub check_sizetypes
141{
142	if ($filename =~ /types.h|int-l64.h|int-ll64.h/o) {
143		return;
144	}
145	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	}
154	if (my $included = ($line =~ /^\s*#\s*include\s+[<"](\S+)[>"]/)[0]) {
155		check_include_typesh($included);
156	}
157	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}