public inbox for virtio-dev@lists.linux.dev
 help / color / mirror / Atom feed
From: "Michael S. Tsirkin" <mst@redhat.com>
To: Parav Pandit <parav@nvidia.com>
Cc: Xuan Zhuo <xuanzhuo@linux.alibaba.com>,
	"virtio-dev@lists.oasis-open.org"
	<virtio-dev@lists.oasis-open.org>,
	"jasowang@redhat.com" <jasowang@redhat.com>,
	"virtio-comment@lists.oasis-open.org"
	<virtio-comment@lists.oasis-open.org>
Subject: [virtio-dev] Re: RE: RE: RE: [virtio-comment] RE: RE: RE: RE: [RFC] virtio-net: support access and control the member devices
Date: Fri, 4 Aug 2023 10:07:13 -0400	[thread overview]
Message-ID: <20230804095255-mutt-send-email-mst@kernel.org> (raw)
In-Reply-To: <PH0PR12MB548152A5677F0508A62587AFDC09A@PH0PR12MB5481.namprd12.prod.outlook.com>

On Fri, Aug 04, 2023 at 12:49:04PM +0000, Parav Pandit wrote:
> 
> 
> > From: Michael S. Tsirkin <mst@redhat.com>
> > Sent: Friday, August 4, 2023 4:03 PM
> 
> > > >
> > > > At this point to have port for owner device requires creating a dedicated
> > switching object, to be located sometimes side by side inside the owner,
> > sometimes outside.
> > > > All of these cases to be crafted, please rethink if this is _really_ needed as
> > virtio object or not.
> > >
> 
> > >
> > > YES.
> > >
> > > We can hear others.
> > >
> > > @Jason @Michael
> > >
> > >
> > > Thanks
> > 
> > 
> > This is so abstract, hard to have any position as I'm not sure what we are
> > discussing. If some virtio devices have an integrated switch then ability to
> > control the switch through virtio seems useful.
> >
> True, for us, at this point we do not have plan to expose virtio switch device because users are not blocked on it.

If someone does though I see no reason not to - we have a variety
of devices, switch would be just another one. The motivation would
be the usual one for virtio - so there's one generic driver.

> > Re:queues - it's not by chance that we have multiple admin queues.
> > So driver can dedicate one queue to filtering commands if that's felt to be
> > important.
> > 
> Admin queue currently do not send non admin command of the device.
> Would you propose admin queue for something else also for rtc or console or cryto device and indicate its role so device can understand what is coming to it.

Well, I guess this switch has group members as its "ports" so
maybe that makes it like an admin command. Again it's all up
in the air too much to be able to judge.

-- 
MST


---------------------------------------------------------------------
To unsubscribe, e-mail: virtio-dev-unsubscribe@lists.oasis-open.org
For additional commands, e-mail: virtio-dev-help@lists.oasis-open.org


  reply	other threads:[~2023-08-04 14:09 UTC|newest]

Thread overview: 55+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-08-03  8:31 [virtio-dev] [RFC] virtio-net: support access and control the member devices Xuan Zhuo
2023-08-03  9:44 ` [virtio-dev] " Michael S. Tsirkin
2023-08-03  9:45   ` [virtio-dev] " Parav Pandit
2023-08-03 10:52   ` [virtio-dev] " Xuan Zhuo
2023-08-03 10:59     ` [virtio-dev] " Parav Pandit
2023-08-03 11:04       ` [virtio-dev] " Xuan Zhuo
2023-08-03 11:15         ` [virtio-dev] " Parav Pandit
2023-08-03 11:18           ` [virtio-dev] " Xuan Zhuo
2023-08-03 11:25             ` [virtio-dev] " Parav Pandit
2023-08-03 11:30               ` [virtio-dev] " Xuan Zhuo
2023-08-03 11:39                 ` [virtio-dev] " Parav Pandit
2023-08-03 11:41                   ` [virtio-dev] Re: [virtio-comment] " Xuan Zhuo
2023-08-03 13:15                     ` [virtio-dev] " Parav Pandit
2023-08-03 17:03                       ` Satananda Burla
2023-08-04  3:29                         ` [virtio-dev] " Jason Wang
2023-08-04  4:22                         ` [virtio-dev] " Parav Pandit
2023-08-04  6:18                       ` [virtio-dev] " Xuan Zhuo
2023-08-04  6:37                         ` [virtio-dev] " Parav Pandit
2023-08-04  6:54                           ` [virtio-dev] " Xuan Zhuo
2023-08-04  7:26                             ` [virtio-dev] " Parav Pandit
2023-08-04  7:44                               ` [virtio-dev] " Xuan Zhuo
2023-08-04 10:33                                 ` Michael S. Tsirkin
2023-08-04 12:49                                   ` [virtio-dev] " Parav Pandit
2023-08-04 14:07                                     ` Michael S. Tsirkin [this message]
2023-08-04 14:12                                       ` Parav Pandit
2023-08-06 10:32                                         ` [virtio-dev] " Michael S. Tsirkin
2023-08-07  6:02                                     ` [virtio-dev] Re: " Xuan Zhuo
2023-08-08  2:18                                       ` Jason Wang
2023-08-08  2:36                                         ` Xuan Zhuo
2023-08-08  3:09                                           ` Jason Wang
2023-08-08  3:13                                             ` Xuan Zhuo
2023-08-08  3:16                                             ` [virtio-dev] " Parav Pandit
2023-08-08  3:50                                               ` [virtio-dev] " Jason Wang
2023-08-08  4:16                                                 ` [virtio-dev] " Parav Pandit
2023-08-08  4:33                                                   ` [virtio-dev] " Jason Wang
2023-08-08  4:37                                                     ` [virtio-dev] " Parav Pandit
2023-08-08  5:21                                                       ` [virtio-dev] " Jason Wang
2023-08-08  6:16                                                         ` Xuan Zhuo
2023-08-08  6:44                                                   ` [virtio-dev] Re: " Xuan Zhuo
2023-08-08  6:56                                                     ` [virtio-dev] " Parav Pandit
2023-08-08  4:00                                               ` [virtio-dev] " Xuan Zhuo
2023-08-08  4:11                                                 ` [virtio-dev] " Parav Pandit
2023-08-08  4:16                                                   ` [virtio-dev] " Xuan Zhuo
2023-08-08  4:34                                                     ` [virtio-dev] " Parav Pandit
2023-08-08  6:02                                                       ` [virtio-dev] " Xuan Zhuo
2023-08-08  4:18                                                   ` [virtio-dev] " Satananda Burla
2023-08-04  3:31   ` [virtio-dev] " Jason Wang
2023-08-04  4:05     ` [virtio-dev] " Parav Pandit
2023-08-08  2:30       ` [virtio-dev] " Jason Wang
2023-08-03 10:23 ` [virtio-dev] " Parav Pandit
2023-08-03 11:05   ` [virtio-dev] " Xuan Zhuo
2023-08-03 11:13     ` [virtio-dev] " Parav Pandit
2023-08-03 11:15       ` [virtio-dev] " Xuan Zhuo
2023-08-03 11:21         ` [virtio-dev] " Parav Pandit
2023-08-03 11:27           ` [virtio-dev] " Xuan Zhuo

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=20230804095255-mutt-send-email-mst@kernel.org \
    --to=mst@redhat.com \
    --cc=jasowang@redhat.com \
    --cc=parav@nvidia.com \
    --cc=virtio-comment@lists.oasis-open.org \
    --cc=virtio-dev@lists.oasis-open.org \
    --cc=xuanzhuo@linux.alibaba.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