Search Linux Wireless

Re: [PATCH v2 0/3] mac80211: fix rate_control_send_low warnings for delbas

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

 



On Tue, Oct 5, 2010 at 12:57 AM, Johannes Berg
<johannes@xxxxxxxxxxxxxxxx> wrote:
> On Mon, 2010-10-04 at 16:50 -0700, Luis R. Rodriguez wrote:
>> On Mon, Oct 4, 2010 at 3:47 PM, Jouni Malinen <j@xxxxx> wrote:
>> > On Mon, Oct 04, 2010 at 02:07:22PM -0700, Luis R. Rodriguez wrote:
>> >> Jouni, just to be clear so you are fine with dropping explicitly the
>> >> tear down of the BA agreement to the old AP?
>> >
>> > If you are talking about not transmitting a frame, then yes, but
>> > obviously, we would still need to locally tear down whatever we have set
>> > up..
>>
>> Sure.
>
> Right -- exactly what my (untested!!) patch did.
>
>> > I think it would be a good default policy to not send any frames to the
>> > old AP after we have decided to roam to a new one. If this does not work
>> > well in some networks, I would like to here more detailed description on
>> > what exactly is happening.
>>
>> No that's fine, I just was curious how the bridge stuff you described
>> was handled in enterprise networks, you would know better :)
>
> Look at net/mac80211/cfg.c - ieee80211_send_layer2_update.
>
>> OK -- so lets go with Johanne's patch instead of patch 2 and 3 as he
>> suggested :)
>
> Have you tested it?

Tested, it works and resolves the issue if I drop my patch 2 and 3 and
instead use yours.

  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


[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