Re: rtlwifi handling of sequence numbers with aggregation

On 08/29/2017 10:18 AM, Larry Finger wrote:

On 08/25/2017 09:15 PM, Larry Finger wrote:

quoted

Based on my prejudices, I would suspect the USB driver before that of 
the PCI version. I have BCc'd my contact at Realtek to see what he has 
to say on the issue.

My contact at Realtek replies that the PCI code is correct. He further 
states that "Since mac80211 uses ieee80211_tx_next_seq() to store next 
sequence number in sta->tid_seq[tid] and fill the sequence number in 
AMPDU parameters, I think driver doesn't need to maintain the seq_number 
anymore. So, let's remove 'u16 seq_number' from 'struct rtl_tid_data', 
please refer to attachment." I have not had a chance to test his patch, 
but I trust his analysis.

I don't see any attachment, but I'm happy with a correct solution if 
you'll work with them on getting that pushed in.

Cheers,
Jes