From: Nick Piggin <nickpiggin@yahoo.com.au>
To: linux-arch@vger.kernel.org
Cc: Andrew Morton <akpm@osdl.org>, Roland Dreier <rdreier@cisco.com>,
"Michael S. Tsirkin" <mst@mellanox.co.il>,
Hugh Dickins <hugh@veritas.com>
Subject: [Fwd: Re: [PATCH] madvise MADV_DONTFORK/MADV_DOFORK]
Date: Wed, 15 Feb 2006 17:18:30 +1100 [thread overview]
Message-ID: <43F2C7B6.3020102@yahoo.com.au> (raw)
Forwarding to linux-arch. Please pipe up on lkml, or send patches
to fix your arch (either way, before 2.6.16!!) if you disagree.
To me it would be more logical if the numbering was made densely
packed on all architectures even if that means MADV_DONTFORK /
DOFORK aren't consistently numbered throughout (why should they
get special treatment?).
Nick
-------- Original Message --------
Subject: Re: [PATCH] madvise MADV_DONTFORK/MADV_DOFORK
Date: Tue, 14 Feb 2006 22:09:34 -0800
From: Roland Dreier <rdreier@cisco.com>
Nick> May I ask, what is the rationale for ignoring the apparent
Nick> conventions of all architectures? For example parisc, you
Nick> appear to even go contrary to the comment.
Looking through include/asm-*/mman.h, I have to agree. The parisc
example seemly especially bad, as (in addition to being in the
reserved range as Nick notes) the DONTFORK/DOFORK values are stuck in
a block with the page size values instead of the previous block where
they seem more sensible. However, in other files like the alpha
version, where the rest of the values are in decimal, the hex defines
look rather jarring.
Michael, what led you to choose 0x30 and 0x31 for the two new values?
It does seem that keeping them uniform across architectures is a
reasonable thing to do, but as far as I can tell the values 9 and 10
are unused on all architectures, and have the added merit of not
falling in the parisc reserved range.
Do we still have a chance to change this?
- R.
--
Send instant messages to your online friends http://au.messenger.yahoo.com
next reply other threads:[~2006-02-15 7:03 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-02-15 6:18 Nick Piggin [this message]
2006-02-15 8:10 ` [Fwd: Re: [PATCH] madvise MADV_DONTFORK/MADV_DOFORK] Michael S. Tsirkin
2006-02-15 8:17 ` Nick Piggin
2006-02-15 8:49 ` Nick Piggin
2006-02-15 12:12 ` Michael S. Tsirkin
2006-02-15 10:18 ` Russell King
2006-02-15 12:31 ` Nick Piggin
2006-02-15 10:20 ` Michael S. Tsirkin
2006-02-15 12:43 ` Matthew Wilcox
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=43F2C7B6.3020102@yahoo.com.au \
--to=nickpiggin@yahoo.com.au \
--cc=akpm@osdl.org \
--cc=hugh@veritas.com \
--cc=linux-arch@vger.kernel.org \
--cc=mst@mellanox.co.il \
--cc=rdreier@cisco.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