Re: [PATCH] mac80211: don't clear all tx flags when requeing

[Johannes Berg <johannes@sipsolutions.net>]

On Fri, 2015-07-17 at 11:07 +0200, Johannes Berg wrote:
> 
> > > Once (a) is not stripped (b) becomes important
> > > because it's the main condition to clear the
> > > Service Period bit of the station when Tx status
> > > for the released frame is reported back.
> > > 
> > > This problem was observed with ath9k acting as P2P
> > > GO in some testing scenarios but isn't limited to
> > > it. AP operation with mac80211 based Tx A-MPDU
> > > control combined with clients using PS-Poll frames
> > > is subject to this race.
> > 
> > I'm not sure I quite understand - how is the aggregation teardown
> > causing frame filtering?
> > 
> 
> Never mind, I was looking at the wrong code. I'll apply this.
> 

However, I'd like to ask you to look at this again - I can see how it
fixes the problem now, but it seems like a fairly unreliable fix since
the frame is sent through TX processing again, and you're relying on
that preserving a flag that's otherwise marked temporary...

So I think it may be better to adjust the station flags in this case to
let a new (the same) frame be the ps-poll response again.

johannes