On Fri, Feb 10, 2012 at 1:06 AM, Luis R. Rodriguez <mcgrof@xxxxxxxxxxxxx> wrote:
> On Thu, Jan 26, 2012 at 4:38 AM, Victor Goldenshtein <victorg@xxxxxx> wrote:
>> diff --git a/include/net/mac80211.h b/include/net/mac80211.h
>> index d08b808..b0f4e67 100644
>> --- a/include/net/mac80211.h
>> +++ b/include/net/mac80211.h
>> @@ -841,12 +841,16 @@ struct ieee80211_conf {
>> * the driver passed into mac80211.
>> * @block_tx: Indicates whether transmission must be blocked before the
>> * scheduled channel switch, as indicated by the AP.
>> + * @post_switch_block_tx: Indicates whether transmission must be blocked after
>> + * the scheduled channel switch, this should be set if the target channel
>> + * is DFS channel.
>
> Here you say that post_switch_block_tx is required for DFS channels
> but yet you do not check for this. You should instead clarify that
> this is required on the nl80211.h command, and also add the sanity
> check there.
>
> Luis
The AP channel switch functionality is not only for DFS, in the future
other usermode applications/clients might also use it (e.g. ACS ;)).
In addition even the DFS-Master may switch to a non DFS channel - so
this post_switch_block_tx won't be set.
--
Thanks,
Victor.
--
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
