On 28 August 2013 06:23, Kalle Valo <kvalo@xxxxxxxxxxxxxxxx> wrote: > Michal Kazior <michal.kazior@xxxxxxxxx> 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@xxxxxxxxx> > > 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 > 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. I'm worried this PS stuff is broken on ath10k anyway because ath10k modifies tx skbuffs for QoS workaround (QoS Control must be removed before submitting to FW). Michał. -- 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
