On Wed, 2010-10-06 at 13:48 -0700, Luis R. Rodriguez wrote: > On Wed, Oct 6, 2010 at 12:19 PM, Johannes Berg > <johannes@xxxxxxxxxxxxxxxx> wrote: > > On Wed, 2010-10-06 at 12:12 -0700, Luis R. Rodriguez wrote: > > > >> Good question, will ieee80211_offchannel_ps_disable() possibly be > >> called during suspend? I can imagine its possible. So how about we > >> check for local->quiescing prior to lifting the flag only: > > > > Or suspended? > > If we're suspended we would be quiesced so I do not see us calling > ieee80211_offchannel_ps_disable() then, but we can also just bail out > if quescing or suspended on ieee80211_offchannel_ps_disable() and > friends. Oh, wait, yeah, while quiescing we stop the workqueue so we can't be processing this, so we should be fine. We might still be off-channel while suspending which is odd, and then we might lift the BA block earlier than you'd want. But then again, come to think of it, why are you doing this anyway? I'm not convinced of the whole idea, it's all racy. I'm starting to think that this patch won't help anything since we'll still be racy and start BA agreements just before we block, _and_ if we're a station and going off-channel we'll be going into powersave which means we won't actually get the addBA frame until we return. johannes -- 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