Re: rt2x00 mesh support

[Johannes Berg <johannes@sipsolutions.net>]

[-- Attachment #1: Type: text/plain, Size: 1149 bytes --]

On Wed, 2009-03-18 at 13:48 +0100, Ivo van Doorn wrote:

> Excellent thanks. :)
> 
> Johannes, my previous assumption is apparently correct,
> The following is happening:
> 
> ieee80211_open()
> 	-> ops->add_interface()
> 	-> ieee80211_start_mesh()
> 		->  ieee80211_if_config()
> 			-> ops->config_interface()
> 	-> ieee80211_hw_config()
> 		-> ops->config()
> 
> Because config_interface() is called before config() the radio
> is not enabled at the moment config_interface() is called and
> the call to config() will effectively disable beaconing.

Ah. I thought you were saying ->start() came after ->config_interface(),
->config() coming after it makes no sense either but we haven't even
tried to avoid that so far... In fact, it is perfectly legal for
->config() to be called after ->config_interface() when, for example,
you change the channel. Not that I disagree -- we should be setting the
beacon interval before enabling beaconing...

What exactly is the problem here? The fact that we don't enable the
radio until after having configured beaconing? I'm not sure how to solve
this, to be honest.

johannes

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 836 bytes --]