From: Jonas Gorski <jogo@openwrt.org>
To: Yogesh Ashok Powar <yogeshp@marvell.com>
Cc: "linux-wireless@vger.kernel.org" <linux-wireless@vger.kernel.org>,
"John W. Linville" <linville@tuxdriver.com>,
Nishant Sarmukadam <nishants@marvell.com>,
Lennert Buytenhek <buytenh@wantstofly.org>,
Johannes Berg <johannes@sipsolutions.net>,
Adrian Chadd <adrian@freebsd.org>,
Felix Fietkau <nbd@openwrt.org>
Subject: Re: [PATCH] mwl8k: remove nonstandard rate 72 Mbps
Date: Mon, 15 Apr 2013 09:53:00 +0200 [thread overview]
Message-ID: <20130415095300.000003d2@unknown> (raw)
In-Reply-To: <20130415063838.GB801@markyou.marvell.com>
On Mon, 15 Apr 2013 12:08:38 +0530
Yogesh Ashok Powar <yogeshp@marvell.com> wrote:
> On Sun, Apr 14, 2013 at 05:11:58AM -0700, Jonas Gorski wrote:
> > This rate causes an overflow in the extended rates IE's data rate
> > field, with the overflowing bit setting the Basic Rate Set
> > membership. This results in a bogus 8 Mpbs basic rate, making
> > clients checking them refuse association.
> >
> > Since the rate is likely unused anyway (HT will yield better rates
> > between supporting chips), we can just remove it.
> >
> > This fixes association from wpa_supplicant and Android 4.x and
> > newer.
>
> Hi Jonas,
>
> If you are using hostapd, have you tried using 'supported_rates'
> and/or 'basic_rates' being set to correct rates in the hostapd.conf.
Not sure if that helps. The main problem is that this rate is
unadvertisable in Beacons, as (Extended) Supported Rates (802.11-2012
8.4.2.3) encodes the rates as u8, with seven bits for the rate in units
of 500 kb/s, which isn't enough for 72 Mbps; the maximum is 63.5 Mbps.
The result is that it spills into the MSB, marking it as a "Basic Rate"
even though it isn't marked as such by hostapd. It's just that 72 Mbps
=> 0x90 => "8 Mbps, Basic Rate".
This is partially a bug in hostapd, it should probably reject rates not
encodable.
So AFAICT the only way to use it in hostapd is to not use it.
Which makes me wonder how or if this rate is usable at all in 802.11
context, without a proprietary way of telling the other end that this
rate is available, and therefore if it even makes sense to keep.
Jonas
prev parent reply other threads:[~2013-04-15 7:53 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-04-13 10:31 [RFC/RFT] mwl8k: don't expose non-standard rates Jonas Gorski
2013-04-13 14:29 ` Adrian Chadd
2013-04-13 16:18 ` Johannes Berg
2013-04-13 18:59 ` Adrian Chadd
2013-04-13 22:08 ` Felix Fietkau
2013-04-14 10:49 ` Jonas Gorski
2013-04-14 12:11 ` [PATCH] mwl8k: remove nonstandard rate 72 Mbps Jonas Gorski
2013-04-15 6:38 ` Yogesh Ashok Powar
2013-04-15 7:53 ` Jonas Gorski [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=20130415095300.000003d2@unknown \
--to=jogo@openwrt.org \
--cc=adrian@freebsd.org \
--cc=buytenh@wantstofly.org \
--cc=johannes@sipsolutions.net \
--cc=linux-wireless@vger.kernel.org \
--cc=linville@tuxdriver.com \
--cc=nbd@openwrt.org \
--cc=nishants@marvell.com \
--cc=yogeshp@marvell.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).