[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <49006671.3010804@bull.net>
Date: Thu, 23 Oct 2008 13:56:33 +0200
From: Benjamin Thery <benjamin.thery@...l.net>
To: "Eric W. Biederman" <ebiederm@...ssion.com>
Cc: netdev <netdev@...r.kernel.org>, Dave Miller <davem@...emloft.net>,
Greg Kroah-Hartman <gregkh@...e.de>,
Al Viro <viro@....linux.org.uk>,
Serge Hallyn <serue@...ibm.com>,
Daniel Lezcano <dlezcano@...ibm.com>,
linux-kernel@...r.kernel.org, Tejun Heo <htejun@...il.com>,
Denis Lunev <den@...nvz.org>,
Linux Containers <containers@...ts.linux-foundation.org>
Subject: Re: [PATCH 0/4][RFC] netns: sysfs: add a netns suffix to net device
sysfs entries
Eric W. Biederman wrote:
> Benjamin Thery <benjamin.thery@...l.net> writes:
>
>> Support for network namespaces in mainline is pretty complete for
>> some time now, but there is still this issue with sysfs that prevents
>> more people to use it easily.
>
> Ben your patchset is completely inappropriate.
>
> Temporarily adding elements to the ABI that we intend to remove
> is not a proper solution to this problem.
In fact, I feared that would be an issue before I sent the patches.
Thanks for confirming this.
But, as this alternative was discussed a bit after Al's comments on your
last patches, I had to give it a try and code a proof-of-concept.
Benjamin
>
> That user space visible ida you add is a namespace identifier that breaks
> nested containers and migration. It is very very very wrong.
>
> Eric
>
>
>
--
B e n j a m i n T h e r y - BULL/DT/Open Software R&D
http://www.bull.com
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists