[Bug 383320] New: udev creates wrong persistent net rules for wlan interfaces
https://bugzilla.novell.com/show_bug.cgi?id=383320 Summary: udev creates wrong persistent net rules for wlan interfaces Product: openSUSE 11.0 Version: Beta 1 Platform: Other OS/Version: Other Status: NEW Severity: Major Priority: P5 - None Component: Basesystem AssignedTo: kasievers@novell.com ReportedBy: cstender@novell.com QAContact: qa@suse.de CC: dkukawka@novell.com, hschaa@novell.com Found By: --- After a fresh beta1 installation the persistent-net-rules file does not contain the ATTR{type}=="1". Without the attribute udev tries to rename wmaster0 to wlan0 while booting. This fails (because wlan0 is already there) and blocks the boot process for 30seconds. /etc/udev/rules.d/70-persistent-net.rules SUBSYSTEM=="net", DRIVERS=="?*", ATTR{address}=="00:19:d2:06:b5:2c", KERNEL=="wlan*", NAME="wlan0" If I delete the wrong line in 70-persistent-net.rules and reload the iwl3945 module, udev creates the following new rule: # PCI device 0x8086:0x4227 (iwl3945) SUBSYSTEM=="net", ACTION=="add", DRIVERS=="?*", ATTR{address}=="00:19:d2:06:b5:2c", ATTR{type}=="1", KERNEL=="wlan*", NAME="wlan0" -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=383320
User mzugec@novell.com added comment
https://bugzilla.novell.com/show_bug.cgi?id=383320#c1
Michal Zugec
https://bugzilla.novell.com/show_bug.cgi?id=383320
User kasievers@novell.com added comment
https://bugzilla.novell.com/show_bug.cgi?id=383320#c2
Kay Sievers
https://bugzilla.novell.com/show_bug.cgi?id=383320
User mzugec@novell.com added comment
https://bugzilla.novell.com/show_bug.cgi?id=383320#c3
Michal Zugec
https://bugzilla.novell.com/show_bug.cgi?id=383320
User kasievers@novell.com added comment
https://bugzilla.novell.com/show_bug.cgi?id=383320#c4
--- Comment #4 from Kay Sievers
https://bugzilla.novell.com/show_bug.cgi?id=383320
User kasievers@novell.com added comment
https://bugzilla.novell.com/show_bug.cgi?id=383320#c6
Kay Sievers
https://bugzilla.novell.com/show_bug.cgi?id=383320
User mzugec@novell.com added comment
https://bugzilla.novell.com/show_bug.cgi?id=383320#c7
--- Comment #7 from Michal Zugec
https://bugzilla.novell.com/show_bug.cgi?id=383320
User mzugec@novell.com added comment
https://bugzilla.novell.com/show_bug.cgi?id=383320#c8
Michal Zugec
https://bugzilla.novell.com/show_bug.cgi?id=383320
User kasievers@novell.com added comment
https://bugzilla.novell.com/show_bug.cgi?id=383320#c9
Kay Sievers
I rewrited all that code but have some problems with udev: - there is no "udevtest" tool anymore
It is a debugging tool, it may change its output any time. Newer udev versions have only a single binary, which can be called as: /sbin/udevadm test <devpath> But still, do not use it in any software, it's output is not stable in any sense.
- udev rules matches only by MAC address, no BusID (in 10.3 I used KERNELS for that) - when I change NAME value, it's ignored and created new rule
If NAME= is set in any earlier rule, the udev persistent net generator should ignore the network interface completely. I guess yast should create and maintain its own rules file, which comes before 70-persistent-net.rules, instead of patching the udev maintained one. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=383320
User mzugec@novell.com added comment
https://bugzilla.novell.com/show_bug.cgi?id=383320#c10
Michal Zugec
I guess yast should create and maintain its own rules file, which comes before 70-persistent-net.rules, instead of patching the udev maintained one.
I don't think we can change it for 11.0 (too late). Why there is comment "You can modify it, as long as you keep each rule on a single line." But yast code is now much better, it keeps all and change only needed options (ATTR{address}, KERNELS and NAME) There is only one issue - how to "match by BusID" instead of MAC address? This was probably changed from 10.3 (it works with KERNELS rule) -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=383320
User kasievers@novell.com added comment
https://bugzilla.novell.com/show_bug.cgi?id=383320#c11
Kay Sievers
Thanks for explanation. udevtest - just to debug, I'm not use it in code
Ah, good.
NAME - I have a typo, already fixed
Fine.
I guess yast should create and maintain its own rules file, which comes before 70-persistent-net.rules, instead of patching the udev maintained one.
I don't think we can change it for 11.0 (too late). Why there is comment "You can modify it, as long as you keep each rule on a single line." But yast code is now much better, it keeps all and change only needed options (ATTR{address}, KERNELS and NAME)
There is only one issue - how to "match by BusID" instead of MAC address? This was probably changed from 10.3 (it works with KERNELS rule)
It should still work the same way, if the rule still matches and sets NAME=. There is was no change regarding matching parent device properties. But it's tricky, as there is hardware out, where the same device offers multiple interfaces with the same type. On such boxes, it might not work as expected without figuring out how to distinguish them properly. That's why we are adding all these keys with the new udev rules now. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=383320
User mzugec@novell.com added comment
https://bugzilla.novell.com/show_bug.cgi?id=383320#c12
Michal Zugec
https://bugzilla.novell.com/show_bug.cgi?id=383320
User casualprogrammer@yahoo.com added comment
https://bugzilla.novell.com/show_bug.cgi?id=383320#c13
--- Comment #13 from Casual J. Programmer
https://bugzilla.novell.com/show_bug.cgi?id=383320
User mzugec@novell.com added comment
https://bugzilla.novell.com/show_bug.cgi?id=383320#c14
Michal Zugec
participants (1)
-
bugzilla_noreply@novell.com