On Sat, 2020-10-03 at 02:30 -0700, Rajkumar Manoharan wrote: > On 2020-09-18 05:05, Johannes Berg wrote: > > On Mon, 2020-09-14 at 21:19 -0700, Rajkumar Manoharan wrote: > > > This allows an option to configure a single HE MCS beacon tx rate. > > > > This breaks the ap_beacon_rate_legacy2 hwsim test but I'm not sure why, > > please check. > > > Ah... Thanks for reporting... Hostapd has to clear VHT/HE rates while > setting > legacy beacon tx rate. Otherwise the default he_mcs is filled and > causing hwsim failure. > Posted a fix[1] for review. Not sure that makes sense to me - why wouldn't the kernel side just default to "no HE MCS" if the attribute isn't present at all?!? johannes