Re: [PATCH] mac80211: Deny new BA agreements from being started during offchannel operation

[Johannes Berg <johannes@sipsolutions.net>]

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@sipsolutions.net> 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