All of lore.kernel.org
 help / color / mirror / Atom feed
From: Guillaume Hetier <Guillaume.Hetier@microsoft.com>
To: Johannes Berg <johannes@sipsolutions.net>,
	"linux-wireless@vger.kernel.org" <linux-wireless@vger.kernel.org>
Cc: "kvalo@codeaurora.org" <kvalo@codeaurora.org>,
	"schuffelen@google.com" <schuffelen@google.com>,
	Shwetha Bhat <shwbhat@microsoft.com>,
	Andrew Beltrano <anbeltra@microsoft.com>,
	Brian Perkins <Brian.Perkins@microsoft.com>,
	KY Srinivasan <kys@microsoft.com>,
	Matteo Croce <mcroce@microsoft.com>
Subject: [RFC PATCH 1/1] Add control path virtualization in virt_wifi
Date: Tue, 28 Sep 2021 20:44:56 +0000	[thread overview]
Message-ID: <MWHPR21MB1547EB364EFA35799C3B69A38FA89@MWHPR21MB1547.namprd21.prod.outlook.com> (raw)

Hi,

> Are there any specific applications you're thinking of?

Our main targets are programs setting up IOT devices: we want to give the guest the capacity to connect to specific networks created by the IOT device for the duration of their setup.
For that, we want to be able to scan, connect and reflect network properties precisely enough to let the guest programs go forward with the setup.

We would like to make our solution as generic as possible to be extendable and potentially support for complex use cases if possible, but we are not aiming for generic network management from the guest.

<snip>

>> Based on the discussion, your recommendations concerning our initial questions seem to be:
>> - we should create a new driver, rather than modifying virt_wifi
>
> Yes, for sure, I don't think virt_wifi makes sense.
>
>> - netlink could be used as a protocol to communicate with the host
>
> That was just a thought really. I've done it before for hwsim's wmediumd virtio abstraction, and it's not all that bad, but you could just as well declare structs for communication and go that simple way. Might over time end up with more compatibility code on the device side though (assuming you want to keep old Linux kernels running) Ultimately, it doesn't really matter to us - I'd think of it as a device, and people come up with all kinds of strange hardware all the time ;-)

Thanks for the confirmation!
For netlink, we think it could be a nice middle ground between the lack of compatibility of raw structures and the overhead of implementing yet another TLV parser. We will experiment with it and use it if it goes well.

Guillaume

             reply	other threads:[~2021-09-28 20:45 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-09-28 20:44 Guillaume Hetier [this message]
2021-09-29  7:59 ` [RFC PATCH 1/1] Add control path virtualization in virt_wifi Johannes Berg
  -- strict thread matches above, loose matches on Subject: below --
2021-09-13 21:03 Guillaume Hetier
2021-09-21  8:27 ` 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=MWHPR21MB1547EB364EFA35799C3B69A38FA89@MWHPR21MB1547.namprd21.prod.outlook.com \
    --to=guillaume.hetier@microsoft.com \
    --cc=Brian.Perkins@microsoft.com \
    --cc=anbeltra@microsoft.com \
    --cc=johannes@sipsolutions.net \
    --cc=kvalo@codeaurora.org \
    --cc=kys@microsoft.com \
    --cc=linux-wireless@vger.kernel.org \
    --cc=mcroce@microsoft.com \
    --cc=schuffelen@google.com \
    --cc=shwbhat@microsoft.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.