From: David Woodhouse <dwmw2@infradead.org>
To: Khem Raj <raj.khem@gmail.com>
Cc: k.shutemov@gmail.com, linux-kernel@vger.kernel.org, avi@qumranet.com
Subject: Re: linux/a.out.h is not exported to userspace
Date: Sun, 18 May 2008 12:16:32 +0100 [thread overview]
Message-ID: <1211109392.3008.186.camel@pmac.infradead.org> (raw)
In-Reply-To: <1210983237.8550.28.camel@isis>
On Fri, 2008-05-16 at 17:13 -0700, Khem Raj wrote:
> Hi,
>
> Looking for a compile failure for a package on uclibc system using
> kernel-headers from 2.6.25 I found that linux/a.out.h is no more
> exported to userland anymore.
>
> This commit below has disabled it from being exported. The commit Do not
> export asm/page.h during make headers_install so I was wondering if it
> was intended to unexport linux/a.out.h or not.
>
> http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commitdiff;h=ed7b1889da256977574663689b598d88950bbd23
>
>
> Looking at the patch from lkml post
>
> http://linux.derkeiler.com/Mailing-Lists/Kernel/2007-10/msg08913.html
>
> second hunk to include/linux/Kbuild was not applied which caused this
> problem.
>
> I think I do not fall into Signed-off-by definition. I tested the
> following patch on 2.6.25.
>
> Thanks
>
> -Khem
>
> Tested-by: Khem Raj <raj.khem@gmail.com>
>
> diff --git a/include/linux/Kbuild b/include/linux/Kbuild
> index b7d81b2..8a3d93e 100644
> --- a/include/linux/Kbuild
> +++ b/include/linux/Kbuild
> @@ -168,6 +168,7 @@ unifdef-y += acct.h
> unifdef-y += adb.h
> unifdef-y += adfs_fs.h
> unifdef-y += agpgart.h
> +unifdef-y += a.out.h
> unifdef-y += apm_bios.h
> unifdef-y += atalk.h
> unifdef-y += atmdev.h
That'll export <linux/a.out.h> even on architectures where <asm/a.out.h>
doesn't exist -- which will cause headers_check to fail.
Perhaps we could do it something like this:
--- a/include/linux/Kbuild
+++ b/include/linux/Kbuild
@@ -168,6 +168,9 @@ unifdef-y += acct.h
unifdef-y += adb.h
unifdef-y += adfs_fs.h
unifdef-y += agpgart.h
+ifeq ($(wildcard include/asm-$(SRCARCH)/a.out.h),include/asm-$(SRCARCH)/a.out.h)
+unifdef-y += a.out.h
+endif
unifdef-y += apm_bios.h
unifdef-y += atalk.h
unifdef-y += atmdev.h
In fact, perhaps we could do <linux/kvm.h> that way too -- I believe I
objected to using a CONFIG symbol, but the mere existence of the file
would be fine. I don't think we thought of that at the time.
--
dwmw2
next prev parent reply other threads:[~2008-05-18 11:16 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-05-17 0:13 linux/a.out.h is not exported to userspace Khem Raj
2008-05-17 13:29 ` Stefan Hellermann
2008-05-17 15:17 ` Pekka Enberg
2008-05-18 11:16 ` David Woodhouse [this message]
2008-05-23 7:32 ` Andrew Morton
2008-05-23 16:02 ` Khem Raj
2008-05-30 4:35 ` Khem Raj
2008-05-30 5:45 ` David Woodhouse
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1211109392.3008.186.camel@pmac.infradead.org \
--to=dwmw2@infradead.org \
--cc=avi@qumranet.com \
--cc=k.shutemov@gmail.com \
--cc=linux-kernel@vger.kernel.org \
--cc=raj.khem@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox