On Wed, Sep 21, 2011 at 10:01 AM, Kalle Valo <kvalo@xxxxxxxxxx> wrote: > On 09/21/2011 07:37 PM, Johannes Berg wrote: >> iwconfig wlan0 power timeout 0 >> iwconfig wlan0 power off >> iwconfig wlan0 power on >> >> => observe that power saving stays disabled forever >> >> Last little bit of http://paste.pocoo.org/raw/479987/ seems to make it >> work somewhat, but is then still completely broken when timeout is >0 >> due to the abuse of the dynps timer/work to process tx status: >> >> Clearly this is completely messed up. Somebody is going to need to pay >> back all the technical debt accumulated here and rewrite it all. > > I think we should just remove the client PS support from mac8011 and > drivers instead should implement power save on their own. The current > setup simply don't work. > > My reasoning: > > 1) all power save implementations in hardware are a bit different, which > makes mac80211 implementation complex (how many different designs do we > support now, three or four?) > > 2) Driver authors care only about their driver and the bugs they are > trying to fix with zero interest how it might affect other drivers. They > dump something to mac80211 and then the change is forgotten. > > 3) Power save bugs are difficult to notice, test and debug. So > regressions are easily missed. > > 4) If the power save support is implemented in the driver, they can't > break other drivers and are free to do whatever optimisations/fixes they > want. > > I'm happy to create a patch to remove client PS code from mac80211 if we > so decide. Removal of code from mac80211 is not sufficient, an equivalent replacement for each driver that does use this code is already required. Luis -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html