All of lore.kernel.org
 help / color / mirror / Atom feed
From: Johannes Berg <johannes@sipsolutions.net>
To: Victor Goldenshtein <victorg@ti.com>
Cc: Michal Kazior <michal.kazior@tieto.com>,
	"linux-wireless@vger.kernel.org" <linux-wireless@vger.kernel.org>,
	"kgiori@qca.qualcomm.com" <kgiori@qca.qualcomm.com>,
	"mcgrof@frijolero.org" <mcgrof@frijolero.org>,
	"zefir.kurtisi@neratec.com" <zefir.kurtisi@neratec.com>,
	"adrian.chadd@gmail.com" <adrian.chadd@gmail.com>,
	"j@w1.fi" <j@w1.fi>, "coelho@ti.com" <coelho@ti.com>,
	"igalc@ti.com" <igalc@ti.com>,
	"adrian@freebsd.org" <adrian@freebsd.org>,
	"nbd@nbd.name" <nbd@nbd.name>,
	"simon.wunderlich@s2003.tu-chemnitz.de"
	<simon.wunderlich@s2003.tu-chemnitz.de>
Subject: Re: [PATCH v4 5/6] nl80211/cfg80211: add ap channel switch command
Date: Mon, 05 Nov 2012 16:29:08 +0100	[thread overview]
Message-ID: <1352129348.9466.23.camel@jlt4.sipsolutions.net> (raw)
In-Reply-To: <50922AAF.3060605@ti.com>

On Thu, 2012-11-01 at 09:54 +0200, Victor Goldenshtein wrote:

> > This however introduces a new problem. Let's suppose we have 2 APs on
> > channel 1. The device doesn't support multi-channel. We won't be able to
> > switch channel on these APs at all.
> >
> > We might want to change the channel switch to resolve around the channel
> > itself (not the interface) - so we'd be saying "move all interfaces with
> > channel X to channel Y" instead of "move interface X to channel Y".
> >
> > Or we could let the driver decide what it'll do - e.g. silently switch
> > more than one interface to a different channel (which makes sense with
> > AP/DFS I guess) and just notify cfg/userspace about it. That would
> > require us to provide a way to switch interfaces (atomically possibly)
> > between channels while keeping in sync with interface combinations though.
> >
> 
> If the driver/device supports MR only on a SC - means it doesn't 

What's MR? SC = single channel?

> supports channel switch in MR, so basically the radar detection event 
> triggers AP channel switch which fails (with this new check) and the AP 
> shut down.

But I was speaking of two interfaces on a single channel.

> Of course there are possible driver specific workarounds (as you 
> mentioned above) but these are not part of this series.

yeah but you should probably allow for some strategy of handling this?

> Just for interest's sake, what is the use case having two APs on the 
> same channel?

Umm, lots of things? P2P groups, multiple BSSIDs, ...

johannes


  parent reply	other threads:[~2012-11-05 15:28 UTC|newest]

Thread overview: 51+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-10-14 14:48 [PATCH v4 0/6] nl/cfg/mac80211: add DFS master ability Victor Goldenshtein
2012-10-14 14:48 ` [PATCH v4 1/6] nl80211/cfg80211: add radar detection command/event Victor Goldenshtein
2012-10-16 19:01   ` Johannes Berg
2012-10-17 16:20     ` Zefir Kurtisi
2012-10-17 16:23       ` Johannes Berg
2012-10-21 16:40     ` Victor Goldenshtein
2012-10-22 12:55       ` Johannes Berg
2012-11-01  7:54         ` Victor Goldenshtein
2012-11-05 15:21           ` Johannes Berg
2012-11-13 15:04             ` Victor Goldenshtein
2012-11-13 18:05               ` Simon Wunderlich
2012-11-20 15:14                 ` Victor Goldenshtein
2012-11-26 10:52                   ` Johannes Berg
2012-11-14 11:19               ` Johannes Berg
2012-11-20 15:15                 ` Victor Goldenshtein
2012-11-14 12:32             ` Michal Kazior
2012-11-14 12:38               ` Johannes Berg
2012-11-20 15:15                 ` Victor Goldenshtein
2012-11-26 10:51                   ` Johannes Berg
2012-10-14 14:48 ` [PATCH v4 2/6] mac80211: " Victor Goldenshtein
2012-10-14 14:48 ` [PATCH v4 3/6] nl80211/cfg80211: add ability to enable TX on op-channel Victor Goldenshtein
2012-10-14 14:48 ` [PATCH v4 4/6] mac80211: " Victor Goldenshtein
2012-10-16 19:03   ` Johannes Berg
2012-10-21 16:40     ` Victor Goldenshtein
2012-10-14 14:48 ` [PATCH v4 5/6] nl80211/cfg80211: add ap channel switch command Victor Goldenshtein
2012-10-16 19:06   ` Johannes Berg
2012-10-21 16:40     ` Victor Goldenshtein
2012-10-22 12:52       ` Johannes Berg
2012-11-01  7:54         ` Victor Goldenshtein
2012-11-05 15:27           ` Johannes Berg
2012-11-13 15:04             ` Victor Goldenshtein
2012-10-23  6:20   ` Michal Kazior
2012-11-01  7:54     ` Victor Goldenshtein
2012-11-01 17:15       ` Adrian Chadd
2012-11-01 19:32         ` Johannes Berg
2012-11-05 15:29       ` Johannes Berg [this message]
2012-11-13 15:04         ` Victor Goldenshtein
2012-10-14 14:48 ` [PATCH v4 6/6] mac80211: add ap channel switch command/event Victor Goldenshtein
2012-10-16 19:07   ` Johannes Berg
2012-10-21 16:40     ` Victor Goldenshtein
2012-10-22 12:56       ` Johannes Berg
2012-11-01  7:54         ` Victor Goldenshtein
2012-11-05 15:23           ` Johannes Berg
2012-11-13 15:04             ` Victor Goldenshtein
2012-11-14 11:23               ` Johannes Berg
2012-11-14 12:19                 ` Michal Kazior
2012-11-14 12:38                   ` Johannes Berg
2012-11-20 15:15                     ` Victor Goldenshtein
2012-11-26 10:50                       ` Johannes Berg
2012-11-20 15:15                 ` Victor Goldenshtein
2012-11-26 10:50                   ` 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=1352129348.9466.23.camel@jlt4.sipsolutions.net \
    --to=johannes@sipsolutions.net \
    --cc=adrian.chadd@gmail.com \
    --cc=adrian@freebsd.org \
    --cc=coelho@ti.com \
    --cc=igalc@ti.com \
    --cc=j@w1.fi \
    --cc=kgiori@qca.qualcomm.com \
    --cc=linux-wireless@vger.kernel.org \
    --cc=mcgrof@frijolero.org \
    --cc=michal.kazior@tieto.com \
    --cc=nbd@nbd.name \
    --cc=simon.wunderlich@s2003.tu-chemnitz.de \
    --cc=victorg@ti.com \
    --cc=zefir.kurtisi@neratec.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 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.