Re: further plan wrt. monitor interfaces

[Ulrich Kunitz <kune@deine-taler.de>]

Johannes Berg wrote:

> > A bunch
> > of parameters are already set with a single function, only config
> > and config_interface need to be replaced by parameter-specific
> > functions. Reset should be removed. 
> 
> Not sure what reset is for.

Now, we are at least two.

> > add_interface() and
> > remove_interface() are comparable to my enable_mac() and
> > disable_mac() as you discovered. 
> 
> Aha, but they're not fully compatible. First of all, you were talking
> about multiple MACs which isn't something we're going to support
> (multiple MAC addresses yes).

I would state we should the support the functionality that the
device supports. Everything else is "form follows function". 

> Secondly, a WDS and AP interface will be
> running with the same MAC address but have quite different
> configuration. Thirdly, always doing lookups by MAC address (even if it
> were possible which it is not due to WDS vs AP) is rather inefficient.

The MAC address lookup issue could be handled. 

Assuming that the device would be able to create beacons on a
specific MAC address, would it need to know about WDS or AP mode?

> > Tell the driver on the real interface, which packets it should
> > report to the mac80211 stack. 
> 
> That's what I proposed.
> 
> > Don't bother the driver with virtual
> > interfaces. The stack should handle it completely transparently.
> 
> What I'm trying to explain to you is that basically you cannot do that.
> AP vs WDS interfaces, lookups by MAC address, beaconing, ...

Beaconing needs to be supported by the device. AP and WDS are
working on the same MAC address, but why should the people know
about it. It doesn't look into the packets transmitted.

> > Don't call the driver before open() or after stop() is called.
> 
> Do we do that? In any case, it's rather orthogonal to the issue at hand.

Sure. Some drivers support the configuration of the channel before
the device is up. There has been the request for the zd1211rw
driver to shadow the configuration in the device data structure.
But I believe that is something the stack should handle.

Ciao,

Uli

-- 
Uli Kunitz