public inbox for virtio-dev@lists.linux.dev
 help / color / mirror / Atom feed
From: Xuan Zhuo <xuanzhuo@linux.alibaba.com>
To: Parav Pandit <parav@nvidia.com>
Cc: "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>,
	"Michael S. Tsirkin" <mst@redhat.com>
Subject: [virtio-dev] Re: RE: RE: RE: RE: [virtio-comment] RE: RE: RE: RE: [RFC] virtio-net: support access and control the member devices
Date: Mon, 7 Aug 2023 14:02:51 +0800	[thread overview]
Message-ID: <1691388171.3192031-1-xuanzhuo@linux.alibaba.com> (raw)
In-Reply-To: <PH0PR12MB548152A5677F0508A62587AFDC09A@PH0PR12MB5481.namprd12.prod.outlook.com>

On Fri, 4 Aug 2023 12:49:04 +0000, Parav Pandit <parav@nvidia.com> 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.


Also for us.

But we need to limit the ip of every member device. That is useful for cloud.
Because the user of each VM is untrustworthy. We must limit the ip traffic of
every member device.

We have two choose:

1. add feature to device by cvq of pf(or admin queue?), that can limit the ip(receive and transmit).
2. add feature to switch, it can limit the ip for every port. If we choose this
   way, I will try introduce the simple switch concept to the virtio-net.
   Because except this we have not more requirement for the switch. So we donot
   plan to introduce a complex switch.

Thanks.


>
> > 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.

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


  parent reply	other threads:[~2023-08-07  6:13 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                                     ` [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                                     ` Xuan Zhuo [this message]
2023-08-08  2:18                                       ` [virtio-dev] Re: " 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=1691388171.3192031-1-xuanzhuo@linux.alibaba.com \
    --to=xuanzhuo@linux.alibaba.com \
    --cc=jasowang@redhat.com \
    --cc=mst@redhat.com \
    --cc=parav@nvidia.com \
    --cc=virtio-comment@lists.oasis-open.org \
    --cc=virtio-dev@lists.oasis-open.org \
    /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