From: Felix Fietkau <nbd@openwrt.org>
To: ath9k-devel@lists.ath9k.org
Subject: [ath9k-devel] ath9k in wireless-testing won't work in AP mode
Date: Sat, 30 Jan 2010 21:46:51 +0100 [thread overview]
Message-ID: <4B649ABB.1040203@openwrt.org> (raw)
In-Reply-To: <1264883839.26739.47.camel@mj>
On 2010-01-30 9:37 PM, Pavel Roskin wrote:
> On Sat, 2010-01-30 at 21:10 +0100, Felix Fietkau wrote:
>
>> The workaround value of '64' is actually wrong. When I had trouble
>> associating in 2.4 GHz in a case where the slot time was actually set
>> correctly, I simply used it, because that's what was being set in the
>> initvals. We shouldn't base the slot time on this though - the initvals
>> don't do this either.
>
> I could reduce the minimal value of acktimeout from 64 to 20. With 20,
> I get a reliable communication every time. With 19, it's doesn't work
> ever. The distance between STA and AP is about 1 meter.
>
>> The slottime == -1 thing is obviously a bug, and I'll send a patch to
>> fix it later.
>
> It turns out that simply increasing the initial value to 54 is not
> enough, as ah->slottime will be set to 9 from sc->beacon.slottime, so
> acktimeout becomes 19.
That's why my patch forces it to a minimum of 64 just before it gets set.
> But using the value of 10 instead of 9 for sc->beacon.slottime does the
> trick. That's the minimal patch that works for me.
>
> Perhaps we should be using ATH9K_SLOT_TIME_9 there (see mac.h) and
> define it to 10.
That value would still be wrong, though. According to the 802.11 docs,
the slot time for 2.4 GHz has to be set to 9 usec, not 10.
If this is a bug in the hardware, we need to know about the implications
before we pick a value that just happened to work in our tests, but may
not work in all cases.
slottime=9, acktimeout=64 is what the initvals use, so that's tested by
Atheros. If we continue to receive no comments from Atheros on this
issue, we should probably use those values by default.
By the way, 5 GHz seems to be unaffected by this issue, which makes this
a little suspicious.
I've Cc'd Luis, maybe he can forward this to the right people.
- Felix
next prev parent reply other threads:[~2010-01-30 20:46 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-01-29 23:05 [ath9k-devel] ath9k in wireless-testing won't work in AP mode Pavel Roskin
2010-01-29 23:34 ` Felix Fietkau
2010-01-30 19:39 ` Pavel Roskin
2010-01-30 20:10 ` Felix Fietkau
2010-01-30 20:37 ` Pavel Roskin
2010-01-30 20:46 ` Felix Fietkau [this message]
2010-01-30 21:11 ` Pavel Roskin
2010-02-03 0:08 ` Luis R. Rodriguez
2010-02-03 0:18 ` Felix Fietkau
2010-02-03 0:27 ` Luis R. Rodriguez
2010-02-03 0:35 ` Felix Fietkau
2010-02-03 0:45 ` Luis R. Rodriguez
2010-02-03 3:29 ` Pavel Roskin
2010-02-03 17:16 ` Luis R. Rodriguez
2010-02-03 4:22 ` Sujith
2010-02-10 10:51 ` Jouni Malinen
2010-02-10 17:33 ` Peter Stuge
2010-02-11 9:12 ` Peter Stuge
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=4B649ABB.1040203@openwrt.org \
--to=nbd@openwrt.org \
--cc=ath9k-devel@lists.ath9k.org \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.