linux-wireless.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Johannes Berg <johannes@sipsolutions.net>
To: Sujith Manoharan <c_manoha@qca.qualcomm.com>
Cc: Adrian Chadd <adrian@freebsd.org>,
	"Manoharan, Rajkumar" <rmanohar@qca.qualcomm.com>,
	linux-wireless <linux-wireless@vger.kernel.org>
Subject: Re: mac80211 20/40 coexist
Date: Mon, 19 Mar 2012 09:26:33 +0100	[thread overview]
Message-ID: <1332145593.3359.6.camel@jlt3.sipsolutions.net> (raw)
In-Reply-To: <20326.44233.811876.892785@gargle.gargle.HOWL>

On Mon, 2012-03-19 at 09:19 +0530, Sujith Manoharan wrote:
> Adrian Chadd wrote:
> > On 18 March 2012 03:19, Johannes Berg <johannes@sipsolutions.net> wrote:
> > 
> > > I'm not sure I can believe this -- surely the firmware has to have a way
> > > of dealing with stations that can't do 40 MHz and stations that can, so
> > > switching between the two doesn't seem like a major proposal? We also
> > > don't currently handle the action frames for that, but it seems like we
> > > should.
> > >
> > > Note, however, that we're now discussing something already done --
> > > Paul's patch already removed the channel type update when all we wanted
> > > was update the RC.
> > 
> > When are stations being notified of a CWM change?
> > 
> > I know FreeBSD handles it with a big stick at the moment - it just
> > does a full interface reset and channel change. I don't necessarily
> > like it, but I haven't yet sat down to look at when and why this is
> > occuring.
> 
> For AP mode, we currently don't have dynamic CWM - and if it is ever implemented,
> it would probably be in hostapd.

Yeah, though I think if we implement the action frame in mac80211 we
could handle that in AP mode too? Might be easier not to though.

> For station mode, HT bandwidth changes can be notified via beacons, probe responses
> or action frames. mac80211 currently processes beacons and the HT operation element.
> 
> Also, only legacy CSA announcements are handled by mac80211 right now, the 11n
> extensions to the CSA action frame are yet to be implemented.

Yeah, at least for VHT I'd guess those will become necessary.

> As far as ath9k/ath9k_htc is concerned, I think we have to reset/reconfigure the HW
> when a channel switch happens, since we reprogram various MAC/PHY registers based on
> the HT bandwidth. Both ath9k and ath9k_htc update their rate control modules
> correctly - ath9k via the rate_update() callback and ath9k_htc using BSS_CHANGED_HT
> in bss_info_changed().

I'm not sure ath9k_htc is behaving correctly. Ok I should say that
differently -- it's probably behaving "correctly" wrt. whatever mac80211
did before, but we're changing the way mac80211 behaves and I believe
that to be (more) correct, see my other email.

johannes


  parent reply	other threads:[~2012-03-19  8:26 UTC|newest]

Thread overview: 24+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-03-15 13:30 mac80211 20/40 coexist Johannes Berg
2012-03-15 20:49 ` Manoharan, Rajkumar
2012-03-16 13:01   ` Johannes Berg
2012-03-16 13:46     ` Johannes Berg
2012-03-16 18:39     ` Manoharan, Rajkumar
2012-03-18 10:19       ` Johannes Berg
2012-03-18 19:19         ` Adrian Chadd
2012-03-19  3:49           ` Sujith Manoharan
2012-03-19  4:49             ` Adrian Chadd
2012-03-19  4:59               ` Sujith Manoharan
2012-03-19  8:26             ` Johannes Berg [this message]
2012-03-19 10:08               ` Sujith Manoharan
2012-03-19 10:13                 ` Johannes Berg
2012-03-19  8:24           ` Johannes Berg
2012-03-19  2:55         ` Sujith Manoharan
2012-03-19  8:23           ` Johannes Berg
2012-03-19 10:04             ` Sujith Manoharan
2012-03-19 10:09               ` Johannes Berg
2012-03-19 10:32                 ` Sujith Manoharan
2012-03-19 11:12                   ` Johannes Berg
2012-03-19 12:18                     ` Johannes Berg
2012-03-20  4:39                       ` Sujith Manoharan
2012-03-20 11:32                         ` Johannes Berg
2012-03-20 15:26                           ` Sujith Manoharan

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=1332145593.3359.6.camel@jlt3.sipsolutions.net \
    --to=johannes@sipsolutions.net \
    --cc=adrian@freebsd.org \
    --cc=c_manoha@qca.qualcomm.com \
    --cc=linux-wireless@vger.kernel.org \
    --cc=rmanohar@qca.qualcomm.com \
    /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;
as well as URLs for NNTP newsgroup(s).