Bug#451906: rt73-common: /etc/udev/rules.d/85-rt73.rules contains deprecated udev rule

2007-11-25 Thread Ben Hutchings
On Sun, 2007-11-25 at 13:47 +0100, Piotr Roszatycki wrote:
 2007/11/25, Ben Hutchings [EMAIL PROTECTED]:
  Now it looks like the Ralink drivers do not allow setting most wireless
  parameters while they are down (though rt2500 alone seems to be more
  flexible).  Unfortunately, when using ifupdown and wireless-tools, this
  is exactly what is attempted - see bug #403983.  Is this what you were
  trying to work around, Piotr?
 
 Yes, it is a workaround for this bug. Please, notice that udev rule is
 required also for i.e. rutilt and other similar utilities. I was very
 disappointed that rutilt didn't work after I plugged the wi-fi dongle.

I patched rutilt to work around this (018_up_and_down.diff).  I believe
this has been applied upstream.

  I think a better workaround would be include a pre-up script that does
  something like:
 
  if [ x$(basename $(readlink /sys/class/net/$IFACE/device/driver)) = 
  xrt73 ]; then
  ifconfig $IFACE up
  fi
 
 I think the pre script for ifupdown should be created even if udev
 rule is existing.

This should replace the udev rule.  I know it's simpler to do this as
soon as the device is plugged in, but no other network interface is
brought up automatically like this, and we know the rule is likely to
break in future udev versions.

Ben.

-- 
Ben Hutchings
Computers are not intelligent.  They only think they are.


signature.asc
Description: This is a digitally signed message part


Bug#451906: rt73-common: /etc/udev/rules.d/85-rt73.rules contains deprecated udev rule

2007-11-25 Thread Piotr Roszatycki
2007/11/25, Ben Hutchings [EMAIL PROTECTED]:
 Now it looks like the Ralink drivers do not allow setting most wireless
 parameters while they are down (though rt2500 alone seems to be more
 flexible).  Unfortunately, when using ifupdown and wireless-tools, this
 is exactly what is attempted - see bug #403983.  Is this what you were
 trying to work around, Piotr?

Yes, it is a workaround for this bug. Please, notice that udev rule is
required also for i.e. rutilt and other similar utilities. I was very
disappointed that rutilt didn't work after I plugged the wi-fi dongle.


 I think a better workaround would be include a pre-up script that does
 something like:

 if [ x$(basename $(readlink /sys/class/net/$IFACE/device/driver)) = 
 xrt73 ]; then
 ifconfig $IFACE up
 fi

I think the pre script for ifupdown should be created even if udev
rule is existing.

-- 
 .''`.Piotr Roszatycki
: :' :mailto:[EMAIL PROTECTED]
`. `' mailto:[EMAIL PROTECTED]
  `-



-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]



Bug#451906: rt73-common: /etc/udev/rules.d/85-rt73.rules contains deprecated udev rule

2007-11-25 Thread Piotr Roszatycki
2007/11/25, Ben Hutchings [EMAIL PROTECTED]:
 On Sun, 2007-11-25 at 13:47 +0100, Piotr Roszatycki wrote:
  2007/11/25, Ben Hutchings [EMAIL PROTECTED]:
   Now it looks like the Ralink drivers do not allow setting most wireless
   parameters while they are down (though rt2500 alone seems to be more
   flexible).  Unfortunately, when using ifupdown and wireless-tools, this
   is exactly what is attempted - see bug #403983.  Is this what you were
   trying to work around, Piotr?
 
  Yes, it is a workaround for this bug. Please, notice that udev rule is
  required also for i.e. rutilt and other similar utilities. I was very
  disappointed that rutilt didn't work after I plugged the wi-fi dongle.

 I patched rutilt to work around this (018_up_and_down.diff).  I believe
 this has been applied upstream.

Do you mean this patch which is removed from Ubuntu?

rutilt (0.15-0ubuntu5.2) gutsy-proposed; urgency=low

  * Revised 018_up_and_down.diff (LP: #164058) :
remove the change to configuration code that would bring the interface
down before setting ESSID, as this is not necessary and does not work
with all drivers (notably the r73 one).
Thanks to Debian/Ben Hutchings.

I know that just legacy rt73 driver is broken but I think our
workarounds can break other things.

-- 
 .''`.Piotr Roszatycki
: :' :mailto:[EMAIL PROTECTED]
`. `' mailto:[EMAIL PROTECTED]
  `-



-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]



Bug#451906: rt73-common: /etc/udev/rules.d/85-rt73.rules contains deprecated udev rule

2007-11-24 Thread Piotr Roszatycki
The interface have to be set up. Without it, the iwconfig can't be
used. I'd rather want to convert it to newer udevd version that remove
it completly.

2007/11/19, Michael Prokop [EMAIL PROTECTED]:
 Package: rt73-common
 Version: 1:1.0.3.6-cvs20071114-dfsg1-1
 Severity: important


 # cat /etc/udev/rules.d/85-rt73.rules
 SUBSYSTEM==net, ACTION==add, ENV{PHYSDEVDRIVER}==rt73, 
 RUN+=/sbin/ifconfig $env{INTERFACE} up

 Current versions of udevd say:

 | PHYSDEV* values are deprecated and will be removed from a future
 | kernel, please fix it in /etc/udev/rules.d/85-rt73.rules:1

 BTW: What's the (real) reason for this rule?

 regards,
 -mika-



-- 
 .''`.Piotr Roszatycki
: :' :mailto:[EMAIL PROTECTED]
`. `' mailto:[EMAIL PROTECTED]
  `-



-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]



Bug#451906: rt73-common: /etc/udev/rules.d/85-rt73.rules contains deprecated udev rule

2007-11-24 Thread Ben Hutchings
On Sat, 2007-11-24 at 13:25 +0100, Piotr Roszatycki wrote: 
 The interface have to be set up. Without it, the iwconfig can't be
 used. I'd rather want to convert it to newer udevd version that remove
 it completly.

IIf the user can run iwconfig, he can also run ifconfig - or indeed
ifup.

Now it looks like the Ralink drivers do not allow setting most wireless
parameters while they are down (though rt2500 alone seems to be more
flexible).  Unfortunately, when using ifupdown and wireless-tools, this
is exactly what is attempted - see bug #403983.  Is this what you were
trying to work around, Piotr?

I think a better workaround would be include a pre-up script that does
something like:

if [ x$(basename $(readlink /sys/class/net/$IFACE/device/driver)) = 
xrt73 ]; then
ifconfig $IFACE up
fi

We should consider applying this to some or all of the other legacy
drivers.

Ben.

-- 
Ben Hutchings
Once a job is fouled up, anything done to improve it makes it worse.


signature.asc
Description: This is a digitally signed message part


Bug#451906: rt73-common: /etc/udev/rules.d/85-rt73.rules contains deprecated udev rule

2007-11-19 Thread Michael Prokop
Package: rt73-common
Version: 1:1.0.3.6-cvs20071114-dfsg1-1
Severity: important


# cat /etc/udev/rules.d/85-rt73.rules
SUBSYSTEM==net, ACTION==add, ENV{PHYSDEVDRIVER}==rt73, 
RUN+=/sbin/ifconfig $env{INTERFACE} up

Current versions of udevd say:

| PHYSDEV* values are deprecated and will be removed from a future
| kernel, please fix it in /etc/udev/rules.d/85-rt73.rules:1

BTW: What's the (real) reason for this rule?

regards,
-mika-



-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]