* READONLY_EXEC is a curious name
@ 2004-04-14 19:06 Jamie Lokier
2004-04-14 19:21 ` Andi Kleen
0 siblings, 1 reply; 3+ messages in thread
From: Jamie Lokier @ 2004-04-14 19:06 UTC (permalink / raw)
To: Andi Kleen; +Cc: discuss, linux-kernel
This is not important.
PAGE_READONLY_EXEC is defined in <asm-x86_64/pgtable.h>.
Does anyone else think PAGE_READONLY_EXEC is an odd name for a set of
flags which enables read _and_ execute permission? What about
PAGE_READEXEC instead?
-- Jamie
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: READONLY_EXEC is a curious name
2004-04-14 19:06 READONLY_EXEC is a curious name Jamie Lokier
@ 2004-04-14 19:21 ` Andi Kleen
2004-04-14 19:41 ` Jamie Lokier
0 siblings, 1 reply; 3+ messages in thread
From: Andi Kleen @ 2004-04-14 19:21 UTC (permalink / raw)
To: Jamie Lokier; +Cc: discuss, linux-kernel
On Wed, 14 Apr 2004 20:06:53 +0100
Jamie Lokier <jamie@shareable.org> wrote:
> This is not important.
>
> PAGE_READONLY_EXEC is defined in <asm-x86_64/pgtable.h>.
>
> Does anyone else think PAGE_READONLY_EXEC is an odd name for a set of
> flags which enables read _and_ execute permission? What about
> PAGE_READEXEC instead?
It just follows the pattern there (default is with NX and _EXEC is the variant
without NX). I don't care much either ways.
-Andi
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: READONLY_EXEC is a curious name
2004-04-14 19:21 ` Andi Kleen
@ 2004-04-14 19:41 ` Jamie Lokier
0 siblings, 0 replies; 3+ messages in thread
From: Jamie Lokier @ 2004-04-14 19:41 UTC (permalink / raw)
To: Andi Kleen; +Cc: discuss, linux-kernel
Andi Kleen wrote:
> > PAGE_READONLY_EXEC is defined in <asm-x86_64/pgtable.h>.
> >
> > Does anyone else think PAGE_READONLY_EXEC is an odd name for a set of
> > flags which enables read _and_ execute permission? What about
> > PAGE_READEXEC instead?
>
> It just follows the pattern there (default is with NX and _EXEC is
> the variant without NX). I don't care much either ways.
I asked because I'm about to submit a patch to clean up the Alpha
definitions and have to pick a name. I see parisc picked
PAGE_EXECREAD for this.
-- Jamie
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2004-04-14 19:42 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2004-04-14 19:06 READONLY_EXEC is a curious name Jamie Lokier
2004-04-14 19:21 ` Andi Kleen
2004-04-14 19:41 ` Jamie Lokier
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox