linux-wireless.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: "Lukáš Turek" <8an@praha12.net>
To: "Luis R. Rodriguez" <lrodriguez@atheros.com>
Cc: Luis Rodriguez <Luis.Rodriguez@atheros.com>,
	"linville@tuxdriver.com" <linville@tuxdriver.com>,
	"johannes@sipsolutions.net" <johannes@sipsolutions.net>,
	"ath5k-devel@lists.ath5k.org" <ath5k-devel@lists.ath5k.org>,
	"linux-wireless@vger.kernel.org" <linux-wireless@vger.kernel.org>
Subject: Re: [ath5k-devel] [PATCH 1/5] nl80211: Add new WIPHY attribute COVERAGE_CLASS
Date: Tue, 15 Dec 2009 23:52:42 +0100	[thread overview]
Message-ID: <200912152352.45977.8an@praha12.net> (raw)
In-Reply-To: <20091215215855.GC2067@tux>

[-- Attachment #1: Type: text/plain, Size: 2410 bytes --]

On 15.12.2009 22:58 you wrote:
> I don't see how this makes sense. Are you saying if we have multiple
> interfaces you must restrict the slot_time, ack_timeout and CTS timeout all
> to the same values for all of them?
If you have multiple interfaces, they must be all on the same channel and you 
can't do anything with it. It's the with slot time: it's a parameter of the 
CSMA/CA method for medium sharing. When stations use a different value of 
slot time, collisions will happen more likely. It doesn't matter they are 
connected to different BSS, there's just one shared medium.

There is no such problem with ACK timeout, but there is still another one: you 
can have packets coming from multiple interfaces in hardware queue. And you 
can't change the register value before sending each one, because the hardware 
sends them on their own terms and just reports it's done via an interrupt. So 
effectively the ACK timeout has to be the same as well.

> I'd say expose it through debugfs first then instead of adding proper APIs
> for userspace. If the country IE is the way to pass this information alog
> to STAs there would be no need to tweak this on the user end. If you're an
> AP though you are likely going to want to change this though so I see, for
> example hostapd wanting to set this through nl80211. It also seems
> reasonable for IBSS but IBSS won't send country IEs unless I guess we use
> wpa_supplicant and somehow leverage the IE generation from hostapd.
The problem is, I don't know any hardware that really sends the coverage 
class. There's a lot of hardware that allows setting only the ACK timeout, 
like all the Ubiquity devices, Ovislink 5000 and so on. So the client 
connecting to these devices will need to specify the coverage class manually.

When the coverage class is sent in beacons, it should take precedence over the 
user-specified value, I'm not against that. It's just not relevant now, until 
there's a hardware that sends it or until the hostap part is done. And I said 
I want to do that, just not in this series, it's already problematic 
enough...

And about the debugfs: originaly I thought i would just implement a simple 
debugfs API for setting ACK/CTS timeout and slot time. However, Johannes Berg 
recommended me to do this coverage class thing and I didn't envision how much 
trouble it's gonna be...

Lukas Turek

[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

  parent reply	other threads:[~2009-12-15 22:52 UTC|newest]

Thread overview: 34+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-12-15 17:56 [PATCH 0/5] Setting coverage class (and ACK timeout and slot time), take two Lukáš Turek
2009-12-15 17:56 ` [PATCH 1/5] nl80211: Add new WIPHY attribute COVERAGE_CLASS Lukáš Turek
2009-12-15 19:00   ` [ath5k-devel] " Luis R. Rodriguez
2009-12-15 19:02     ` Luis R. Rodriguez
2009-12-15 21:07       ` Lukáš Turek
2009-12-15 21:44         ` Luis R. Rodriguez
2009-12-16  8:03       ` Holger Schurig
2009-12-15 20:56     ` Lukáš Turek
2009-12-15 21:58       ` Luis R. Rodriguez
2009-12-15 22:48         ` Felix Fietkau
2009-12-15 22:52         ` Lukáš Turek [this message]
2009-12-16  8:30           ` Luis R. Rodriguez
2009-12-18 16:33             ` Lukáš Turek
2009-12-18 17:20               ` Luis R. Rodriguez
2009-12-15 17:56 ` [PATCH 2/5] mac80211: Add new callback set_coverage_class Lukáš Turek
2009-12-15 18:07   ` Johannes Berg
2009-12-15 18:11   ` [ath5k-devel] " Luis R. Rodriguez
2009-12-15 21:23     ` Lukáš Turek
2009-12-15 21:25     ` Johannes Berg
2009-12-15 17:56 ` [PATCH 3/5] ath5k: Fix functions for getting/setting slot time Lukáš Turek
2009-12-15 17:56 ` [PATCH 4/5] ath5k: Reimplement clock rate to usec conversion Lukáš Turek
2009-12-21 10:26   ` [ath5k-devel] " 海藻敬之
2009-12-21 12:38     ` Lukáš Turek
     [not found]       ` <4B301FE9.2020702@thinktube.com>
2009-12-22 16:08         ` Lukáš Turek
     [not found]     ` <4B2F50DD.60701@thinktube.com>
2009-12-21 12:40       ` Lukáš Turek
2009-12-21 15:08         ` Bob Copeland
2009-12-21 15:28           ` Lukáš Turek
2009-12-22  3:28             ` Bob Copeland
2009-12-15 17:56 ` [PATCH 5/5] ath5k: Implement mac80211 callback set_coverage_class Lukáš Turek
2009-12-15 18:50   ` [ath5k-devel] " Luis R. Rodriguez
2009-12-15 19:01     ` Luis R. Rodriguez
2009-12-15 21:35     ` Lukáš Turek
2009-12-15 22:07       ` Luis R. Rodriguez
2009-12-15 17:56 ` [PATCH] iw: Add support for NL80211_ATTR_WIPHY_COVERAGE_CLASS Lukáš Turek

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=200912152352.45977.8an@praha12.net \
    --to=8an@praha12.net \
    --cc=Luis.Rodriguez@atheros.com \
    --cc=ath5k-devel@lists.ath5k.org \
    --cc=johannes@sipsolutions.net \
    --cc=linux-wireless@vger.kernel.org \
    --cc=linville@tuxdriver.com \
    --cc=lrodriguez@atheros.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).