From: Alan Cox <alan@lxorguk.ukuu.org.uk>
To: Chris Wedgwood <cw@f00f.org>
Cc: akpm@osdl.org, linux-arch@vger.kernel.org, jbaron@redhat.com,
ak@muc.de, arjan@linux.intel.com, benh@kernel.crashing.org,
geert@linux-m68k.org, hugh@veritas.com, kkojima@rr.iij4u.or.jp,
lethal@linux-sh.org, paulus@samba.org, rmk@arm.linux.org.uk,
spyro@f2s.com, tony.luck@intel.com, zippel@linux-m68k.org
Subject: Re: [patch 1/1] make PROT_WRITE imply PROT_READ
Date: Mon, 18 Sep 2006 10:26:33 +0100 [thread overview]
Message-ID: <1158571594.6069.82.camel@localhost.localdomain> (raw)
In-Reply-To: <20060918005723.GB12662@tuatara.stupidest.org>
Ar Sul, 2006-09-17 am 17:57 -0700, ysgrifennodd Chris Wedgwood:
> On Sun, Sep 17, 2006 at 10:14:06PM +0100, Alan Cox wrote:
>
> > Unfortunately yes. This was discovered in the real world.
>
> Are there a lot of these applications and does it break badly? Do you
> know which ones they are?
I don't know precisely which applications but Red Hat has customers
hitting the random behaviour. By choice I'd like that to turn into
consistent failure but its outside the hardware capabilities right now.
> It's not an intuitive change and as the current behavior isn't
> entirely consistent and (presumably) these applications work, so is
> this change really needed?
>
> Fixing userspace assumptions but setting a (counter-intuitive)
> precedent in the kernel seems wrong.
The current behaviour is "randomly fails". The expected behaviour is
"consistently fails" or "consistently succeeds". POSIX explicitly says
that passing just PROT_WRITE may give you a readable mapping (just as
PROT_READ may give you exec in turn).
The change would thus be logically consistent and in keeping with the
standard.
> Even with VT/Pacifica?
For VT at the moment the answer appears to be no.
Alan
prev parent reply other threads:[~2006-09-18 9:05 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-09-15 1:39 [patch 1/1] make PROT_WRITE imply PROT_READ akpm
2006-09-15 4:53 ` Arjan van de Ven
2006-09-15 11:10 ` Alan Cox
2006-09-15 8:06 ` Geert Uytterhoeven
2006-09-15 10:58 ` Alan Cox
2006-09-15 8:47 ` Andi Kleen
2006-09-15 11:12 ` Alan Cox
2006-09-16 0:40 ` Ralf Baechle
2006-09-17 11:59 ` Paul Mundt
2006-09-17 20:24 ` Chris Wedgwood
2006-09-17 20:48 ` Arjan van de Ven
2006-09-17 21:14 ` Alan Cox
2006-09-17 21:05 ` Arjan van de Ven
2006-09-18 0:57 ` Chris Wedgwood
2006-09-18 2:03 ` Paul Mackerras
2006-09-18 4:31 ` Chris Wedgwood
2006-09-18 8:15 ` Geert Uytterhoeven
2006-09-18 9:26 ` Alan Cox [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=1158571594.6069.82.camel@localhost.localdomain \
--to=alan@lxorguk.ukuu.org.uk \
--cc=ak@muc.de \
--cc=akpm@osdl.org \
--cc=arjan@linux.intel.com \
--cc=benh@kernel.crashing.org \
--cc=cw@f00f.org \
--cc=geert@linux-m68k.org \
--cc=hugh@veritas.com \
--cc=jbaron@redhat.com \
--cc=kkojima@rr.iij4u.or.jp \
--cc=lethal@linux-sh.org \
--cc=linux-arch@vger.kernel.org \
--cc=paulus@samba.org \
--cc=rmk@arm.linux.org.uk \
--cc=spyro@f2s.com \
--cc=tony.luck@intel.com \
--cc=zippel@linux-m68k.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox