Linux wireless drivers development
 help / color / mirror / Atom feed
* [PATCH] cfg80211: use debugfs_rename
@ 2007-05-21 19:04 Johannes Berg
  0 siblings, 0 replies; 3+ messages in thread
From: Johannes Berg @ 2007-05-21 19:04 UTC (permalink / raw)
  To: linux-wireless; +Cc: John Linville

Use the new debugfs_rename function in mac80211's debugfs where
necessary and fix a bug with renaming.

Signed-off-by: Johannes Berg <johannes@sipsolutions.net>

---
debugfs_rename hasn't actually been merged yet, but I hope it will be
merged soon. URL is
http://article.gmane.org/gmane.linux.file-systems/14819

This patch should also apply against net-2.6 (with some fuzz though due
to the nl80211 bit)

 net/wireless/core.c |    9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

--- wireless-dev.orig/net/wireless/core.c	2007-05-21 18:37:40.541805077 +0200
+++ wireless-dev/net/wireless/core.c	2007-05-21 21:01:03.481611481 +0200
@@ -162,10 +162,15 @@ int cfg80211_dev_rename(struct cfg80211_
 
 	/* this will check for collisions */
 	result = device_rename(&rdev->wiphy.dev, newname);
-	if (!result)
+	if (result)
 		return result;
 
-	/* TODO: do debugfs rename! */
+	if (!debugfs_rename(rdev->wiphy.debugfsdir->d_parent,
+			    rdev->wiphy.debugfsdir,
+			    rdev->wiphy.debugfsdir->d_parent,
+			    newname))
+		printk(KERN_ERR "cfg80211: failed to rename debugfs dir to %s!\n",
+		       newname);
 
 	nl80211_notify_dev_rename(rdev);
 



^ permalink raw reply	[flat|nested] 3+ messages in thread

* [PATCH] cfg80211: use debugfs_rename
@ 2007-06-21 16:30 Johannes Berg
  2007-07-16 12:34 ` Johannes Berg
  0 siblings, 1 reply; 3+ messages in thread
From: Johannes Berg @ 2007-06-21 16:30 UTC (permalink / raw)
  To: John W. Linville; +Cc: linux-wireless


Signed-off-by: Johannes Berg <johannes@sipsolutions.net>

---
 net/wireless/core.c |    9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

--- wireless-dev.orig/net/wireless/core.c	2007-05-21 18:37:40.541805077 +0200
+++ wireless-dev/net/wireless/core.c	2007-05-21 21:01:03.481611481 +0200
@@ -162,10 +162,15 @@ int cfg80211_dev_rename(struct cfg80211_
 
 	/* this will check for collisions */
 	result = device_rename(&rdev->wiphy.dev, newname);
-	if (!result)
+	if (result)
 		return result;
 
-	/* TODO: do debugfs rename! */
+	if (!debugfs_rename(rdev->wiphy.debugfsdir->d_parent,
+			    rdev->wiphy.debugfsdir,
+			    rdev->wiphy.debugfsdir->d_parent,
+			    newname))
+		printk(KERN_ERR "cfg80211: failed to rename debugfs dir to %s!\n",
+		       newname);
 
 	nl80211_notify_dev_rename(rdev);
 



^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] cfg80211: use debugfs_rename
  2007-06-21 16:30 Johannes Berg
@ 2007-07-16 12:34 ` Johannes Berg
  0 siblings, 0 replies; 3+ messages in thread
From: Johannes Berg @ 2007-07-16 12:34 UTC (permalink / raw)
  To: John W. Linville; +Cc: linux-wireless

[-- Attachment #1: Type: text/plain, Size: 162 bytes --]

This patch and the one titled "mac80211: use debugfs_rename" should be
going in now as the debugfs_rename functionality has been merged by
GregKH.

johannes

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 190 bytes --]

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2007-07-17 10:28 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2007-05-21 19:04 [PATCH] cfg80211: use debugfs_rename Johannes Berg
  -- strict thread matches above, loose matches on Subject: below --
2007-06-21 16:30 Johannes Berg
2007-07-16 12:34 ` Johannes Berg

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox