<feed xmlns='http://www.w3.org/2005/Atom'>
<title>user/sven/linux.git/scripts/headers_check.pl, branch v3.2.54</title>
<subtitle>Linux Kernel
</subtitle>
<id>https://git.stealer.net/cgit.cgi/user/sven/linux.git/atom?h=v3.2.54</id>
<link rel='self' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/atom?h=v3.2.54'/>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/'/>
<updated>2010-12-14T16:05:08Z</updated>
<entry>
<title>headers_check: Fix warning text</title>
<updated>2010-12-14T16:05:08Z</updated>
<author>
<name>akpm@linux-foundation.org</name>
<email>akpm@linux-foundation.org</email>
</author>
<published>2010-11-30T21:52:14Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=d52784eb3607bf887628742f99041b4f18d7d1de'/>
<id>urn:sha1:d52784eb3607bf887628742f99041b4f18d7d1de</id>
<content type='text'>
Fix the warning text too, per Randy.

Cc: Alexander Shishkin &lt;virtuoso@slind.org&gt;
Cc: Michal Marek &lt;mmarek@suse.cz&gt;
Cc: Randy Dunlap &lt;randy.dunlap@oracle.com&gt;
Cc: Stephen Hemminger &lt;shemminger@vyatta.com&gt;
Cc: WANG Cong &lt;amwang@redhat.com&gt;
Signed-off-by: Andrew Morton &lt;akpm@linux-foundation.org&gt;
Signed-off-by: Michal Marek &lt;mmarek@suse.cz&gt;
</content>
</entry>
<entry>
<title>headers_check: better search for functions in headers</title>
<updated>2010-12-14T16:02:45Z</updated>
<author>
<name>akpm@linux-foundation.org</name>
<email>akpm@linux-foundation.org</email>
</author>
<published>2010-11-30T21:51:13Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=de323f22a83b024b7432e813609c6efb74b1bbfc'/>
<id>urn:sha1:de323f22a83b024b7432e813609c6efb74b1bbfc</id>
<content type='text'>
Some headers don't bother with "extern" in function prototypes, which
results in said prototypes being unnoticed and exported to userland.

This patch slightly improves detection of such cases by checking for C
type names as well in the beginning of a line.

Signed-off-by: Alexander Shishkin &lt;virtuoso@slind.org&gt;
Cc: Stephen Hemminger &lt;shemminger@vyatta.com&gt;
Cc: Michal Marek &lt;mmarek@suse.cz&gt;
Cc: WANG Cong &lt;amwang@redhat.com&gt;
Cc: Randy Dunlap &lt;randy.dunlap@oracle.com&gt;
Signed-off-by: Andrew Morton &lt;akpm@linux-foundation.org&gt;
Signed-off-by: Michal Marek &lt;mmarek@suse.cz&gt;
</content>
</entry>
<entry>
<title>headers_check: fix perl warnings</title>
<updated>2010-03-07T20:43:07Z</updated>
<author>
<name>Stephen Hemminger</name>
<email>shemminger@vyatta.com</email>
</author>
<published>2010-02-22T23:17:24Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=dbbe33e99f41a6f07e61dbce455964112d8ac72b'/>
<id>urn:sha1:dbbe33e99f41a6f07e61dbce455964112d8ac72b</id>
<content type='text'>
According to PBP; best way practice is to use local reference for file
handle and three argument open. Also perl prototypes are a mistake.

Signed-off-by: Stephen Hemminger &lt;shemminger@vyatta.com&gt;
Acked-by: WANG Cong &lt;amwang@redhat.com&gt;
Cc: Michal Marek &lt;mmarek@suse.cz&gt;
Signed-off-by: Andrew Morton &lt;akpm@linux-foundation.org&gt;
Signed-off-by: Michal Marek &lt;mmarek@suse.cz&gt;
</content>
</entry>
<entry>
<title>kbuild/headers_check: refine extern check</title>
<updated>2009-06-09T20:37:56Z</updated>
<author>
<name>Amerigo Wang</name>
<email>amwang@redhat.com</email>
</author>
<published>2009-06-05T02:12:01Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=67b7ebe091cd92fd69f732da3170720d79c4e632'/>
<id>urn:sha1:67b7ebe091cd92fd69f732da3170720d79c4e632</id>
<content type='text'>
'extern' checking information is not clear, refine it.
Plus, fix a comment.

Signed-off-by: WANG Cong &lt;amwang@redhat.com&gt;
[sam: redid the extern error message]
Acked-by: Arnd Bergmann &lt;arnd@arndb.de&gt;
Signed-off-by: Sam Ravnborg &lt;sam@ravnborg.org&gt;
</content>
</entry>
<entry>
<title>scripts/headers_check.pl: correct RE in header CONFIG leak check</title>
<updated>2009-06-09T20:37:52Z</updated>
<author>
<name>Robert P. J. Day</name>
<email>rpjday@crashcourse.ca</email>
</author>
<published>2009-05-12T20:43:36Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=1581c1cede3b180b77aa08024c6ca406252cc21f'/>
<id>urn:sha1:1581c1cede3b180b77aa08024c6ca406252cc21f</id>
<content type='text'>
Correct the regular expression in scripts/headers_check.pl to include '_'
as a valid character in the class; otherwise, the check will report a
"leaked" symbol of CONFIG_A_B_C as merely CONFIG_A.

This patch will make no difference whatsoever in the current kernel tree
as the call to the perl routine that does that check is currently
commented out:

                &amp;check_include();
                &amp;check_asm_types();
                &amp;check_sizetypes();
                &amp;check_prototypes();
                # Dropped for now. Too much noise &amp;check_config();

However, I noticed that problem when I was building the yum downloadable
kernel source rpm for fedora 11 (beta), which *does* run that check, and
that's where the problem became obvious.

Signed-off-by: Robert P. J. Day &lt;rpjday@crashcourse.ca&gt;
Cc: David Woodhouse &lt;dwmw2@infradead.org&gt;
Signed-off-by: Andrew Morton &lt;akpm@linux-foundation.org&gt;
Signed-off-by: Sam Ravnborg &lt;sam@ravnborg.org&gt;
</content>
</entry>
<entry>
<title>kbuild: drop check for CONFIG_ in headers_check</title>
<updated>2009-01-30T23:36:15Z</updated>
<author>
<name>Sam Ravnborg</name>
<email>sam@ravnborg.org</email>
</author>
<published>2009-01-30T22:56:42Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=7e3fa56141175026136b392fd026d5d07c49720e'/>
<id>urn:sha1:7e3fa56141175026136b392fd026d5d07c49720e</id>
<content type='text'>
The check for references to CONFIG_ symbols in exported headers turned
out to be too agressive with the current state of affairs.
After the work of Jaswinder to clean up all relevant cases we are down
to almost pure noise.

So lets drop the check for now - we can always add it back later
should our headers be ready for that.

Signed-off-by: Sam Ravnborg &lt;sam@ravnborg.org&gt;
Signed-off-by: Ingo Molnar &lt;mingo@elte.hu&gt;
</content>
</entry>
<entry>
<title>kbuild: ignore a few files in headers_check</title>
<updated>2009-01-02T19:43:27Z</updated>
<author>
<name>Sam Ravnborg</name>
<email>sam@ravnborg.org</email>
</author>
<published>2008-12-31T08:32:30Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=b67ff8ce122f3353bd741db48ce1756c12fb5f2d'/>
<id>urn:sha1:b67ff8ce122f3353bd741db48ce1756c12fb5f2d</id>
<content type='text'>
The new check for asm/types.h and linux/types.h had
a few false positives.

o We cannot let linux/types.h include linux/types.h
o The int-ll64.h and int-ll64.h define the types
  and are included by linux/types.h

Handle this by hardcoding the filenames in the headers_check script.

Signed-off-by: Sam Ravnborg &lt;sam@ravnborg.org&gt;
</content>
</entry>
<entry>
<title>kbuild: add checks for include of linux/types in userspace headers</title>
<updated>2009-01-02T19:43:26Z</updated>
<author>
<name>Sam Ravnborg</name>
<email>sam@ravnborg.org</email>
</author>
<published>2008-12-30T10:34:58Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=483b41218fa9d5172312a9e294aaf78e22b266e6'/>
<id>urn:sha1:483b41218fa9d5172312a9e294aaf78e22b266e6</id>
<content type='text'>
If we see __[us](8|16|32|64) then we must include &lt;linux/types.h&gt;
If wee see include of &lt;asm/types.h&gt; then we recommend &lt;linux/types.h&gt;

Original script from Mike but modified by me.

Cc: Mike Frysinger &lt;vapier@gentoo.org&gt;
Signed-off-by: Sam Ravnborg &lt;sam@ravnborg.org&gt;
</content>
</entry>
<entry>
<title>kbuild: check for leaked CONFIG_ symbols to userspace</title>
<updated>2009-01-02T19:43:23Z</updated>
<author>
<name>Sam Ravnborg</name>
<email>sam@ravnborg.org</email>
</author>
<published>2008-12-27T18:52:20Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=7e557a2509f9e1477c10295b74e29e4e93fa2392'/>
<id>urn:sha1:7e557a2509f9e1477c10295b74e29e4e93fa2392</id>
<content type='text'>
Signed-off-by: Sam Ravnborg &lt;sam@ravnborg.org&gt;
</content>
</entry>
<entry>
<title>headers_check.pl: disallow extern's</title>
<updated>2009-01-02T19:43:22Z</updated>
<author>
<name>Mike Frysinger</name>
<email>vapier@gentoo.org</email>
</author>
<published>2008-12-27T07:43:36Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=46b8af50ba5c072b74740c5fa8ba08e6eabb22f8'/>
<id>urn:sha1:46b8af50ba5c072b74740c5fa8ba08e6eabb22f8</id>
<content type='text'>
Since prototypes with "extern" refer to kernel functions, they make no
sense in userspace, so reject them automatically.

Signed-off-by: Mike Frysinger &lt;vapier@gentoo.org&gt;
[sam: made it into a warning]
Signed-off-by: Sam Ravnborg &lt;sam@ravnborg.org&gt;
</content>
</entry>
</feed>
