linux-wireless.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Johannes Berg <johannes@sipsolutions.net>
To: James Prestwood <prestwoj@gmail.com>,
	Arend van Spriel <aspriel@gmail.com>,
	linux-wireless@vger.kernel.org
Subject: Re: Adding CMD_SET_CHANNEL for station iftypes
Date: Thu, 03 Feb 2022 23:06:50 +0100	[thread overview]
Message-ID: <a5c2cbdc9f0c1587f0d27850ca3bbac2d5103e9c.camel@sipsolutions.net> (raw)
In-Reply-To: <8557b62e0f1c6a94f7ec9b27a596d7499fd072d7.camel@gmail.com>

On Mon, 2022-01-10 at 09:13 -0800, James Prestwood wrote:
> > 
> > Even if it only supports STA mode it can. The constraint being that
> > it is 
> > not associated (or busy trying to associate) to an AP. When it is 
> > associated it has to sit on the channel of the AP, as announced in
> > it's 
> > beacon and/or probe response, at regular intervals. You referred to
> > DPP to 
> > provision the STA so I assume it is not associated, right? Could you
> > write 
> > out the whole scenario as you think it should/could be done?
> 
> Correct, I'm only talking about doing this while not associated.
> 
> As for my intentended scenario I simply want to call CMD_SET_CHANNEL
> then run the entire DPP auth/config protocol while on this channel.
> Then, once finished, call (a new API) CMD_SET_CHANNEL_CANCEL.
> 

So I think Arend and I are approaching this from a different angle :-)

I was mostly thinking about the iwlwifi firmware implementation, to be
honest, which simply doesn't have such a thing today. The closest would
probably be putting it into monitor mode, which of course means you
don't get proper ACK behaviour etc.

I think Arend was probably more thinking of semantics rather than
implementation, surely it'd be _possible_ to do it, but it's not
something that you can _practically_ do today on all hardware.

So practically, if we add such API

 1) it might fix the issue on some devices that can implement it

 2) it would leave us having to emulate it using remain-on-channel or
    similar when the firmware doesn't support

 3) you'd still have to implement it for older kernels

(or we leave out 2, and you have to have the implementation for 3 for
newer kernels too)

I guess I wouldn't particularly mind adding APIs for it, but I'm not
sure I'd want to have to implement and maintain (2).

johannes

      reply	other threads:[~2022-02-03 22:06 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-11-19  0:11 Adding CMD_SET_CHANNEL for station iftypes James Prestwood
2021-11-19  8:31 ` Johannes Berg
2021-11-19 17:43   ` James Prestwood
2022-01-06 22:01     ` Johannes Berg
2022-01-07 19:35       ` James Prestwood
2022-01-09 12:00         ` Arend van Spriel
2022-01-10 17:13           ` James Prestwood
2022-02-03 22:06             ` Johannes Berg [this message]

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=a5c2cbdc9f0c1587f0d27850ca3bbac2d5103e9c.camel@sipsolutions.net \
    --to=johannes@sipsolutions.net \
    --cc=aspriel@gmail.com \
    --cc=linux-wireless@vger.kernel.org \
    --cc=prestwoj@gmail.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).