All of lore.kernel.org
 help / color / mirror / Atom feed
* Mesh mode broken since 2.6.34-rc1
@ 2010-03-24  9:06 Ледовских Николай Валерьевич
  2010-03-25  1:16 ` Javier Cardona
  0 siblings, 1 reply; 5+ messages in thread
From: Ледовских Николай Валерьевич @ 2010-03-24  9:06 UTC (permalink / raw)
  To: linux-wireless

Mesh mode does not work any more. I have some x86 MPs with 2.6.33.1, which work fine. 
But there are establishment problems with 2.6.34-rc1. 
The peering status changes between LISTEN, OPN_SNT and HOLDING, but does not reach ESTAB.

-- 
Best regards, Nick.

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: Mesh mode broken since 2.6.34-rc1
  2010-03-24  9:06 Mesh mode broken since 2.6.34-rc1 Ледовских Николай Валерьевич
@ 2010-03-25  1:16 ` Javier Cardona
  2010-03-26 18:14   ` Javier Cardona
  0 siblings, 1 reply; 5+ messages in thread
From: Javier Cardona @ 2010-03-25  1:16 UTC (permalink / raw)
  To: Ледовских
	Николай
	Валерьевич
  Cc: linux-wireless, devel, Porsch, Marco, steve.derosier

Nick,

2010/3/24 Ледовских Николай Валерьевич <nledovskih@mail.ru>:
> Mesh mode does not work any more. I have some x86 MPs with 2.6.33.1, which work fine.
> But there are establishment problems with 2.6.34-rc1.
> The peering status changes between LISTEN, OPN_SNT and HOLDING, but does not reach ESTAB.

Yes, confirmed on our testbed that wireless-testing commit
60b341b778cc2929df16c0a504c91621b3c6a4ad is good.  We'll investigate.

Thanks,

Javier


-- 
Javier Cardona
cozybit Inc.
http://www.cozybit.com

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: Mesh mode broken since 2.6.34-rc1
  2010-03-25  1:16 ` Javier Cardona
@ 2010-03-26 18:14   ` Javier Cardona
  2010-03-26 21:10     ` Johannes Berg
  0 siblings, 1 reply; 5+ messages in thread
From: Javier Cardona @ 2010-03-26 18:14 UTC (permalink / raw)
  To: Ледовских
	Николай
	Валерьевич
  Cc: linux-wireless, devel, Porsch, Marco, steve.derosier, Johannes Berg

Hi Nick,

The commit that broke the mesh seems to be:

commit 8404080568613d93ad7cf0a16dfb68459b42a264
Author: Johannes Berg <johannes@sipsolutions.net>
Date:   Mon Feb 15 12:46:39 2010 +0200

    mac80211: reject unhandled action frames

    802.11-2007 7.3.1.11 mandates that we need to
    reject action frames we don't handle by setting
    the 0x80 bit in the category and returning them
    to the sender, so do that. In AP mode, hostapd
    is responsible for this.

    Additionally, drop completely malformed action
    frames or ones that should've been encrypted as
    unusable, userspace shouldn't see those.

    Signed-off-by: Johannes Berg <johannes@sipsolutions.net>
    Signed-off-by: Jouni Malinen <jouni.malinen@atheros.com>
    Signed-off-by: John W. Linville <linville@tuxdriver.com>

After that commit we observe invalid action category codes (0x9e) in
response to valid peer link open action frames (0x1e) and so no peer
links are established.  We don't have a fix at this time.

Cheers,

Javier


2010/3/24 Javier Cardona <javier@cozybit.com>:
> Nick,
>
> 2010/3/24 Ледовских Николай Валерьевич <nledovskih@mail.ru>:
>> Mesh mode does not work any more. I have some x86 MPs with 2.6.33.1, which work fine.
>> But there are establishment problems with 2.6.34-rc1.
>> The peering status changes between LISTEN, OPN_SNT and HOLDING, but does not reach ESTAB.
>
> Yes, confirmed on our testbed that wireless-testing commit
> 60b341b778cc2929df16c0a504c91621b3c6a4ad is good.  We'll investigate.
>
> Thanks,
>
> Javier
>
>
> --
> Javier Cardona
> cozybit Inc.
> http://www.cozybit.com
>



-- 
Javier Cardona
cozybit Inc.
http://www.cozybit.com

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: Mesh mode broken since 2.6.34-rc1
  2010-03-26 18:14   ` Javier Cardona
@ 2010-03-26 21:10     ` Johannes Berg
  2010-03-26 21:16       ` Javier Cardona
  0 siblings, 1 reply; 5+ messages in thread
From: Johannes Berg @ 2010-03-26 21:10 UTC (permalink / raw)
  To: Javier Cardona
  Cc: Ледовских
	Николай
	Валерьевич,
	linux-wireless, devel, Porsch, Marco, steve.derosier

Hi Javier,

> commit 8404080568613d93ad7cf0a16dfb68459b42a264
> Author: Johannes Berg <johannes@sipsolutions.net>
> Date:   Mon Feb 15 12:46:39 2010 +0200
> 
>     mac80211: reject unhandled action frames
> 
>     802.11-2007 7.3.1.11 mandates that we need to
>     reject action frames we don't handle by setting
>     the 0x80 bit in the category and returning them
>     to the sender, so do that. In AP mode, hostapd
>     is responsible for this.
> 
>     Additionally, drop completely malformed action
>     frames or ones that should've been encrypted as
>     unusable, userspace shouldn't see those.
> 
>     Signed-off-by: Johannes Berg <johannes@sipsolutions.net>
>     Signed-off-by: Jouni Malinen <jouni.malinen@atheros.com>
>     Signed-off-by: John W. Linville <linville@tuxdriver.com>
> 
> After that commit we observe invalid action category codes (0x9e) in
> response to valid peer link open action frames (0x1e) and so no peer
> links are established.  We don't have a fix at this time.

Whooops, good catch, this actually affects mlme.c as well! I'll fix
this, but I'm unlikely to get to it today and am travelling over the
weekend. For reference, the problem is that in rx.c we reject unknown
action frames in ieee80211_rx_h_action(), but the mesh/mlme code only
gets to see the action frames it wants in ieee80211_rx_h_mgmt() which
comes after rx_h_action. The proper fix will probably be to make
ieee80211_rx_h_action() aware of the frames that mesh/mlme wants to
handle it and not reject then.

johannes


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: Mesh mode broken since 2.6.34-rc1
  2010-03-26 21:10     ` Johannes Berg
@ 2010-03-26 21:16       ` Javier Cardona
  0 siblings, 0 replies; 5+ messages in thread
From: Javier Cardona @ 2010-03-26 21:16 UTC (permalink / raw)
  To: Johannes Berg
  Cc: Ледовских
	Николай
	Валерьевич,
	linux-wireless, devel, Porsch, Marco, Steve deRosier

Johannes,

> The proper fix will probably be to make
> ieee80211_rx_h_action() aware of the frames that mesh/mlme wants to
> handle it and not reject then.

Thanks, Johannes.  We are working on a patch along these lines.  No
need to trouble yourself except for reviewing it when we send it.

Cheers,

Javier



-- 
Javier Cardona
cozybit Inc.
http://www.cozybit.com

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2010-03-26 21:16 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-03-24  9:06 Mesh mode broken since 2.6.34-rc1 Ледовских Николай Валерьевич
2010-03-25  1:16 ` Javier Cardona
2010-03-26 18:14   ` Javier Cardona
2010-03-26 21:10     ` Johannes Berg
2010-03-26 21:16       ` Javier Cardona

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.