On Sat, Mar 22, 2008 at 12:43:04PM -0000, Nicolò Chieffo wrote: > Steve, I've had a look at the patch, ok it works in most cases, but it has > a problem (especially if some new methods to check the wifi status are > added): it is not enought that _one_ method reports the card on, since > there could be another that reports it off. to report a device on _every_ > methods must say "on"
Yes, one of the assumptions I've made in this patch is that, if multiple interfaces are exposed by the driver for this information, each of these interfaces is equally valid. If a driver exposes both rf_kill and power/state, and they disagree, then this is a bug in the kernel driver. Since I'm not aware of any kernel drivers with this bug, I currently don't think it's worth the additional effort to guard against this possibility. (Even in the ASUS case, it appears that the only other interface which *might* provide conflicting information is one not currently supported by isAnyWirelessPoweredOn().) -- change isAnyWirelessPoweredOn() accordingly to the removal of the old status kernel interface https://bugs.launchpad.net/bugs/180598 You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
