From: Luca Coelho <luca@coelho.fi>
To: Michal Kazior <michal.kazior@tieto.com>
Cc: Johannes Berg <johannes@sipsolutions.net>,
linux-wireless <linux-wireless@vger.kernel.org>,
"Otcheretianski, Andrei" <andrei.otcheretianski@intel.com>
Subject: Re: [PATCH 5/7] mac80211: improve CSA locking
Date: Fri, 24 Jan 2014 09:44:17 +0200 [thread overview]
Message-ID: <1390549457.19968.8.camel@porter.coelho.fi> (raw)
In-Reply-To: <CA+BoTQmGrx0Ko-E9N6ntzurGgZHK+nTyY78U43xXw8yctGbf4A@mail.gmail.com>
On Thu, 2014-01-23 at 13:41 +0100, Michal Kazior wrote:
> On 23 January 2014 13:09, Johannes Berg <johannes@sipsolutions.net> wrote:
> > On Thu, 2014-01-23 at 08:57 +0100, Michal Kazior wrote:
> >
> >> This is actually the tricky part. How do you automatically switch an
> >> AP interface? You should be able to at least generate CSA IE - true -
> >> but what about beacon after CSA? You need to update HT IE and VHT IE
> >> at least.
> >>
> >> Perhaps we could split CSA into two parts somehow and make it more
> >> userspace coordinated. The first part of CSA - announcing it, is
> >> trivial and can actually be done entirely within mac80211 if I'm not
> >> mistaken. What needs userspace input is what happens after the
> >> announcement. This would mean that there is a possible time gap
> >> between announcement being completed and the actual channel switch &
> >> operation resuming.
> >
> > I believe that even announcing it isn't necessarily possible without
> > userspace involvement, particularly for things like 11ac's channel
> > switch wrapper IE and/or operating class changes.
>
> Aww, too bad then.
>
> But splitting would still be useful (at least in some sense) if you
> consider CSA to "usable" DFS channels.
I don't like the split idea that much. If we send the CS announcement
automatically but the userspace never reacts to our indication, we will
be stuck in the middle of the channel switch...
--
Luca.
next prev parent reply other threads:[~2014-01-24 7:44 UTC|newest]
Thread overview: 50+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-01-20 14:21 [PATCH 0/7] mac80211: CSA related fixes Michal Kazior
2014-01-20 14:21 ` [PATCH 1/7] mac80211: fix possible memory leak on AP CSA failure Michal Kazior
2014-01-21 14:55 ` Johannes Berg
2014-01-22 6:54 ` Michal Kazior
2014-01-20 14:21 ` [PATCH 2/7] mac80211: treat IBSS CSA finish failure seriously Michal Kazior
2014-01-21 15:00 ` Johannes Berg
2014-01-22 11:38 ` Luca Coelho
2014-01-20 14:21 ` [PATCH 3/7] mac80211: move csa_active setting in STA CSA Michal Kazior
2014-01-22 11:40 ` Luca Coelho
2014-01-20 14:21 ` [PATCH 4/7] mac80211: fix sdata->radar_required locking Michal Kazior
2014-01-20 14:21 ` [PATCH 5/7] mac80211: improve CSA locking Michal Kazior
2014-01-20 15:41 ` Michal Kazior
2014-01-21 15:06 ` Johannes Berg
2014-01-22 6:51 ` Michal Kazior
2014-01-22 8:52 ` Johannes Berg
2014-01-22 9:07 ` Michal Kazior
2014-01-22 9:13 ` Johannes Berg
2014-01-22 10:13 ` Michal Kazior
2014-01-22 10:19 ` Johannes Berg
2014-01-22 12:36 ` Luca Coelho
2014-01-22 15:10 ` Johannes Berg
2014-01-22 15:13 ` Luca Coelho
2014-01-23 6:22 ` Michal Kazior
2014-01-23 6:31 ` Luca Coelho
2014-01-23 6:41 ` Michal Kazior
2014-01-23 7:31 ` Luca Coelho
2014-01-23 7:50 ` Otcheretianski, Andrei
2014-01-23 7:57 ` Michal Kazior
2014-01-23 9:50 ` Luca Coelho
2014-01-23 10:33 ` Michal Kazior
2014-01-23 12:20 ` Johannes Berg
2014-01-23 12:38 ` Michal Kazior
2014-01-24 7:41 ` Luca Coelho
2014-01-24 8:40 ` Johannes Berg
2014-01-24 9:52 ` Luca Coelho
2014-01-24 10:40 ` Michal Kazior
2014-01-24 12:55 ` Luca Coelho
2014-01-23 12:09 ` Johannes Berg
2014-01-23 12:41 ` Michal Kazior
2014-01-24 7:44 ` Luca Coelho [this message]
2014-01-23 6:35 ` Luca Coelho
2014-01-23 12:07 ` Johannes Berg
2014-01-23 6:35 ` Michal Kazior
2014-01-23 12:06 ` Johannes Berg
2014-01-20 14:21 ` [PATCH 6/7] mac80211: deny attempts at using chanctx during CSA Michal Kazior
2014-01-20 21:41 ` Luca Coelho
2014-01-21 6:07 ` Michal Kazior
2014-01-21 15:14 ` Johannes Berg
2014-01-20 14:21 ` [PATCH 7/7] Revert "cfg80211: disable CSA for all drivers" Michal Kazior
2014-01-21 15:16 ` [PATCH 0/7] mac80211: CSA related fixes 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=1390549457.19968.8.camel@porter.coelho.fi \
--to=luca@coelho.fi \
--cc=andrei.otcheretianski@intel.com \
--cc=johannes@sipsolutions.net \
--cc=linux-wireless@vger.kernel.org \
--cc=michal.kazior@tieto.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).