Re: [PATCH] mac80211: update outdated comment

[Johannes Berg <johannes@sipsolutions.net>]

On Thu, 2026-02-26 at 21:57 +1100, Lachlan Hodges wrote:
> On Tue, Feb 24, 2026 at 10:15:48AM +0800, kexinsun wrote:
> > The function ieee80211_start_scan() was refactored and replaced by
> > __ieee80211_start_scan() in commit f3b85252f081 ("mac80211: fix scan
> > races and rework scanning"). Update the comment in
> > ieee80211_tx_h_check_assoc() accordingly.
> > 
> > Signed-off-by: kexinsun <kexinsun@smail.nju.edu.cn>
> > ---
> >  net/mac80211/tx.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/net/mac80211/tx.c b/net/mac80211/tx.c
> > index 8cdbd417d7be..85c48b42474c 100644
> > --- a/net/mac80211/tx.c
> > +++ b/net/mac80211/tx.c
> > @@ -288,7 +288,7 @@ ieee80211_tx_h_check_assoc(struct ieee80211_tx_data *tx)
> >  		 * sent and we should not get here, but if we do
> >  		 * nonetheless, drop them to avoid sending them
> >  		 * off-channel. See the link below and
> > -		 * ieee80211_start_scan() for more.
> > +		 * __ieee80211_start_scan() for more.
> >  		 *
> >  		 * http://article.gmane.org/gmane.linux.kernel.wireless.general/30089
> 
> Might be good to remove that link aswell and update the comment? Looks
> to be dead.

Hm, yeah, I can't say what this link was now.

I did find where it came from though:
https://lore.kernel.org/all/1237368810.5100.15.camel@johannes.local/

I suspect that maybe it was meant to point to
https://lore.kernel.org/all/20090316085741.GA29986@jm.kir.nu/

but I don't really know.

johannes