From: Ralf Baechle <ralf@linux-mips.org>
To: Linus Torvalds <torvalds@linux-foundation.org>
Cc: "Eric W. Biederman" <ebiederm@xmission.com>,
linux-kernel@vger.kernel.org, linux-arch@vger.kernel.org,
Daniel Lezcano <dlezcano@fr.ibm.com>
Subject: Re: [PATCH] ns: Wire up the setns system call for 2.6.40-rc1 or whatever
Date: Sun, 29 May 2011 09:49:40 +0100 [thread overview]
Message-ID: <20110529084940.GA26714@linux-mips.org> (raw)
In-Reply-To: <BANLkTi=-DHpvAm_9J9y+rOxB25e=4kCjYA@mail.gmail.com>
On Sat, May 28, 2011 at 10:55:55AM -0700, Linus Torvalds wrote:
> On Fri, May 27, 2011 at 7:28 PM, Eric W. Biederman
> <ebiederm@xmission.com> wrote:
> >
> > 32bit and 64bit on x86 are tested and working. The rest I have looked
> > at closely and I can't find any problems.
>
> So I really don't think this was even worth it. I applied the patch,
> but I think that you should just have done the architecture you
> tested, and left it to arch maintainers to add it as they will.
>
> That's how we tend to do this, and it works. It also avoids surprises
> when people then invariably end up having clashes due to system calls
> being added. Even in just the 15 hours since you sent the email, I had
> merged more code from ARM, and the patch no longer applied to my tree.
> It's trivial to fix up, so that's not the problem, but the problem is
> with different people adding system calls resulting in re-numbering.
Which just happened on MIPS; I had a conflict between sendmmsg and sysns.
People other than the maintainer adding new syscalls routinely goes wrong
for this or other reasons.
> In other words, it's simply better to strive to have *one* entity in
> charge of picking the system call number, rather than do it like this.
> Ergo: leave it to architecture maintainers to minimize the issue of
> system call renumbering.
Amen.
Ralf
prev parent reply other threads:[~2011-05-29 8:49 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-05-28 2:28 [PATCH] ns: Wire up the setns system call for 2.6.40-rc1 or whatever Eric W. Biederman
2011-05-28 2:28 ` Eric W. Biederman
2011-05-28 17:55 ` Linus Torvalds
2011-05-29 8:49 ` Ralf Baechle [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=20110529084940.GA26714@linux-mips.org \
--to=ralf@linux-mips.org \
--cc=dlezcano@fr.ibm.com \
--cc=ebiederm@xmission.com \
--cc=linux-arch@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=torvalds@linux-foundation.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.