diff options
| author | Jim Cromie <jim.cromie@gmail.com> | 2011-05-23 12:44:57 -0600 |
|---|---|---|
| committer | Michal Marek <mmarek@suse.cz> | 2011-05-24 16:07:07 +0200 |
| commit | ca995cbf77f3df599b7e751c2d08d90787c65c45 (patch) | |
| tree | 3004eea8ebd0caada01d4e012d95810249ea0b20 /scripts | |
| parent | bdabc7a345db97b3839c2c3deef40023cf8017ef (diff) | |
export_report: use warn() to issue WARNING, so they go to stderr
Also count CONFIG_MODVERSIONS warnings, and print a NOTE at start of
SECTION 2 if any were issued. Section 2 will be empty if the build is
lacking this CONFIG_ item, and user may have missed the warnings, as
they're off screen.
Signed-off-by: Jim Cromie <jim.cromie@gmail.com>
Signed-off-by: Michal Marek <mmarek@suse.cz>
Diffstat (limited to 'scripts')
| -rw-r--r-- | scripts/export_report.pl | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/scripts/export_report.pl b/scripts/export_report.pl index 48398a19fbb..8f79b701de8 100644 --- a/scripts/export_report.pl +++ b/scripts/export_report.pl @@ -102,6 +102,8 @@ close($module_symvers); # # collect the usage count of each symbol. # +my $modversion_warnings = 0; + foreach my $thismod (@allcfiles) { my $module; @@ -132,7 +134,8 @@ foreach my $thismod (@allcfiles) { } } if ($state != 2) { - print "WARNING:$thismod is not built with CONFIG_MODVERSION enabled\n"; + warn "WARNING:$thismod is not built with CONFIG_MODVERSIONS enabled\n"; + $modversion_warnings++; } close($module); } @@ -166,6 +169,9 @@ printf("SECTION 2:\n\tThis section reports export-symbol-usage of in-kernel modules. Each module lists the modules, and the symbols from that module that it uses. Each listed symbol reports the number of modules using it\n"); +print "\nNOTE: Got $modversion_warnings CONFIG_MODVERSIONS warnings\n\n" + if $modversion_warnings; + print "~"x80 , "\n"; for my $thismod (sort keys %MODULE) { my $list = $MODULE{$thismod}; |