Re: [RFC 0/3] mac80211: new API for handling broadcast / multicast on sw scan

[Johannes Berg <johannes@sipsolutions.net>]

On Sat, 2010-08-28 at 03:13 -0400, Luis R. Rodriguez wrote:
> mac80211's sw scan doesn't play nice with buffered broadcast and
> multicast frames. Since drivers are required to keep track of the DTIM
> count 

I said they were required for powersave, not necessarily for scanning.

> this series provides some new basic functionality to let drivers
> help mac80211 make more prudent decisions for sw scan.

I'm not convinced of this. Why does this need to be so complicated?
Also, some of your comments seem like you're confused about being an AP
vs. being associated to one?

What are you actually trying to achieve? It seems mac80211 can, no
matter what the driver is
 - wait for DTIM beacons, and the end of mcast traffic, before going off
   channel (should be relevant for p2p as well)
 - attempt to do better wrt. scheduling between DTIM, by scheduling
   closer (but if DTIM period is 1, this may fail)
 - subject to the driver advertising guaranteed TX status, it can also
   wait for the nullfunc ACK -- you should also implement flush which
   will probably improve things by itself for ath9k

The other things in your patch set I don't understand, but those I think
should be done more generically?

johannes