[PATCH] nl80211: clear VHT and HE MCS rates in legacy beacon tx rate

Rajkumar Manoharan rmanohar at codeaurora.org
Sat Oct 3 17:33:39 EDT 2020


On 2020-10-03 04:01, Johannes Berg wrote:
> On Sat, 2020-10-03 at 02:25 -0700, Rajkumar Manoharan wrote:
>> While configuring legacy beacon rate, VHT and HE MCS rates must be 
>> cleared.
>> Otherwise the default MCS rates is filled by cfg80211 while validating 
>> beacon
>> tx rate netlink attribute. This issue is exposed by 
>> ap_beacon_rate_legacy2
>> test case while validating HE MCS beacon tx rate support.
> 
> As I said with the kernel patch, I don't think that the kernel's side 
> is
> approaching this the right way - better to default to no rate if the
> attribute isn't present, at least for the purposes of beacon rate
> settings. Maybe not for other purposes.
> 
Understood. I submit the patch for cfg80211. Shall we retain this patch 
or drop it?

Rajkumar



More information about the Hostap mailing list