From: "Luka Gejak" <luka.gejak@linux.dev>
To: "Mehmet Fide" <mehmet.fide@gmail.com>,
"Toke Høiland-Jørgensen" <toke@toke.dk>
Cc: "Johannes Berg" <johannes@sipsolutions.net>,
linux-wireless@vger.kernel.org, linux-kernel@vger.kernel.org,
mehmet.fide@screeningeagle.com
Subject: Re: [PATCH ath-current] wifi: ath9k_htc: clear csa_vif when a switch is aborted
Date: Tue, 27 Oct 2026 10:00:04 +0000 [thread overview]
Message-ID: <cf94eda9237629a07b9d5064870d71fe417d5483@linux.dev> (raw)
In-Reply-To: <20261026152537.1272836-1-mehmet.fide@gmail.com>
October 26, 2026 at 4:25 PM, "Mehmet Fide" <mehmet.fide@gmail.com> wrote:
>
> From: Mehmet Fide <mehmet.fide@screeningeagle.com>
>
> priv->csa_vif is cleared only when the countdown completes or the
> interface is removed. When mac80211 aborts a switch and stops
> beaconing, as ieee80211_stop_ap() and ieee80211_stop_mesh() do, it
> stays set, and the next switch on the interface hits the WARN_ON() in
> ath9k_htc_channel_switch_beacon().
>
> Clear it when beaconing stops and the switch is no longer active.
> Off channel, beaconing stops with the switch still active and the
> countdown continues afterwards, so keep it then.
>
> Fixes: f0e449627e4a ("ath9k_htc: add support of channel switch")
> Reported-by: Luka Gejak <luka.gejak@linux.dev>
> Closes: https://lore.kernel.org/linux-wireless/c8bc5affa75d7519b424ea98c07c1d6b65942347@linux.dev/
> Signed-off-by: Mehmet Fide <mehmet.fide@screeningeagle.com>
> ---
Not an expert on ath family but, to my knowledge this is good.
Reviewed-by: Luka Gejak <luka.gejak@linux.dev>
Best regards,
Luka Gejak
prev parent reply other threads:[~2026-10-27 10:00 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-10-26 15:25 Mehmet Fide
2026-10-27 10:00 ` Luka Gejak [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=cf94eda9237629a07b9d5064870d71fe417d5483@linux.dev \
--to=luka.gejak@linux.dev \
--cc=johannes@sipsolutions.net \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-wireless@vger.kernel.org \
--cc=mehmet.fide@gmail.com \
--cc=mehmet.fide@screeningeagle.com \
--cc=toke@toke.dk \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®