public inbox for virtio-dev@lists.linux.dev
 help / color / mirror / Atom feed
From: "Michael S. Tsirkin" <mst@redhat.com>
To: Xuan Zhuo <xuanzhuo@linux.alibaba.com>
Cc: Parav Pandit <parav@nvidia.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 06:33:20 -0400	[thread overview]
Message-ID: <20230804053958-mutt-send-email-mst@kernel.org> (raw)
In-Reply-To: <1691135097.4908528-3-xuanzhuo@linux.alibaba.com>

On Fri, Aug 04, 2023 at 03:44:57PM +0800, Xuan Zhuo wrote:
> On Fri, 4 Aug 2023 07:26:01 +0000, Parav Pandit <parav@nvidia.com> wrote:
> >
> >
> > > From: Xuan Zhuo <xuanzhuo@linux.alibaba.com>
> > > Sent: Friday, August 4, 2023 12:24 PM
> >
> > > I know the gap. For us, the switch is under all devices(including the owner
> > > device).
> > > So we need to control the switch separately.
> > >
> > Right. Same for other devices too, including ours.
> > Hence, it is also located outside of the owner device (on a different system).
> >
> > If the switch is not a virtio device, does it really need to be exposed as virtio object?
> >
> > > In your structure, the owner device works as the switch. It has port to connect
> > > to the vf devices.
> > >
> > Right. I just put owner device as switch for simplicity of the discussion.
> >
> > 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.

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.


-- 
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 10:33 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 [this message]
2023-08-04 12:49                                   ` [virtio-dev] " Parav Pandit
2023-08-04 14:07                                     ` [virtio-dev] " Michael S. Tsirkin
2023-08-04 14:12                                       ` [virtio-dev] " 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=20230804053958-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