Skip to contentSkip to conversation

iwlwifi initial bugs/thanks

DISCUSSION Jason Lunz 2 participants 2 messages Feb 13, 2007 1 message not in the archive

Subsystems (2)
lore
Share & tools
lore

Jason Lunz

author
#1 lore raw
In reply to a message not in the archive
Quoted 6 lines · Ben Gamari

That being said, now come the problems. When I first loaded the driver, I found that it had created two interfaces as mentioned in earlier threads. On my machine these are named eth1 and wlan0_rename (can someone confirm what the names are supposed to be and which interface does what?). Strangely, it appears that wlan0_rename is the actual wireless adapter as evidenced by the attached console session. In that

The "_rename" suffix is an indication that you have udev rules that are attempting to assign the same name to two different netdevs. Try to disable udev device renaming, or make the rules more specific.

The attached kernel patch makes it easier to figure out what udev is trying to do in situations like yours. It keeps getting rejected from linux upstream, but that could change with enough encouragement.

Jason

---

Keep track about which network interface names were renamed after the network device driver printed its banner.

---
 net/core/dev.c |    4 ++++
 1 file changed, 4 insertions(+)
net/core/dev.c +4 -0
Index: linux-2.6.20-uml/net/core/dev.c
===================================================================
@@ -754,7 +754,11 @@
 	else if (__dev_get_by_name(newname))
 		return -EEXIST;
 	else
+	{
+		if (strncmp(newname, dev->name, IFNAMSIZ))
+			printk(KERN_INFO "%s renamed to %s\n", dev->name, newname);
 		strlcpy(dev->name, newname, IFNAMSIZ);
+	}
 
 	err = class_device_rename(&dev->class_dev, dev->name);
 	if (!err) {
#2 lore raw
Quoted 7 lines · Jason Lunz

The "_rename" suffix is an indication that you have udev rules that are attempting to assign the same name to two different netdevs. Try to disable udev device renaming, or make the rules more specific.

The attached kernel patch makes it easier to figure out what udev is trying to do in situations like yours. It keeps getting rejected from linux upstream, but that could change with enough encouragement.

I think it's a good idea. Many drivers use the device name in the diagnostic messages. What's the point in seeing "wlan0: buffer overflow" if there is no way to find out from the log what that "wlan0" used to be?

net/core/dev.c line 759

Quoted 2 lines
net/core/dev.c +2 -0
+		if (strncmp(newname, dev->name, IFNAMSIZ))
+			printk(KERN_INFO "%s renamed to %s\n", dev->name, newname);

I think KERN_INFO is the right level, but maybe KERN_DEBUG would be more readily accepted.

Signature · 9 lines
Regards,
Pavel Roskin


-------------------------------------------------------------------------
Using Tomcat but need to do more? Need to support web services, security?
Get stuff done quickly with pre-integrated technology to make your job easier.
Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642