From: Benjamin Thery <benjamin.thery@bull.net>
To: Greg KH <gregkh@suse.de>
Cc: Daniel Lezcano <dlezcano@fr.ibm.com>,
linux-kernel@vger.kernel.org,
"Eric W. Biederman" <ebiederm@xmission.com>,
Tejun Heo <htejun@gmail.com>, Al Viro <viro@ftp.linux.org.uk>,
"Serge E. Hallyn" <serue@us.ibm.com>,
Pavel Emelyanov <xemul@openvz.org>,
netdev@vger.kernel.org, Andrew Morton <akpm@linux-foundation.org>
Subject: Re: [RESEND][PATCH 00/11] sysfs tagged directories
Date: Wed, 14 May 2008 17:07:15 +0200 [thread overview]
Message-ID: <482B0023.7090704@bull.net> (raw)
In-Reply-To: <4821AC4A.1010905@fr.ibm.com>
So, what is the status for this patchset?
So far, we discussed patch 10 a bit and Eric found a better way to fix
the kobject_rename() issue. But there hasn't been much discussion on the
rest of the patchset this time.
Does this mean that there is no strong objection against it now?
Do you think the approach is suitable for what we're aiming to?
Reminder: We need this for network namespaces. Without it network
namespaces testing is still a bit complicated in 2.6.26 (need to disable
sysfs). Also, Serge uses it too for user namespaces.
Should I resend it one more time ported on top of a more recent tree in
the hope it can be merged?
BTW which tree is the best? Linus', Greg's, -mm?
Regards,
Benjamin
Daniel Lezcano wrote:
> Greg KH wrote:
>> On Tue, May 06, 2008 at 07:30:30PM +0200, Benjamin Thery wrote:
>>> This is still the same port of Eric Biederman's patchset to implement
>>> tagged directories in sysfs that was discussed a few days ago.
>>>
>>> This time it applies on top of 2.6.26-rc1, which includes a fix from
>>> Daniel Lezcano to fix net device renaming
>>> for sysfs (and of course all the network namespaces stuff that was
>>> committed in net-2.6.26).
>>>
>>> This patchset still contains the patch from Serge Hallyn that implements
>>> tagging for user namespaces.
>>>
>>> It also contains a patch from Daniel Lezcano which allows to have net
>>> devices with the same name in two different network namespaces.
>>
>> Does this all work properly with both CONFIG_SYSFS_DEPRECATED enabled
>> and disabled?
>>
>> Can you show some outputs of 'tree' in both cases on the /sys/class/net/
>> directory?
>>
>> thanks,
>>
>> greg k-h
>
> Here is the output from an ls -l of /sys/class/net. I tryed with
> different configurations and checked the renamed network devices are
> propagated to the sysfs.
>
> 1) CONFIG_SYSFS_DEPRECATED && CONFIG_SYS && CONFIG_NET_NS
> =========================================================
>
> Outside a namespace:
> --------------------
>
> qemu:~> ip l
> 1: lo: <LOOPBACK,UP,10000> mtu 16436 qdisc noqueue
> link/loopback 00:00:00:00:00:00 brd 00:00:00:00:00:00
> 2: eth0: <BROADCAST,MULTICAST,UP,10000> mtu 1500 qdisc pfifo_fast qlen 1000
> link/ether 52:54:00:12:34:56 brd ff:ff:ff:ff:ff:ff
> 3: dummy0: <BROADCAST,NOARP> mtu 1500 qdisc noop
> link/ether 1e:43:9e:1f:1f:55 brd ff:ff:ff:ff:ff:ff
> 4: br0: <BROADCAST,MULTICAST,UP,10000> mtu 1500 qdisc noqueue
> link/ether 52:54:00:12:34:56 brd ff:ff:ff:ff:ff:ff
> 16: veth1: <BROADCAST,MULTICAST,PROMISC> mtu 1500 qdisc pfifo_fast qlen
> 1000
> link/ether 8e:94:e6:72:63:cf brd ff:ff:ff:ff:ff:ff
>
> qemu:~/lxc> ll /sys/class/net/
> total 0
> drwxr-xr-x 5 root root 0 2008-05-07 15:24 br0
> drwxr-xr-x 3 root root 0 2008-05-07 13:16 dummy0
> drwxr-xr-x 4 root root 0 2008-05-07 15:24 eth0
> drwxr-xr-x 3 root root 0 2008-05-07 13:16 lo
> drwxr-xr-x 4 root root 0 2008-05-07 15:26 veth1
>
>
> Inside the namespace:
> ----------------------
>
> qemu:~> ip l
> 14: lo: <LOOPBACK,UP,10000> mtu 16436 qdisc noqueue
> link/loopback 00:00:00:00:00:00 brd 00:00:00:00:00:00
> 15: veth2: <BROADCAST,MULTICAST,UP,10000> mtu 1500 qdisc pfifo_fast qlen
> 1000
> link/ether 46:85:e5:76:90:05 brd ff:ff:ff:ff:ff:ff
>
> virtnode:~lxc # ll /sys/class/net/
> total 0
> drwxr-xr-x 3 root root 0 May 7 15:28 lo
> drwxr-xr-x 3 root root 0 May 7 15:26 veth2
>
> 2) CONFIG_SYSFS && CONFIG_NET_NS
> ================================
>
>
> Outside the namespace:
> ----------------------
>
> qemu:~> ip l
> 1: lo: <LOOPBACK,UP,10000> mtu 16436 qdisc noqueue
> link/loopback 00:00:00:00:00:00 brd 00:00:00:00:00:00
> 2: eth0: <BROADCAST,MULTICAST,UP,10000> mtu 1500 qdisc pfifo_fast qlen 1000
> link/ether 52:54:00:12:34:56 brd ff:ff:ff:ff:ff:ff
> 3: dummy0: <BROADCAST,NOARP> mtu 1500 qdisc noop
> link/ether 72:16:3d:88:11:70 brd ff:ff:ff:ff:ff:ff
> 4: br0: <BROADCAST,MULTICAST,UP,10000> mtu 1500 qdisc noqueue
> link/ether 52:54:00:12:34:56 brd ff:ff:ff:ff:ff:ff
> 13: veth1: <BROADCAST,MULTICAST,PROMISC> mtu 1500 qdisc pfifo_fast qlen
> 1000
> link/ether aa:6f:e8:dd:b8:ba brd ff:ff:ff:ff:ff:ff
>
> * Content of /sys/class/net:
>
> qemu:~/lxc> ll /sys/class/net/
> total 0
> lrwxrwxrwx 1 root root 0 2008-05-07 14:31 br0 ->
> ../../devices/virtual/net/br0
> lrwxrwxrwx 1 root root 0 2008-05-07 14:28 dummy0 ->
> ../../devices/virtual/net/dummy0
> lrwxrwxrwx 1 root root 0 2008-05-07 14:28 eth0 ->
> ../../devices/pci0000:00/0000:00:03.0/net/eth0
> lrwxrwxrwx 1 root root 0 2008-05-07 14:28 lo ->
> ../../devices/virtual/net/lo
> lrwxrwxrwx 1 root root 0 2008-05-07 14:35 veth1 ->
> ../../devices/virtual/net/veth1
>
> Inside the namespace:
> ----------------------
>
> qemu:~> ip l
> 11: lo: <LOOPBACK,UP,10000> mtu 16436 qdisc noqueue
> link/loopback 00:00:00:00:00:00 brd 00:00:00:00:00:00
> 12: veth2: <BROADCAST,MULTICAST,UP,10000> mtu 1500 qdisc pfifo_fast qlen
> 1000
> link/ether ee:c5:a3:7c:58:bd brd ff:ff:ff:ff:ff:ff
>
> virtnode:~/lxc # ll /sys/class/net/
> total 0
> lrwxrwxrwx 1 root root 0 May 7 14:40 lo -> ../../devices/virtual/net/lo
> lrwxrwxrwx 1 root root 0 May 7 14:40 veth2 ->
> ../../devices/virtual/net/veth2
>
>
> 3) CONFIG_SYSFS
> ===============
>
> qemu:~> ip l
> 1: lo: <LOOPBACK,UP,10000> mtu 16436 qdisc noqueue
> link/loopback 00:00:00:00:00:00 brd 00:00:00:00:00:00
> 2: eth0: <BROADCAST,MULTICAST,UP,10000> mtu 1500 qdisc pfifo_fast qlen 1000
> link/ether 52:54:00:12:34:56 brd ff:ff:ff:ff:ff:ff
> 3: dummy0: <BROADCAST,NOARP> mtu 1500 qdisc noop
> link/ether 6e:3a:5f:2f:fa:cb brd ff:ff:ff:ff:ff:ff
> 4: br0: <BROADCAST,MULTICAST,UP,10000> mtu 1500 qdisc noqueue
> link/ether 52:54:00:12:34:56 brd ff:ff:ff:ff:ff:ff
>
> qemu:~/lxc> ll /sys/class/net/
> total 0
> lrwxrwxrwx 1 root root 0 2008-05-07 15:01 br0 ->
> ../../devices/virtual/net/br0
> lrwxrwxrwx 1 root root 0 2008-05-07 14:59 dummy0 ->
> ../../devices/virtual/net/dummy0
> lrwxrwxrwx 1 root root 0 2008-05-07 14:59 eth0 ->
> ../../devices/pci0000:00/0000:00:03.0/net/eth0
> lrwxrwxrwx 1 root root 0 2008-05-07 14:59 lo ->
> ../../devices/virtual/net/lo
>
> 4) CONFIG_SYSFS_DEPRECATED && CONFIG_SYSFS
> ==========================================
>
> qemu:~> ip l
> 1: lo: <LOOPBACK,UP,10000> mtu 16436 qdisc noqueue
> link/loopback 00:00:00:00:00:00 brd 00:00:00:00:00:00
> 2: eth0: <BROADCAST,MULTICAST,UP,10000> mtu 1500 qdisc pfifo_fast qlen 1000
> link/ether 52:54:00:12:34:56 brd ff:ff:ff:ff:ff:ff
> 3: dummy0: <BROADCAST,NOARP> mtu 1500 qdisc noop
> link/ether 16:85:f0:a8:34:44 brd ff:ff:ff:ff:ff:ff
> 4: br0: <BROADCAST,MULTICAST,UP,10000> mtu 1500 qdisc noqueue
> link/ether 52:54:00:12:34:56 brd ff:ff:ff:ff:ff:ff
>
> qemu:~> ll /sys/class/net/
> total 0
> drwxr-xr-x 5 root root 0 2008-05-07 15:07 br0
> drwxr-xr-x 3 root root 0 2008-05-07 15:06 dummy0
> drwxr-xr-x 4 root root 0 2008-05-07 15:07 eth0
> drwxr-xr-x 3 root root 0 2008-05-07 15:06 lo
>
>
--
B e n j a m i n T h e r y - BULL/DT/Open Software R&D
http://www.bull.com
prev parent reply other threads:[~2008-05-14 15:07 UTC|newest]
Thread overview: 48+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-05-06 17:30 [RESEND][PATCH 00/11] sysfs tagged directories Benjamin Thery
2008-05-06 17:30 ` [PATCH 01/11] sysfs: Support for preventing unmounts Benjamin Thery
2008-05-06 17:30 ` [PATCH 02/11] sysfs: sysfs_get_dentry add a sb parameter Benjamin Thery
2008-05-06 17:31 ` [PATCH 03/11] sysfs: Implement __sysfs_get_dentry Benjamin Thery
2008-05-06 17:31 ` [PATCH 04/11] sysfs: Rename Support multiple superblocks Benjamin Thery
2008-05-06 17:31 ` [PATCH 05/11] sysfs: sysfs_chmod_file handle " Benjamin Thery
2008-05-06 17:31 ` [PATCH 06/11] sysfs: Implement sysfs tagged directory support Benjamin Thery
2008-05-06 17:31 ` [PATCH 07/11] sysfs: Implement sysfs_delete_link and sysfs_rename_link Benjamin Thery
2008-05-06 17:31 ` [PATCH 08/11] driver core: Implement tagged directory support for device classes Benjamin Thery
2008-05-06 17:32 ` [PATCH 09/11] netns: Enable tagging for net_class directories in sysfs Benjamin Thery
2008-05-06 17:32 ` [PATCH 10/11] avoid kobject name conflict with different namespaces Benjamin Thery
2008-05-07 18:49 ` Eric W. Biederman
2008-05-07 19:08 ` Greg KH
2008-05-07 20:54 ` Eric W. Biederman
2008-05-08 8:28 ` Cornelia Huck
2008-05-08 19:28 ` Eric W. Biederman
2008-05-09 5:35 ` Cornelia Huck
2008-05-09 18:16 ` Eric W. Biederman
2008-05-08 19:25 ` Eric W. Biederman
2008-05-08 21:30 ` [PATCH] wireless: Add missing locking to cfg80211_dev_rename Eric W. Biederman
2008-05-08 22:12 ` Serge E. Hallyn
2008-05-08 22:18 ` Johannes Berg
2008-05-08 21:41 ` [PATCH] Fix kobject_rename and !CONFIG_SYSFS Eric W. Biederman
2008-05-12 22:02 ` kobject: " Greg KH
2008-05-13 7:00 ` Eric W. Biederman
2008-05-13 14:25 ` Benjamin Thery
2008-05-13 16:44 ` Greg KH
2008-05-13 17:55 ` [PATCH] Fix kobject_rename and !CONFIG_SYSFS v2 Eric W. Biederman
2008-05-13 18:23 ` Randy.Dunlap
2008-05-13 20:43 ` Eric W. Biederman
2008-05-13 20:16 ` Greg KH
2008-05-13 20:45 ` [PATCH] Fix kobject_rename and !CONFIG_SYSFS v3 Eric W. Biederman
2008-05-13 21:18 ` Randy Dunlap
2008-05-14 4:39 ` [PATCH] Fix kobject_rename and !CONFIG_SYSFS v4 Eric W. Biederman
2008-05-14 5:03 ` Andrew Morton
2008-05-14 9:01 ` Eric W. Biederman
2008-05-14 9:20 ` Andrew Morton
2008-05-14 9:51 ` Benjamin Thery
2008-05-14 9:56 ` Andrew Morton
2008-05-13 19:33 ` kobject: Fix kobject_rename and !CONFIG_SYSFS Benjamin Thery
2008-05-13 20:42 ` Eric W. Biederman
2008-05-06 17:32 ` [PATCH 11/11] sysfs: user namespaces: add ns to user_struct Benjamin Thery
2008-05-06 19:03 ` Serge E. Hallyn
2008-05-06 17:53 ` [RESEND][PATCH 00/11] sysfs tagged directories Greg KH
2008-05-06 18:41 ` Benjamin Thery
2008-05-07 13:19 ` Daniel Lezcano
2008-05-07 13:47 ` Benjamin Thery
2008-05-14 15:07 ` Benjamin Thery [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=482B0023.7090704@bull.net \
--to=benjamin.thery@bull.net \
--cc=akpm@linux-foundation.org \
--cc=dlezcano@fr.ibm.com \
--cc=ebiederm@xmission.com \
--cc=gregkh@suse.de \
--cc=htejun@gmail.com \
--cc=linux-kernel@vger.kernel.org \
--cc=netdev@vger.kernel.org \
--cc=serue@us.ibm.com \
--cc=viro@ftp.linux.org.uk \
--cc=xemul@openvz.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).