[PATCH v2 1/4] ath10k: synchronize tx/rx reporting to mac80211

Kalle Valo kvalo at qca.qualcomm.com
Wed Aug 28 07:04:01 EDT 2013


Michal Kazior <michal.kazior at tieto.com> writes:

> On 28 August 2013 06:23, Kalle Valo <kvalo at qca.qualcomm.com> wrote:
>> Michal Kazior <michal.kazior at tieto.com> writes:
>>
>>> According to mac80211 docs tx and rx reporting
>>> routines must not run concurrently unless they are
>>> _irqsafe variants.
>>>
>>> This doesn't fix any visible bug but is apparently
>>> the right thing to do as per the documentation.
>>>
>>> Signed-off-by: Michal Kazior <michal.kazior at tieto.com>
>>
>> I had a quick chat with Johannes and Sujith about this. The concurrency
>> requirements are for the STA PS race in AP mode (see
>> ieee80211_handle_filtered_frame()).
>>
>> I think we should drop this frame, revisit later and properly analyse

"drop this frame". Heh, of course I was trying to say "drop this
patch" :)

>> how to fix the race. But it would be good add this to the todo list:
>>
>> http://wireless.kernel.org/en/users/Drivers/ath10k#TODO
>
> I've added this to the TODO list.

Thanks!

-- 
Kalle Valo



More information about the ath10k mailing list