public inbox for linux-wireless@vger.kernel.org
 help / color / mirror / Atom feed
From: Simon Wunderlich <simon.wunderlich@s2003.tu-chemnitz.de>
To: Johannes Berg <johannes@sipsolutions.net>
Cc: Christian Lamparter <chunkeey@googlemail.com>,
	Adrian Chadd <adrian@freebsd.org>,
	Simon Wunderlich <simon.wunderlich@s2003.tu-chemnitz.de>,
	linux-wireless@vger.kernel.org, linville@tuxdriver.com,
	mathias.kretschmer@fokus.fraunhofer.de,
	Simon Wunderlich <siwu@hrz.tu-chemnitz.de>
Subject: Re: [PATCH] nl80211: allow ad-hoc to set WMM parameters from outside
Date: Tue, 8 Jan 2013 15:13:28 +0100	[thread overview]
Message-ID: <20130108141328.GA18920@pandem0nium> (raw)
In-Reply-To: <1357137688.9839.43.camel@jlt4.sipsolutions.net>

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

On Wed, Jan 02, 2013 at 03:41:28PM +0100, Johannes Berg wrote:
> On Wed, 2013-01-02 at 14:36 +0100, Christian Lamparter wrote:
> 
> > I can't find anything useful in the 802.11-2012 spec either. There's
> > something vague in 4.7 "Difference between ESS and IBSS LANs" on page 80:
> > "The services that apply to an IBSS are the SSs [Station ServiceS]. [...].
> > The parameters that  control differentiation of the delivery of MSDUs with
> > different priority using EDCA are >FIXED<. [...]".
> > 
> > But what does this mean? Does it translate into: "Every peer of an IBSS
> > truly independent of each other when it comes to the WMM parameters.
> > Each peer can advertise (in beacons, probes, ...) and have its own set
> > of fixed parameters and no one needs to sync those from anyone else"?
> > 
> > Or does "fixed" just mean that everyone has to go with the default WMM
> > parameters from 8-105 [i.e.: no one can change them at all]?
> 
> Yeah it's a little vague, but I think it means *fixed* as you describe.
> The WMM spec describes them as fixed and also doesn't include the WMM
> Parameter IE in beacons/probe responses, so all of this would be an
> extension to the spec and we have to be careful not to break the spec
> (default) case (which is correct right now, afaict, with default
> parameters and no parameters IE)

It appears that WMM IEs never carry EDCA parameters, and this is also not
specified anywhere that IBSS-stations should do that. All I can find is to use
the default EDCA parameters, and this is done right now in Linux - external WMM
IEs are ignored.

For the original patch, I'd still opt to have it integrated:
 * if no WMM EDCA parameters are provided from userspace, the default behaviour
   is still to use the default EDCA parameters - 99% of the people won't change
   that
 * changed parameter sets don't have to be exposed to other nodes - the BSS
   won't break if nodes will have other parameters, it might just behave
   unexpected in terms of priority. Normal users won't tamper with EDCA parameters
   and shouldn't notice that at all.
 * Exposing EDCA parameters in the WMM IEs is technically possible, but will
   probably give us more headache (syncing them, most clients won't support that,
   etc) then solve problems.

With the current patch, one can just set the parameters locally, using the
already existing nl80211-commands for AP. One thing we still need to consider
is the ibss-merge case (mac80211 will reset to EDCA defaults then, we want
to keep the custom values). I can provide a patch for that, of course.

> 
> Interoperating with FreeBSD would be useful though.

Yeah I agree. Don't know what the state in FreeBSD is now, but I'd still suggest
to not use EDCA-parameters in WMM IEs for reasons above. Even if Linux and FreeBSD
supports it in their open source drivers, there will probably tons of other drivers
not supporting these parameters and breaking the mechanism - this is not standard
behaviour, after all.

Adrian, others, what do you think?

Cheers,
	Simon

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 198 bytes --]

  reply	other threads:[~2013-01-08 14:13 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-11-27 18:50 [PATCH] Allow to set WMM parameters in IBSS mode Simon Wunderlich
2012-11-27 18:50 ` [PATCH] nl80211: allow ad-hoc to set WMM parameters from outside Simon Wunderlich
2012-11-28 14:01   ` Johannes Berg
2012-11-30 13:43     ` Simon Wunderlich
2012-12-28 15:05       ` Johannes Berg
2012-12-30 23:33         ` Simon Wunderlich
2013-01-01 23:46           ` Adrian Chadd
2013-01-02 12:58             ` Johannes Berg
2013-01-02 13:36               ` Christian Lamparter
2013-01-02 14:41                 ` Johannes Berg
2013-01-08 14:13                   ` Simon Wunderlich [this message]
2013-01-08 17:28                     ` Adrian Chadd
2013-01-09 13:15                       ` Simon Wunderlich
2013-01-09 14:10                         ` Christian Lamparter
2013-01-09 14:33                           ` Simon Wunderlich
2013-01-09 20:01                             ` Christian Lamparter
2012-11-27 18:50 ` [PATCH] iw: allow to set wmm parameters from iw Simon Wunderlich
2012-11-27 21:14   ` [PATCHv2] " Simon Wunderlich

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=20130108141328.GA18920@pandem0nium \
    --to=simon.wunderlich@s2003.tu-chemnitz.de \
    --cc=adrian@freebsd.org \
    --cc=chunkeey@googlemail.com \
    --cc=johannes@sipsolutions.net \
    --cc=linux-wireless@vger.kernel.org \
    --cc=linville@tuxdriver.com \
    --cc=mathias.kretschmer@fokus.fraunhofer.de \
    --cc=siwu@hrz.tu-chemnitz.de \
    /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