From: Vasanthakumar Thiagarajan <vasanth@atheros.com>
To: "Luis R. Rodriguez" <lrodriguez@atheros.com>
Cc: Vasanth Thiagarajan <Vasanth.Thiagarajan@Atheros.com>,
Jouni Malinen <j@w1.fi>,
"linville@tuxdriver.com" <linville@tuxdriver.com>,
"stable@kernel.org" <stable@kernel.org>,
"linux-wireless@vger.kernel.org" <linux-wireless@vger.kernel.org>,
Paul Stewart <pstew@google.com>,
Amod Bodas <Amod.Bodas@Atheros.com>,
Johannes Berg <johannes@sipsolutions.net>
Subject: Re: [PATCH] mac80211: fix rate_control_send_low warnings for delbas
Date: Wed, 29 Sep 2010 14:26:57 +0530 [thread overview]
Message-ID: <20100929085657.GD18450@vasanth-laptop> (raw)
In-Reply-To: <AANLkTinKkTgUUGPxY82G1B6taToKeAMOmrJ+tYvzLY52@mail.gmail.com>
On Wed, Sep 29, 2010 at 12:54:27PM +0530, Luis R. Rodriguez wrote:
> > (where we just authenticated with another AP)
>
> Not sure I get this part.
I mean we are trying to tx frames on the channel where we got
newly authenticated to another AP.
>
> > just before starting association with the new AP.
>
> Right, but when we are already switched to the new channel, and the
> reason is that when we try to send a frame we never kept track of the
> peer's original channel. Now, I also see the teardown happen right
> before mac80211 tells us its associated to the new AP as well but it
> not sure what cfg80211 thinks at that point. At least the cfg80211
> patch's intention seemed to be to allow association to both, and
> tearing down the association to the other once you completed
> association to the other. I do see, what you describe though too --
> only authentication to the new AP, and right before association, we
> send the teardown. The problem is though that when we send the
> teardown we are already set on the new channel.
Ok. From my understanding, the sequence of operations during roaming
are
1. NL80211_CMD_AUTHENTICATE to an (better) AP from user space
- Change to New Ap's channel and send auth req (done by
ieee80211_work_work()).
- Notify auth resp to userspace (ieee80211_probe_auth_done())
2. NL80211_CMD_ASSOCIATE to newly authenticated AP from user space
- Clean up any existing BA session with an AP, if any.
Here the assumption is, ieee80211_work_work() has already
configured the device back to it's operating channel (the
old one) after the authentication is complete with the new
AP. This should have happened part of 1.
- Start the association process with the new AP.
Vasanth
next prev parent reply other threads:[~2010-09-29 8:57 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-09-28 22:29 [PATCH] mac80211: fix rate_control_send_low warnings for delbas Luis R. Rodriguez
2010-09-28 23:02 ` Jouni Malinen
2010-09-28 23:58 ` Luis R. Rodriguez
2010-09-29 5:20 ` Vasanthakumar Thiagarajan
2010-09-29 7:24 ` Luis R. Rodriguez
2010-09-29 8:56 ` Vasanthakumar Thiagarajan [this message]
2010-09-29 17:16 ` Luis R. Rodriguez
2010-09-29 17:24 ` Johannes Berg
2010-09-29 8:18 ` Johannes Berg
2010-09-29 17:10 ` Luis R. Rodriguez
2010-09-29 17:25 ` Johannes Berg
2010-09-30 17:53 ` Luis R. Rodriguez
2010-10-01 20:07 ` Luis R. Rodriguez
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=20100929085657.GD18450@vasanth-laptop \
--to=vasanth@atheros.com \
--cc=Amod.Bodas@Atheros.com \
--cc=Vasanth.Thiagarajan@Atheros.com \
--cc=j@w1.fi \
--cc=johannes@sipsolutions.net \
--cc=linux-wireless@vger.kernel.org \
--cc=linville@tuxdriver.com \
--cc=lrodriguez@atheros.com \
--cc=pstew@google.com \
--cc=stable@kernel.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.