[prev in list] [next in list] [prev in thread] [next in thread] 

List:       busybox
Subject:    Re: ifplugd
From:       Vladimir Dronnikov <dronnikov () gmail ! com>
Date:       2009-04-30 11:22:02
Message-ID: 6784529b0904300422ta75697et54f7eafdf5e30b83 () mail ! gmail ! com
[Download RAW message or body]

2009/4/29 Vladimir Dronnikov <dronnikov@gmail.com>:
> ifplugd works fine for wired interfaces. However, when I try to cope
> with wireless interface (iwl3945 module):
>
> # ifplugd -i wlan -nsIM
>
> it bails out saying it "can't set interface flags: Operation not
> supported". Adding -m wlan to force wireless mode doesn't help.
> Can anyone report success?
>

The bug was in that the firmware for iwl3945 module was not loaded --
the traces have led me to mdev: we mangle device path in
make_device(), and then firmware loader uses the wrong path.

Attached patch fixes the issue.

Denys, please, consider applying.

--
Vladimir

["mdev.patch" (application/octet-stream)]

--- busybox.orig/util-linux/mdev.c	Mon Apr 20 16:54:20 2009
+++ busybox/util-linux/mdev.c	Wed Apr 29 20:56:03 2009
@@ -564,7 +565,10 @@
 				make_device(temp, 1);
 		}
 		else if (strcmp(action, "add") == 0) {
-			make_device(temp, 0);
+			/* N.B. make_device() mangles the device path, so save it */
+			char *s = xstrdup(temp);
+			make_device(s, 0);
+			free(s);
 			if (ENABLE_FEATURE_MDEV_LOAD_FIRMWARE) {
 				if (fw)
 					load_firmware(fw, temp);


_______________________________________________
busybox mailing list
busybox@busybox.net
http://lists.busybox.net/mailman/listinfo/busybox

[prev in list] [next in list] [prev in thread] [next in thread]