All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Michael S. Tsirkin" <mst@redhat.com>
To: Jason Wang <jasowang@redhat.com>
Cc: Cindy Lu <lulu@redhat.com>, qemu-devel <qemu-devel@nongnu.org>,
	Gautam Dawar <gdawar@xilinx.com>, eperezma <eperezma@redhat.com>,
	Eli Cohen <elic@nvidia.com>,
	Zhu Lingshan <lingshan.zhu@intel.com>
Subject: Re: [PATCH V4 00/10] vhost-vDPA multiqueue
Date: Tue, 19 Oct 2021 06:44:34 -0400	[thread overview]
Message-ID: <20211019064428-mutt-send-email-mst@kernel.org> (raw)
In-Reply-To: <CACGkMEvEQfQwR1nN-G19XbWRanPYSydeyWGbt_4Z5XNdLajmtA@mail.gmail.com>

On Tue, Oct 19, 2021 at 03:24:36PM +0800, Jason Wang wrote:
> On Tue, Oct 19, 2021 at 3:21 PM Michael S. Tsirkin <mst@redhat.com> wrote:
> >
> > On Mon, Oct 11, 2021 at 12:28:19PM +0800, Jason Wang wrote:
> > > Hi All:
> > >
> > > This patch implements the multiqueue support for vhost-vDPA. The most
> > > important requirement si the control virtqueue support. The virtio-net
> > > and vhost-net core are tweak to support control virtqueue as if what
> > > data queue pairs are done: a dedicated vhost_net device which is
> > > coupled with the NetClientState is intrdouced so most of the existing
> > > vhost codes could be reused with minor changes. This means the control
> > > virtqueue will bypass the Qemu. With the control virtqueue, vhost-vDPA
> > > are extend to support creating and destroying multiqueue queue pairs
> > > plus the control virtqueue.
> > >
> > > For the future, if we want to support live migration, we can either do
> > > shadow cvq on top or introduce new interfaces for reporting device
> > > states.
> >
> >
> > I had to defer this due to rework I picked up.
> > I pushed an experimental tree - it still has issues but
> > if you rebase on top of that I can apply.
> 
> Will do. For the tree, is it
> git://git.kernel.org/pub/scm/virt/kvm/mst/qemu.git next?
> 
> Thanks
> 

pci

> >
> > > Tests are done via the vp_vdpa driver in L1 guest.
> > >
> > > Changes since V3:
> > >
> > > - fix build with vhost disabled
> > >
> > > Changes since V2:
> > >
> > > - rebase to qemu master
> > > - use "queue_pairs" instead of "qps"
> > > - typo fixes
> > >
> > > Changes since V1:
> > >
> > > - start and stop vhost devices when all queues were setup
> > > - fix the case when driver doesn't support MQ but device support
> > > - correctly set the batching capability to avoid a map/unmap storm
> > > - various other tweaks
> > >
> > > Jason Wang (10):
> > >   vhost-vdpa: open device fd in net_init_vhost_vdpa()
> > >   vhost-vdpa: classify one time request
> > >   vhost-vdpa: prepare for the multiqueue support
> > >   vhost-vdpa: let net_vhost_vdpa_init() returns NetClientState *
> > >   net: introduce control client
> > >   vhost-net: control virtqueue support
> > >   virtio-net: use "queue_pairs" instead of "queues" when possible
> > >   vhost: record the last virtqueue index for the virtio device
> > >   virtio-net: vhost control virtqueue support
> > >   vhost-vdpa: multiqueue support
> > >
> > >  hw/net/vhost_net-stub.c        |   4 +-
> > >  hw/net/vhost_net.c             |  55 ++++++++---
> > >  hw/net/virtio-net.c            | 165 ++++++++++++++++++---------------
> > >  hw/virtio/vhost-vdpa.c         |  56 +++++++++--
> > >  include/hw/virtio/vhost-vdpa.h |   1 +
> > >  include/hw/virtio/vhost.h      |   2 +
> > >  include/hw/virtio/virtio-net.h |   5 +-
> > >  include/net/net.h              |   5 +
> > >  include/net/vhost_net.h        |   6 +-
> > >  net/net.c                      |  24 ++++-
> > >  net/vhost-vdpa.c               | 127 ++++++++++++++++++++++---
> > >  11 files changed, 330 insertions(+), 120 deletions(-)
> > >
> > > --
> > > 2.25.1
> >



  reply	other threads:[~2021-10-19 10:47 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-10-11  4:28 [PATCH V4 00/10] vhost-vDPA multiqueue Jason Wang
2021-10-11  4:28 ` [PATCH V4 01/10] vhost-vdpa: open device fd in net_init_vhost_vdpa() Jason Wang
2021-10-11  4:28 ` [PATCH V4 02/10] vhost-vdpa: classify one time request Jason Wang
2021-10-11  4:28 ` [PATCH V4 03/10] vhost-vdpa: prepare for the multiqueue support Jason Wang
2021-10-18 15:44   ` Stefano Garzarella
2021-10-20  2:50     ` Jason Wang
2021-10-11  4:28 ` [PATCH V4 04/10] vhost-vdpa: let net_vhost_vdpa_init() returns NetClientState * Jason Wang
2021-10-11  4:28 ` [PATCH V4 05/10] net: introduce control client Jason Wang
2021-10-11  4:28 ` [PATCH V4 06/10] vhost-net: control virtqueue support Jason Wang
2021-10-11  4:28 ` [PATCH V4 07/10] virtio-net: use "queue_pairs" instead of "queues" when possible Jason Wang
2021-10-11  4:28 ` [PATCH V4 08/10] vhost: record the last virtqueue index for the virtio device Jason Wang
2021-10-11  4:28 ` [PATCH V4 09/10] virtio-net: vhost control virtqueue support Jason Wang
2021-10-11  4:28 ` [PATCH V4 10/10] vhost-vdpa: multiqueue support Jason Wang
2021-10-19  7:21 ` [PATCH V4 00/10] vhost-vDPA multiqueue Michael S. Tsirkin
2021-10-19  7:24   ` Jason Wang
2021-10-19 10:44     ` Michael S. Tsirkin [this message]
2021-10-19 11:17       ` Michael S. Tsirkin

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=20211019064428-mutt-send-email-mst@kernel.org \
    --to=mst@redhat.com \
    --cc=elic@nvidia.com \
    --cc=eperezma@redhat.com \
    --cc=gdawar@xilinx.com \
    --cc=jasowang@redhat.com \
    --cc=lingshan.zhu@intel.com \
    --cc=lulu@redhat.com \
    --cc=qemu-devel@nongnu.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 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.