login
Login
/
Register
Search
Search this site:
Forums
News
Blogs
Features
Site
Home
»
Mailing list archives
»
linux-kernel
»
2008
»
August
»
21
Re: [PATCH 6/8] Revert "netns: Fix device renaming for sysfs"
view
thread
Previous message: [
thread
] [
date
] [
author
]
Next message: [
thread
] [
date
] [
author
]
[view in full thread]
From: David Miller
Subject:
Re: [PATCH 6/8] Revert "netns: Fix device renaming for sysfs"
Date: Wednesday, August 20, 2008 - 11:47 pm
From:
ebiederm@xmission.com
(Eric W. Biederman) Date: Wed, 20 Aug 2008 23:38:31 -0700
quoted text
> > This reverts commit aaf8cdc34ddba08122f02217d9d684e2f9f5d575. > > Drivers like the ipw2100 call device_create_group when they > are initialized and device_remove_group when they are shutdown. > Moving them between namespaces deletes their sysfs groups early. > > In particular the following call chain results. > netdev_unregister_kobject -> device_del -> kobject_del -> sysfs_remove_dir > With sysfs_remove_dir recursively deleting all of it's subdirectories, > and nothing adding them back. > > Ouch! > > Therefore we need to call something that ultimate calls sysfs_mv_dir > as that sysfs function can move sysfs directories between namespaces > without deleting their subdirectories or their contents. Allowing > us to avoid placing extra boiler plate into every driver that does > something interesting with sysfs. > > Currently the function that provides that capability is device_rename. > That is the code works without nasty side effects as originally written. > > So remove the misguided fix for moving devices between namespaces. The > bug in the kobject layer that inspired it has now been recognized and > fixed. > > Signed-off-by: Eric W. Biederman <ebiederm@xmission.com>
Acked-by: David S. Miller <davem@davemloft.net> --
unsubscribe notice
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to
majordomo@vger.kernel.org
More majordomo info at
http://vger.kernel.org/majordomo-info.html
Please read the FAQ at
http://www.tux.org/lkml/
Previous message: [
thread
] [
date
] [
author
]
Next message: [
thread
] [
date
] [
author
]
Messages in current thread:
[PATCH 06/11] sysfs: Implement sysfs tagged directory support.
, Benjamin Thery
, (Wed Jun 18, 10:08 am)
Re: [PATCH 06/11] sysfs: Implement sysfs tagged directory ...
, Tejun Heo
, (Sun Jun 22, 7:05 pm)
Re: [PATCH 06/11] sysfs: Implement sysfs tagged directory ...
, Eric W. Biederman
, (Thu Jun 26, 1:21 pm)
Re: [PATCH 06/11] sysfs: Implement sysfs tagged directory ...
, Tejun Heo
, (Sat Jun 28, 8:51 pm)
Re: [PATCH 06/11] sysfs: Implement sysfs tagged directory ...
, Eric W. Biederman
, (Mon Jun 30, 11:56 am)
Re: [PATCH 06/11] sysfs: Implement sysfs tagged directory ...
, Serge E. Hallyn
, (Mon Jun 30, 2:44 pm)
Re: [PATCH 06/11] sysfs: Implement sysfs tagged directory ...
, Tejun Heo
, (Mon Jun 30, 11:47 pm)
Re: [PATCH 06/11] sysfs: Implement sysfs tagged directory ...
, Eric W. Biederman
, (Tue Jul 1, 12:50 am)
Re: [PATCH 06/11] sysfs: Implement sysfs tagged directory ...
, Eric W. Biederman
, (Tue Jul 1, 2:20 am)
Re: [PATCH 06/11] sysfs: Implement sysfs tagged directory ...
, Tejun Heo
, (Tue Jul 1, 3:30 am)
Re: [PATCH 06/11] sysfs: Implement sysfs tagged directory ...
, Eric W. Biederman
, (Tue Jul 1, 5:30 am)
Re: [PATCH 06/11] sysfs: Implement sysfs tagged directory ...
, Tejun Heo
, (Tue Jul 1, 8:24 pm)
Re: [PATCH 06/11] sysfs: Implement sysfs tagged directory ...
, Eric W. Biederman
, (Tue Jul 1, 8:53 pm)
Re: [PATCH 06/11] sysfs: Implement sysfs tagged directory ...
, Tejun Heo
, (Tue Jul 1, 9:37 pm)
Re: [PATCH 06/11] sysfs: Implement sysfs tagged directory ...
, Eric W. Biederman
, (Wed Jul 2, 9:49 am)
Re: [PATCH 06/11] sysfs: Implement sysfs tagged directory ...
, Greg KH
, (Wed Jul 2, 5:15 pm)
Re: [PATCH 06/11] sysfs: Implement sysfs tagged directory ...
, Tejun Heo
, (Wed Jul 2, 8:18 pm)
Re: [PATCH 06/11] sysfs: Implement sysfs tagged directory ...
, Eric W. Biederman
, (Wed Jul 2, 10:11 pm)
Re: [PATCH 06/11] sysfs: Implement sysfs tagged directory ...
, Daniel Lezcano
, (Thu Jul 3, 3:56 am)
Re: [PATCH 06/11] sysfs: Implement sysfs tagged directory ...
, Eric W. Biederman
, (Thu Jul 3, 5:27 am)
Re: [PATCH 06/11] sysfs: Implement sysfs tagged directory ...
, Benjamin Thery
, (Thu Jul 3, 5:37 am)
Re: [PATCH 06/11] sysfs: Implement sysfs tagged directory ...
, Daniel Lezcano
, (Thu Jul 3, 5:55 am)
Re: [PATCH 06/11] sysfs: Implement sysfs tagged directory ...
, Tejun Heo
, (Thu Jul 3, 8:58 am)
Re: [PATCH 06/11] sysfs: Implement sysfs tagged directory ...
, Daniel Lezcano
, (Thu Jul 3, 11:29 am)
Re: [PATCH 06/11] sysfs: Implement sysfs tagged directory ...
, Eric W. Biederman
, (Thu Jul 3, 12:57 pm)
Re: [PATCH 06/11] sysfs: Implement sysfs tagged directory ...
, Eric W. Biederman
, (Thu Jul 3, 1:08 pm)
[PATCH 00/15] sysfs support for namespaces
, Eric W. Biederman
, (Thu Jul 3, 5:48 pm)
[PATCH 01/15] kobject: Cleanup kobject_rename and !CONFIG_ ...
, Eric W. Biederman
, (Thu Jul 3, 6:05 pm)
[PATCH 02/15] sysfs: Support for preventing unmounts.
, Eric W. Biederman
, (Thu Jul 3, 6:07 pm)
[PATCH 03/15] sysfs: sysfs_get_dentry add a sb parameter
, Eric W. Biederman
, (Thu Jul 3, 6:08 pm)
[PATCH 04/15] sysfs: Implement __sysfs_get_dentry
, Eric W. Biederman
, (Thu Jul 3, 6:09 pm)
[PATCH 05/15] sysfs: Rename Support multiple superblocks
, Eric W. Biederman
, (Thu Jul 3, 6:10 pm)
[PATCH 06/15] Introduce sysfs_sd_setattr and fix sysfs_chmod
, Eric W. Biederman
, (Thu Jul 3, 6:11 pm)
[PATCH 07/15] sysfs: sysfs_chmod_file handle multiple supe ...
, Eric W. Biederman
, (Thu Jul 3, 6:13 pm)
[PATCH 08/15] sysfs: Make sysfs_mount static once again.
, Eric W. Biederman
, (Thu Jul 3, 6:14 pm)
[PATCH 09/15] sysfs: Implement sysfs tagged directory support.
, Eric W. Biederman
, (Thu Jul 3, 6:16 pm)
[PATCH 10/15] sysfs: Merge sysfs_rename_dir and sysfs_move_dir
, Eric W. Biederman
, (Thu Jul 3, 6:17 pm)
[PATCH 11/15] sysfs: Implement sysfs_delete_link and sysfs ...
, Eric W. Biederman
, (Thu Jul 3, 6:18 pm)
[PATCH 12/15] driver core: Implement tagged directory supp ...
, Eric W. Biederman
, (Thu Jul 3, 6:20 pm)
[PATCH 13/15] Revert "netns: Fix device renaming for sysfs"
, Eric W. Biederman
, (Thu Jul 3, 6:21 pm)
[PATCH 14/15] netns: Enable tagging for net_class director ...
, Eric W. Biederman
, (Thu Jul 3, 6:22 pm)
[PATCH 15/15] sysfs: user namespaces: fix bug with clone(C ...
, Eric W. Biederman
, (Thu Jul 3, 6:23 pm)
Re: [PATCH 00/15] sysfs support for namespaces
, Eric W. Biederman
, (Thu Jul 3, 6:27 pm)
Re: [PATCH 01/15] kobject: Cleanup kobject_rename and !CON ...
, Tejun Heo
, (Thu Jul 3, 11:33 pm)
Re: [PATCH 06/15] Introduce sysfs_sd_setattr and fix sysfs ...
, Tejun Heo
, (Thu Jul 3, 11:40 pm)
Re: [PATCH 07/15] sysfs: sysfs_chmod_file handle multiple ...
, Tejun Heo
, (Thu Jul 3, 11:44 pm)
Re: [PATCH 08/15] sysfs: Make sysfs_mount static once again.
, Tejun Heo
, (Thu Jul 3, 11:44 pm)
Re: [PATCH 12/15] driver core: Implement tagged directory ...
, Tejun Heo
, (Fri Jul 4, 12:50 am)
Re: [PATCH 12/15] driver core: Implement tagged directory ...
, Eric W. Biederman
, (Fri Jul 4, 6:31 am)
Re: [PATCH 12/15] driver core: Implement tagged directory ...
, Tejun Heo
, (Fri Jul 4, 6:57 am)
Re: [PATCH 12/15] driver core: Implement tagged directory ...
, Greg KH
, (Fri Jul 4, 9:12 am)
Re: [PATCH 12/15] driver core: Implement tagged directory ...
, Eric W. Biederman
, (Fri Jul 4, 2:49 pm)
Re: [PATCH 12/15] driver core: Implement tagged directory ...
, Eric W. Biederman
, (Fri Jul 4, 3:00 pm)
Re: [PATCH 00/15] sysfs support for namespaces
, Eric W. Biederman
, (Sat Jul 5, 9:42 pm)
Re: [PATCH 00/15] sysfs support for namespaces
, Cornelia Huck
, (Mon Jul 7, 4:41 am)
Re: [PATCH 00/15] sysfs support for namespaces
, Eric W. Biederman
, (Mon Jul 7, 5:22 am)
Re: [PATCH 12/15] driver core: Implement tagged directory ...
, Eric W. Biederman
, (Sun Jul 13, 6:54 pm)
Re: [PATCH 12/15] driver core: Implement tagged directory ...
, Tejun Heo
, (Tue Jul 15, 8:25 pm)
Re: [PATCH 12/15] driver core: Implement tagged directory ...
, Eric W. Biederman
, (Tue Jul 15, 10:41 pm)
Re: [PATCH 12/15] driver core: Implement tagged directory ...
, Tejun Heo
, (Tue Jul 15, 10:50 pm)
Re: [PATCH 12/15] driver core: Implement tagged directory ...
, Eric W. Biederman
, (Tue Jul 15, 11:32 pm)
Re: [PATCH 12/15] driver core: Implement tagged directory ...
, Tejun Heo
, (Tue Jul 15, 11:48 pm)
Re: [PATCH 12/15] driver core: Implement tagged directory ...
, Tejun Heo
, (Wed Jul 16, 12:02 am)
Re: [PATCH 12/15] driver core: Implement tagged directory ...
, Eric W. Biederman
, (Wed Jul 16, 12:07 pm)
Re: [PATCH 12/15] driver core: Implement tagged directory ...
, Eric W. Biederman
, (Wed Jul 16, 2:09 pm)
Re: [PATCH 12/15] driver core: Implement tagged directory ...
, Greg KH
, (Thu Jul 17, 4:08 pm)
Re: [PATCH 12/15] driver core: Implement tagged directory ...
, Tejun Heo
, (Fri Jul 18, 5:41 am)
Re: [PATCH 12/15] driver core: Implement tagged directory ...
, Greg KH
, (Fri Jul 18, 11:49 am)
Re: [PATCH 12/15] driver core: Implement tagged directory ...
, Eric W. Biederman
, (Fri Jul 18, 1:19 pm)
Re: [PATCH 12/15] driver core: Implement tagged directory ...
, Tejun Heo
, (Fri Jul 18, 6:07 pm)
Re: [PATCH 12/15] driver core: Implement tagged directory ...
, Eric W. Biederman
, (Sat Aug 2, 11:59 pm)
Re: [PATCH 09/15] sysfs: Implement sysfs tagged directory ...
, Greg KH
, (Tue Aug 19, 7:17 pm)
Re: [PATCH 09/15] sysfs: Implement sysfs tagged directory ...
, Eric W. Biederman
, (Tue Aug 19, 11:58 pm)
[PATCH 0/8] sysfs namespace support
, Eric W. Biederman
, (Wed Aug 20, 11:31 pm)
[PATCH 1/8] sysfs: Implement sysfs tagged directory support.
, Eric W. Biederman
, (Wed Aug 20, 11:33 pm)
[PATCH 2/8] sysfs: Merge sysfs_rename_dir and sysfs_move_dir
, Eric W. Biederman
, (Wed Aug 20, 11:34 pm)
[PATCH 3/8] sysfs: Implement sysfs_delete_link and sysfs_r ...
, Eric W. Biederman
, (Wed Aug 20, 11:35 pm)
[PATCH 5/8] sysfs: Remove sysfs_create_link_nowarn
, Eric W. Biederman
, (Wed Aug 20, 11:36 pm)
Re: [PATCH 0/8] sysfs namespace support
, David Miller
, (Wed Aug 20, 11:37 pm)
[PATCH 4/8] driver core: Implement tagged directory suppor ...
, Eric W. Biederman
, (Wed Aug 20, 11:37 pm)
[PATCH 6/8] Revert "netns: Fix device renaming for sysfs"
, Eric W. Biederman
, (Wed Aug 20, 11:38 pm)
[PATCH 7/8] netns: Enable tagging for net_class directorie ...
, Eric W. Biederman
, (Wed Aug 20, 11:39 pm)
[PATCH 8/8] sysfs: user namespaces: fix bug with clone(CLO ...
, Eric W. Biederman
, (Wed Aug 20, 11:40 pm)
Re: [PATCH 6/8] Revert "netns: Fix device renaming for sysfs"
, David Miller
, (Wed Aug 20, 11:47 pm)
Re: [PATCH 7/8] netns: Enable tagging for net_class direct ...
, David Miller
, (Wed Aug 20, 11:47 pm)
Re: [PATCH 1/8] sysfs: Implement sysfs tagged directory su ...
, Benjamin Thery
, (Wed Aug 27, 8:18 am)
Re: [PATCH 1/8] sysfs: Implement sysfs tagged directory su ...
, Mark Ryden
, (Tue Sep 2, 6:54 am)
Re: [PATCH 1/8] sysfs: Implement sysfs tagged directory su ...
, Benjamin Thery
, (Tue Sep 2, 7:03 am)
Re: [PATCH 1/8] sysfs: Implement sysfs tagged directory su ...
, Greg KH
, (Tue Sep 2, 10:01 am)
Re: [PATCH 1/8] sysfs: Implement sysfs tagged directory su ...
, David Shwatrz
, (Wed Sep 3, 10:33 pm)
Re: [PATCH 1/8] sysfs: Implement sysfs tagged directory su ...
, Benjamin Thery
, (Wed Sep 3, 11:44 pm)
Re: [PATCH 1/8] sysfs: Implement sysfs tagged directory su ...
, Mark Ryden
, (Mon Sep 8, 11:39 am)
Re: [PATCH 12/15] driver core: Implement tagged directory ...
, Jiri Slaby
, (Thu Sep 11, 5:45 am)
Re: [PATCH 12/15] driver core: Implement tagged directory ...
, Benjamin Thery
, (Thu Sep 11, 6:05 am)
Re: [PATCH 12/15] driver core: Implement tagged directory ...
, Jiri Slaby
, (Thu Sep 11, 11:32 pm)
Re: [PATCH 1/8] sysfs: Implement sysfs tagged directory su ...
, Mark Ryden
, (Tue Oct 7, 9:39 am)
Re: [PATCH 1/8] sysfs: Implement sysfs tagged directory su ...
, Greg KH
, (Tue Oct 7, 9:48 am)
Re: [PATCH 1/8] sysfs: Implement sysfs tagged directory su ...
, Daniel Lezcano
, (Tue Oct 7, 9:52 am)
Re: [PATCH 1/8] sysfs: Implement sysfs tagged directory su ...
, Eric W. Biederman
, (Tue Oct 7, 1:31 pm)
Re: [PATCH 1/8] sysfs: Implement sysfs tagged directory su ...
, Greg KH
, (Tue Oct 7, 2:09 pm)
Re: [PATCH 1/8] sysfs: Implement sysfs tagged directory su ...
, Eric W. Biederman
, (Tue Oct 7, 3:27 pm)
Re: [PATCH 1/8] sysfs: Implement sysfs tagged directory su ...
, Christoph Hellwig
, (Wed Oct 8, 6:00 am)
Re: [PATCH 1/8] sysfs: Implement sysfs tagged directory su ...
, Eric W. Biederman
, (Mon Oct 13, 8:20 pm)
Navigation
Create content
Mailing list archives
Recent posts
Popular discussions
linux-kernel
:
Josef Sipek
Re: [PATCH 01/24] Unionfs: Documentation
Greg Kroah-Hartman
[PATCH 20/36] Driver core: Call device_pm_add() after bus_add_device() in device_a...
Oleg Nesterov
Re: init's children list is long and slows reaping children.
Randy Dunlap
Re: mmotm 2010-06-11-16-40 uploaded
Pete Clements
2.6.20-git8 fails compile -- net/built-in.o __ipv6_addr_type
git
:
Stephen R. van den Berg
Re: [RFC] origin link for cherry-pick and revert
Christian Stimming
git-gui: Fix broken revert confirmation.
Junio C Hamano
Re: git-svnimport
Anuj Gakhar
Git Architecture Question
Johannes Schindelin
Re: [PATCH] Fix approxidate("never") to always return 0
linux-netdev
:
Gerrit Renker
v2 [PATCH 1/4] dccp: Limit feature negotiation to connection setup phase
Nick Piggin
Re: Kernel WARNING: at net/core/dev.c:1330 __netif_schedule+0x2c/0x98()
Daniel Lezcano
getsockopt(TCP_DEFER_ACCEPT) value change
David Miller
Re: 2.6.27.18: bnx2/tg3: BUG: "scheduling while atomic" trying to ifenslave a seco...
Ingo Molnar
Re: [regression] nf_iterate(), BUG: unable to handle kernel NULL pointer dereference
git-commits-head
:
Linux Kernel Mailing List
ath9k_htc: Allocate URBs properly
Linux Kernel Mailing List
[ARM] dma: use new dmabounce_sync_for_xxx() for dma_sync_single_xxx()
Linux Kernel Mailing List
MIPS: Cavium: Remove unused watchdog code.
Linux Kernel Mailing List
V4L/DVB (8976): af9015: Add USB ID for AVerMedia A309
Linux Kernel Mailing List
ARM: 5670/1: bcmring: add default configuration for bcmring arch
openbsd-misc
:
Christophe Rioux
Implementation example of snmp
Ryan McBride
Re: Packets Per Second Limit?
Nick Holland
Re: booting openbsd on eee without cd-rom
Bryan Irvine
Re: OpenBSD 4.7 Released, May 19 2010
Jacob Yocom-Piatt
Re: Same shit all over again
Colocation donated by:
Syndicate