Re: [PATCH] mac80211: Fix sw connection monitor and sw scan when STA is connected HE

[Johannes Berg <johannes@sipsolutions.net>]

On Wed, 2021-06-23 at 12:49 +0000, Pkshih wrote:
> > -----Original Message-----
> > From: Johannes Berg [mailto:johannes@sipsolutions.net]
> > Sent: Wednesday, June 23, 2021 6:33 PM
> > To: Pkshih
> > Cc: linux-wireless@vger.kernel.org
> > Subject: Re: [PATCH] mac80211: Fix sw connection monitor and sw scan
> > when STA is connected HE
> > 
> > > Hi mac80211 team,
> > > 
> > > We are working on Realtek 11ax driver, rtw89, not being merged
> > > yet.
> > > This driver uses software connection monitor and software scan,
> > > like
> > > our 11ac driver rtw88,
> > > but we found these two functions don't work properly due to
> > > mac80211
> > > doesn't send
> > > null data frame when STA is connected HE.
> > > 
> > > We search for the reason why it can't send null frame, but we
> > > don't
> > > get the answer.
> > > Possibly, it is because existing HE hardware supports hardware
> > > connection monitor
> > > and scan, or supports TWT to avoid sending null data frame to tell
> > > AP
> > > its PS mode.
> > > Could anyone enlighten me the reason?
> > 
> > It was just a workaround for an iwlwifi specific issue that got left
> > in.
> > I just merged a patch from Felix to resolve that.
> > 
> 
> I check Felix's patch:
> https://patchwork.kernel.org/project/linux-wireless/patch/20210619101517.90806-1-nbd@nbd.name/
> 
> 
> I think we should revert the commit f39b07fdfb68 ("mac80211: HE STA
> disassoc due to QOS NULL not sent"),
> which fixes connection monitor due to QOS NULL not send.
> 

I agree, that makes sense.

> 
> Another place is ieee80211_send_null_response(); I'm not sure if we
> can remove
> the checking as well.

Yes, I think so.

Want to send patch(es) for that?

johannes