Search Linux Wireless

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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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


[Index of Archives]     [Linux Host AP]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [Linux Kernel]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]
  Powered by Linux