[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20110524071628.GA31612@elte.hu>
Date: Tue, 24 May 2011 09:16:28 +0200
From: Ingo Molnar <mingo@...e.hu>
To: "Eric W. Biederman" <ebiederm@...ssion.com>
Cc: James Bottomley <James.Bottomley@...senPartnership.com>,
Linus Torvalds <torvalds@...ux-foundation.org>,
linux-kernel@...r.kernel.org,
Linux Containers <containers@...ts.osdl.org>,
netdev@...r.kernel.org, Geert Uytterhoeven <geert@...ux-m68k.org>
Subject: Re: [GIT PULL] Namespace file descriptors for 2.6.40
* Eric W. Biederman <ebiederm@...ssion.com> wrote:
> > Also, system call table conflicts are trivial to resolve. Merging in
> > net-next to avoid such a conflict is like cracking a nut with a
> > sledgehammer.
>
> Well I still have trauma from how nasty it was to test with syscall numbers
> continuing to change when I was working on the kexec_load system call.
>
> As far as I can tell any one system call conflict on any one
> architecture is easy to resolve. Resolving a conflict on all
> architectures would amount to at least 50 files that need to be resolved
> that feels a bit more than trivial.
Of course - and the straightforward solution is to not do it but concentrate on
the 2-3 archs you find to be the primary target of your patches. How many
parisc systems are there on the planet, which in the future will be upgraded to
both kernel and user-space running your new syscall for real? Less than 10? How
many ARM and x86 systems?
> My gut feel says we should really implement an
> include/asm-generic/unistd-common.h to include all new system calls.
>
> That way there would be only one file to touch instead of 50. Certainly it
> works for include/asm-generic/unistd.h for the architectures that use it.
> And all we really need is just a little abstraction on that concept.
I suppose that could be tried, although in practice it would probably be
somewhat complex due to the various compat syscall handling differences.
So i guess this is one of the 'lets see how ugly/fragile it becomes' patches.
Thanks,
Ingo
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists