From: Martin Schwidefsky <schwidefsky@de.ibm.com>
To: Heiko Carstens <heiko.carstens@de.ibm.com>
Cc: Christoph Hellwig <hch@lst.de>,
akpm@osdl.org, davem@davemloft.net, linux-kernel@vger.kernel.org
Subject: Re: [PATCH 2/3] disallow modular binfmt_elf32
Date: Mon, 10 Jul 2006 10:27:58 +0200 [thread overview]
Message-ID: <1152520078.5834.7.camel@localhost> (raw)
In-Reply-To: <20060710060603.GA9440@osiris.boeblingen.de.ibm.com>
On Mon, 2006-07-10 at 08:06 +0200, Heiko Carstens wrote:
> On Sat, Jul 08, 2006 at 08:05:54PM +0200, Christoph Hellwig wrote:
> > Currently most architectures either always build binfmt_elf32 in the
> > kernel image or make it a boolean option. Only sparc64 and s390 allow
> > to build it modularly. This patch turns the option into a boolean
> > aswell because elf requires various symbols that shouldn't be available
> > to modules. The most urgent one is tasklist_lock whos export this patch
> > series kills, but there are others like force_sgi aswell.
> > [...]
> > Index: linux-2.6/arch/s390/Kconfig
> > ===================================================================
> > --- linux-2.6.orig/arch/s390/Kconfig 2006-07-06 14:21:17.000000000 +0200
> > +++ linux-2.6/arch/s390/Kconfig 2006-07-08 19:08:46.000000000 +0200
> > @@ -119,7 +119,7 @@
> > default y
> >
> > config BINFMT_ELF32
> > - tristate "Kernel support for 31 bit ELF binaries"
> > + bool "Kernel support for 31 bit ELF binaries"
> > depends on COMPAT
> > help
>
> Martin and I discussed this already a few days ago. This config option
> should go away on s390, since everybody who wants CONFIG_COMPAT also wants
> CONFIG_BINFMT_ELF32. See patch below which applies on top of yours:
Yes, the removal of the BINFMT_ELF32 Kconfig option has been part of the
s390 Kconfig cleanup patch as well, it is supposed to go away.
--
blue skies,
Martin.
Martin Schwidefsky
Linux for zSeries Development & Services
IBM Deutschland Entwicklung GmbH
"Reality continues to ruin my life." - Calvin.
prev parent reply other threads:[~2006-07-10 8:27 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-07-08 18:05 [PATCH 2/3] disallow modular binfmt_elf32 Christoph Hellwig
2006-07-08 20:48 ` David Miller
2006-07-10 6:06 ` Heiko Carstens
2006-07-10 8:27 ` Martin Schwidefsky [this message]
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=1152520078.5834.7.camel@localhost \
--to=schwidefsky@de.ibm.com \
--cc=akpm@osdl.org \
--cc=davem@davemloft.net \
--cc=hch@lst.de \
--cc=heiko.carstens@de.ibm.com \
--cc=linux-kernel@vger.kernel.org \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.