linux-toradex.git/scripts/headers_check.pl, branch v4.11-rc1

headers_check: don't warn about c++ guards

2016-05-24T04:12:48+00:00

A recent addition to the DRM tree for 4.7 added 'extern "C"' guards
for c++ to all the DRM headers, and that now causes warnings
in 'make headers_check':

usr/include/drm/amdgpu_drm.h:38: userspace cannot reference function or variable defined in the kernel
usr/include/drm/drm.h:63: userspace cannot reference function or variable defined in the kernel
usr/include/drm/drm.h:699: userspace cannot reference function or variable defined in the kernel
usr/include/drm/drm_fourcc.h:30: userspace cannot reference function or variable defined in the kernel
usr/include/drm/drm_mode.h:33: userspace cannot reference function or variable defined in the kernel
usr/include/drm/drm_sarea.h:38: userspace cannot reference function or variable defined in the kernel
usr/include/drm/exynos_drm.h:21: userspace cannot reference function or variable defined in the kernel
usr/include/drm/i810_drm.h:7: userspace cannot reference function or variable defined in the kernel

This changes the headers_check.pl script to not warn about this.
I'm listing the merge commit as introducing the problem, because
there are several patches in this branch that each do this for
one file.

Signed-off-by: Arnd Bergmann 
Fixes: 7c10ddf87472 ("Merge branch 'drm-uapi-extern-c-fixes' of https://github.com/evelikov/linux into drm-next")
Reviewed-by: Emil Velikov 
Signed-off-by: Dave Airlie

kbuild: Make scripts executable

2014-08-20T14:03:45+00:00

The Makefiles call the respective interpreter explicitly, but this makes
it easier to use the scripts manually.

Signed-off-by: Michal Marek

headers_check: special case seqbuf_dump()

2014-01-24T00:36:55+00:00

"make headers_check" warns about soundcard.h for (at least) five years
now:
    [...]/usr/include/linux/soundcard.h:1054: userspace cannot reference function or variable defined in the kernel

We're apparently stuck with providing OSSlib-3.8 compatibility, so let's
special case this declaration just to silence it.

Notes:

0) Support for OSSlib post 3.8 was already removed in commit 43a990765a
   ("sound: Remove OSSlib stuff from linux/soundcard.h").  Five years have
   passed since that commit: do people still care about OSSlib-3.8?  If
   not, quite a bit of code could be remove from soundcard.h (and probably
   ultrasound.h).

2) By the way, what is actually meant by:
    It is no longer possible to actually link against OSSlib with this
    header, but we still provide these macros for programs using them.

Doesn't that mean compatibility to OSSlib isn't even useful?

3) Anyhow, a previous discussion soundcard.h, which led to that commit,
   starts at https://lkml.org/lkml/2009/1/20/349 .

4) And, yes, I sneaked in a whitespace fix.

Signed-off-by: Paul Bolle 
Cc: Takashi Iwai 
Acked-by: Arnd Bergmann 
Cc: Michal Marek 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

headers_check: recursively search for linux/types.h inclusion

2012-03-26T12:54:27+00:00

headers_check.pl currently emits some spurious warnings, especially for
the drm headers, about using __[us]{8,16,32,64} types without including
linux/types.h.  Recursively search for types.h inclusion, avoiding
circular references.

Signed-off-by: Bobby Powers 
Cc: Sam Ravnborg 
Cc: Dave Airlie 
Signed-off-by: Andrew Morton 
Signed-off-by: Michal Marek

headers_check: Fix warning text

2010-12-14T16:05:08+00:00

Fix the warning text too, per Randy.

Cc: Alexander Shishkin 
Cc: Michal Marek 
Cc: Randy Dunlap 
Cc: Stephen Hemminger 
Cc: WANG Cong 
Signed-off-by: Andrew Morton 
Signed-off-by: Michal Marek

headers_check: better search for functions in headers

2010-12-14T16:02:45+00:00

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 
Cc: Stephen Hemminger 
Cc: Michal Marek 
Cc: WANG Cong 
Cc: Randy Dunlap 
Signed-off-by: Andrew Morton 
Signed-off-by: Michal Marek

headers_check: fix perl warnings

2010-03-07T20:43:07+00:00

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 
Acked-by: WANG Cong 
Cc: Michal Marek 
Signed-off-by: Andrew Morton 
Signed-off-by: Michal Marek

kbuild/headers_check: refine extern check

2009-06-09T20:37:56+00:00

'extern' checking information is not clear, refine it.
Plus, fix a comment.

Signed-off-by: WANG Cong 
[sam: redid the extern error message]
Acked-by: Arnd Bergmann 
Signed-off-by: Sam Ravnborg

scripts/headers_check.pl: correct RE in header CONFIG leak check

2009-06-09T20:37:52+00:00

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:

                &check_include();
                &check_asm_types();
                &check_sizetypes();
                &check_prototypes();
                # Dropped for now. Too much noise &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 
Cc: David Woodhouse 
Signed-off-by: Andrew Morton 
Signed-off-by: Sam Ravnborg

kbuild: drop check for CONFIG_ in headers_check

2009-01-30T23:36:15+00:00

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 
Signed-off-by: Ingo Molnar