| Message ID | 1463587653-3035181-2-git-send-email-arnd@arndb.de (mailing list archive) |
|---|---|
| State | Not Applicable |
| Headers | show |
On 18 May 2016 at 17:07, Arnd Bergmann <arnd@arndb.de> wrote: > 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 <arnd@arndb.de> > Fixes: 7c10ddf87472 ("Merge branch 'drm-uapi-extern-c-fixes' of https://github.com/evelikov/linux into drm-next") Reviewed-by: Emil Velikov <emil.l.velikov@gmail.com> Thanks Arnd ! As Dave mentioned - the best solution would be to have the hunks generated on the fly. Sadly Perl and me don't go hand in hand so if you're interested I'll be really grateful. -Emil -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/scripts/headers_check.pl b/scripts/headers_check.pl index 62320f93e903..8b2da054cdc3 100755 --- a/scripts/headers_check.pl +++ b/scripts/headers_check.pl @@ -69,6 +69,10 @@ sub check_declarations if ($line =~ m/^void seqbuf_dump\(void\);/) { return; } + # drm headers are being C++ friendly + if ($line =~ m/^extern "C"/) { + return; + } if ($line =~ m/^(\s*extern|unsigned|char|short|int|long|void)\b/) { printf STDERR "$filename:$lineno: " . "userspace cannot reference function or " .
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 <arnd@arndb.de> Fixes: 7c10ddf87472 ("Merge branch 'drm-uapi-extern-c-fixes' of https://github.com/evelikov/linux into drm-next") --- scripts/headers_check.pl | 4 ++++ 1 file changed, 4 insertions(+)