Re: [PATCH v3 7/7] mac80211: add DFS support to monitor interface

[Victor Goldenshtein <victorg@ti.com>]

On 10/09/2012 08:42 PM, Johannes Berg wrote:
> On Wed, 2012-10-03 at 15:54 +0200, Goldenshtein, Victor wrote:
>> On Mon, Sep 10, 2012 at 2:26 PM, Johannes Berg
>> <johannes@sipsolutions.net> wrote:
>>> On Wed, 2012-08-08 at 14:53 +0300, Victor Goldenshtein wrote:
>>>> AP mode uses monitor interfaces to transmit
>>>> management frames which were blocked for "radar
>>>> channels" until now.
>>>>
>>>> Stop dropping packets if we are on 'radar channel'
>>>> and the DFS is enabled.
>>>
>>> Umm, no? Check for dfs_en_tx().
>>>
>>
>> no need, the AP is started only after the dfs_en_tx().
>
> but this is on monitor mode, no? and monitor mode isn't used just for
> AP. In fact, AP no longer even uses monitor mode, so you don't need this
> at all since you should be working with a new hostapd version that will
> get the DFS feature ... and then you're working on a current version
> that already no longer uses monitor mode for DFS. Both kernel & hostapd
> have to be new enough, so monitor mode won't be used...
>

right, this patch here from the days AP used monitor mode.
I"ll remove it.


-- 
Thanks,
Victor.