Johannes Berg wrote: > On Mon, 2008-10-20 at 02:01 -0700, Luis R. Rodriguez wrote: > > > > Well the fact that it's in tx_h_sequence is a bit weird, but we should > > > be able to get around not using an extra bit in skb I think. Especially > > > for this since it's only really between the driver and mac80211, and > > > after a requeue the ampdu status might actually change. > > > > Good point. I failed to see that. So in that case it should be as > > sujith had it before my suggestion and just handle it as he had it in > > the tx handlers. The problem though is that the RC algorithm should be > > the one to determine if an skb is part of an ampdu for aggregation or > > not IMHO. Actually this is the real question and hot topic. > > > > When should we aggregate and should mac80211 decide that or be left to the RC. > > No, the RC algorithm cannot decided that, the RC algorithm can merely > decide when to set up/tear down aggregation sessions. When a session is > up all packet belonging to that TID should go into it, surely? Right, ath9k currently tries to open an aggregation session for each station that it is associated with. RC doesn't determine anything now. Sujith -- 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