linux-wireless.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Johannes Berg <johannes@sipsolutions.net>
To: Bruno Randolf <br1@einfach.org>
Cc: linville@tuxdriver.com, linux-wireless@vger.kernel.org
Subject: Re: [PATCH 2/2] mac80211: Initialize IBSS basic rates according to band
Date: Mon, 12 Apr 2010 10:40:52 +0200	[thread overview]
Message-ID: <1271061652.3877.8.camel@jlt3.sipsolutions.net> (raw)
In-Reply-To: <201004121734.39700.br1@einfach.org>

On Mon, 2010-04-12 at 17:34 +0900, Bruno Randolf wrote:

> > No, you don't understand. It will break _talking to_ 11b only stations
> > when the local station is 11g capable.
> 
> i understood that this is what you mean. note that i said *without* the other 
> patch! the first patch breaks talking to 11b only stations, true. the second 
> doesn't because:
> 
> as it is now ieee80211_mandatory_rates(local, sband->band) returns 1 (1Mbps) 
> in a 2GHz channel. which is o.k. for B only stations.

Oh ok, right.

> before we unconditionally used 3 (1Mbps, 2 Mbps) as basic rates. this clearly 
> is wrong in the 5GHz band.

No, it's fine. Look again at how this works.

> > > anyhow, i wouldn't mind adding something to configure this from
> > > userspace. can you point me to the right direction? i mean i can add
> > > something to IBSS setup but as i said i can't believe this is only
> > > relevant to IBSS mode.
> > 
> > What other modes do you think it is relevant to?
> 
> i'm not so familiar with the way STA and AP operate, but i thought that also 
> there the distiction between B, B/G and pure-G could make sense.

Not unless you want your 11g+ STA to operate in 11b-only mode.

johannes


  reply	other threads:[~2010-04-12  8:40 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-04-12  7:36 [PATCH 1/2] mac80211: Use G mandatory rates in 2GHz band Bruno Randolf
2010-04-12  7:37 ` [PATCH 2/2] mac80211: Initialize IBSS basic rates according to band Bruno Randolf
2010-04-12  7:41   ` Johannes Berg
2010-04-12  7:58     ` Bruno Randolf
2010-04-12  8:10       ` Johannes Berg
2010-04-12  8:34         ` Bruno Randolf
2010-04-12  8:40           ` Johannes Berg [this message]
2010-04-13  0:16             ` Bruno Randolf
2010-04-13  7:21               ` Johannes Berg
2010-04-13  7:56                 ` Bruno Randolf
2010-04-12  7:39 ` [PATCH 1/2] mac80211: Use G mandatory rates in 2GHz band Johannes Berg

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=1271061652.3877.8.camel@jlt3.sipsolutions.net \
    --to=johannes@sipsolutions.net \
    --cc=br1@einfach.org \
    --cc=linux-wireless@vger.kernel.org \
    --cc=linville@tuxdriver.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).