[PATCH] ath10k: refactor monitor code
Michal Kazior
michal.kazior at tieto.com
Tue Apr 8 03:03:15 EDT 2014
On 8 April 2014 08:53, Kalle Valo <kvalo at qca.qualcomm.com> wrote:
> Kalle Valo <kvalo at qca.qualcomm.com> writes:
>
>> Michal Kazior <michal.kazior at tieto.com> writes:
>>
>>> It was possible to create/delete/start/stop
>>> monitor vdev from a few places that were not
>>> exclusively protected against each other. This
>>> resulted in monitor vdev being stopped/removed by
>>> one call origin while another one was expecting it
>>> to continue running.
>>>
>>> For example if CAC was started and interface's
>>> promiscuous mode was toggled monitor vdev was
>>> removed from the driver meaning no radar would be
>>> detected. In additional a warning would be printed
>>> upon CAC completion complaining it tried to stop
>>> non-running monitor vdev.
>>>
>>> The patch simplifies monitor code by removing
>>> IEEE80211_HW_WANT_MONITOR_VIF (which wasn't really
>>> ever needed) and improves state tracking. It also
>>> unifies prints.
>>>
>>> Signed-off-by: Michal Kazior <michal.kazior at tieto.com>
>>
>> Thanks, applied.
>
> Actually there were two checkpatch warnings which I had missed:
>
> drivers/net/wireless/ath/ath10k/mac.c:2587: WARNING: line over 80 characters
> drivers/net/wireless/ath/ath10k/mac.c:2852: WARNING: line over 80 characters
>
> I amended it like this:
>
[...]
Sorry for the trouble and thanks!
Michał
More information about the ath10k
mailing list