Linux wireless drivers development
 help / color / mirror / Atom feed
From: "Tomas Winkler" <tomasw@gmail.com>
To: "Johannes Berg" <johannes@sipsolutions.net>
Cc: "Jouni Malinen" <j@w1.fi>,
	"Ron Rindjunsky" <rindjon@googlemail.com>,
	linux-wireless <linux-wireless@vger.kernel.org>
Subject: Re: HT action frame code
Date: Tue, 9 Sep 2008 13:53:17 +0300	[thread overview]
Message-ID: <1ba2fa240809090353o1ab1340ck82a9208dd364587a@mail.gmail.com> (raw)
In-Reply-To: <1220956032.31304.135.camel@johannes.berg>

On Tue, Sep 9, 2008 at 1:27 PM, Johannes Berg <johannes@sipsolutions.net> wrote:
> On Tue, 2008-09-09 at 12:25 +0200, Johannes Berg wrote:
>> On Tue, 2008-09-09 at 13:21 +0300, Tomas Winkler wrote:
>>
>> > > Right. And if mac80211 is the AP, then I don't see how the STA can start
>> > > aggregation since I couldn't find where the relevant action frames are
>> > > handled. But you said it's not part of the code now, so I couldn't find
>> > > it anyway.
>> >
>> > That's probably the current situation.
>>
>> So back to square one, where should it be handled? Does it really make
>> sense to let hostapd have influence over the decision, or should we just
>> move the code that we have a bit and handle it in the kernel? After all,
>> we do have all the code necessary to handle it, but it's currently
>> restricted to STA mode, and I don't see a fundamental reason for that.

In basic AP mode everything can be handled by hostapd. There are no
performance sensitive management task, therefore the original mac80211
flow  didn't rout management frames withing mac80211 mlme.  But I see
benefit of keeping for example BA handshake, BAR, and MS/MIMO PS (not
implemented yet)  inside mac. BA session requires knowledge of
sequence counters, BAR handles reordering buffer. MS-PS requires
knowledge of number or rx chains. It will expose too much guts to the
users space.

> In fact, what if we're in STA mode with userspace MLME? Do we want to
> handle all that in userspace then? This doesn't seem sensible to me.

Also in this case I would leave this particular features + 11h
(channel switch, TPC) inside mac80211.
Tomas

  reply	other threads:[~2008-09-09 10:53 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-09-08 14:22 HT action frame code Johannes Berg
2008-09-08 20:09 ` Jouni Malinen
2008-09-08 20:48   ` Tomas Winkler
2008-09-09  6:30     ` Johannes Berg
2008-09-09  6:33   ` Johannes Berg
2008-09-09  8:14     ` Tomas Winkler
2008-09-09  8:20       ` Johannes Berg
2008-09-09  8:46         ` Tomas Winkler
2008-09-09  8:52           ` Johannes Berg
2008-09-09  9:41             ` Tomas Winkler
2008-09-09  9:42               ` Johannes Berg
2008-09-09  9:52                 ` Tomas Winkler
2008-09-09  9:55                   ` Johannes Berg
2008-09-09 10:21                     ` Tomas Winkler
2008-09-09 10:25                       ` Johannes Berg
2008-09-09 10:27                         ` Johannes Berg
2008-09-09 10:53                           ` Tomas Winkler [this message]
2008-09-09 11:01                             ` Johannes Berg
2008-09-09 11:29                               ` [RFC] mac80211: make BA session handling independent of STA mode Johannes Berg
2008-09-09 12:28                                 ` 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=1ba2fa240809090353o1ab1340ck82a9208dd364587a@mail.gmail.com \
    --to=tomasw@gmail.com \
    --cc=j@w1.fi \
    --cc=johannes@sipsolutions.net \
    --cc=linux-wireless@vger.kernel.org \
    --cc=rindjon@googlemail.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