[Qemu-devel] [PATCH 0/4] libvhost-user: VHOST_USER_PROTOCOL_F_MQ support

Stefan Hajnoczi posted 4 patches 6 years, 4 months ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20190621094005.4134-1-stefanha@redhat.com
Test s390x passed
Test checkpatch passed
Test asan passed
Test docker-mingw@fedora passed
Test docker-clang@ubuntu passed
Test FreeBSD passed
Maintainers: "Michael S. Tsirkin" <mst@redhat.com>, "Marc-André Lureau" <marcandre.lureau@redhat.com>, Gerd Hoffmann <kraxel@redhat.com>
There is a newer version of this series
contrib/libvhost-user/libvhost-user-glib.h |  2 +-
contrib/libvhost-user/libvhost-user.h      | 10 +++-
contrib/libvhost-user/libvhost-user-glib.c | 12 +++-
contrib/libvhost-user/libvhost-user.c      | 65 +++++++++++++---------
contrib/vhost-user-blk/vhost-user-blk.c    | 16 +++---
contrib/vhost-user-gpu/main.c              |  9 ++-
contrib/vhost-user-input/main.c            | 11 +++-
contrib/vhost-user-scsi/vhost-user-scsi.c  | 21 +++----
tests/vhost-user-bridge.c                  | 42 +++++++++-----
docs/interop/vhost-user.rst                | 21 +++----
10 files changed, 132 insertions(+), 77 deletions(-)
[Qemu-devel] [PATCH 0/4] libvhost-user: VHOST_USER_PROTOCOL_F_MQ support
Posted by Stefan Hajnoczi 6 years, 4 months ago
Sebastien Boeuf <sebastien.boeuf@intel.com> pointed out that libvhost-user
doesn't advertise VHOST_USER_PROTOCOL_F_MQ.  Today this prevents vhost-user-net
multiqueue from working.

In virtio-fs we also want to support multiqueue so I'm sending patches to add
this.  It's free to advertise VHOST_USER_PROTOCOL_F_MQ for all devices so we
can do it unconditionally in libvhost-user.

Several related improvements are included:
Patch 1 - clean up duplicated and risky VhostUserMsg reply building code
Patch 2 - remove hardcoded 8 virtqueue limit in libvhost-user
Patch 4 - clarify vhost-user multiqueue specification

Stefan Hajnoczi (4):
  libvhost-user: add vmsg_set_reply_u64() helper
  libvhost-user: support many virtqueues
  libvhost-user: implement VHOST_USER_PROTOCOL_F_MQ
  docs: avoid vhost-user-net specifics in multiqueue section

 contrib/libvhost-user/libvhost-user-glib.h |  2 +-
 contrib/libvhost-user/libvhost-user.h      | 10 +++-
 contrib/libvhost-user/libvhost-user-glib.c | 12 +++-
 contrib/libvhost-user/libvhost-user.c      | 65 +++++++++++++---------
 contrib/vhost-user-blk/vhost-user-blk.c    | 16 +++---
 contrib/vhost-user-gpu/main.c              |  9 ++-
 contrib/vhost-user-input/main.c            | 11 +++-
 contrib/vhost-user-scsi/vhost-user-scsi.c  | 21 +++----
 tests/vhost-user-bridge.c                  | 42 +++++++++-----
 docs/interop/vhost-user.rst                | 21 +++----
 10 files changed, 132 insertions(+), 77 deletions(-)

-- 
2.21.0


Re: [Qemu-devel] [PATCH 0/4] libvhost-user: VHOST_USER_PROTOCOL_F_MQ support
Posted by Stefan Hajnoczi 6 years, 4 months ago
On Fri, Jun 21, 2019 at 10:40:01AM +0100, Stefan Hajnoczi wrote:
> Sebastien Boeuf <sebastien.boeuf@intel.com> pointed out that libvhost-user
> doesn't advertise VHOST_USER_PROTOCOL_F_MQ.  Today this prevents vhost-user-net
> multiqueue from working.
> 
> In virtio-fs we also want to support multiqueue so I'm sending patches to add
> this.  It's free to advertise VHOST_USER_PROTOCOL_F_MQ for all devices so we
> can do it unconditionally in libvhost-user.
> 
> Several related improvements are included:
> Patch 1 - clean up duplicated and risky VhostUserMsg reply building code
> Patch 2 - remove hardcoded 8 virtqueue limit in libvhost-user
> Patch 4 - clarify vhost-user multiqueue specification
> 
> Stefan Hajnoczi (4):
>   libvhost-user: add vmsg_set_reply_u64() helper
>   libvhost-user: support many virtqueues
>   libvhost-user: implement VHOST_USER_PROTOCOL_F_MQ
>   docs: avoid vhost-user-net specifics in multiqueue section
> 
>  contrib/libvhost-user/libvhost-user-glib.h |  2 +-
>  contrib/libvhost-user/libvhost-user.h      | 10 +++-
>  contrib/libvhost-user/libvhost-user-glib.c | 12 +++-
>  contrib/libvhost-user/libvhost-user.c      | 65 +++++++++++++---------
>  contrib/vhost-user-blk/vhost-user-blk.c    | 16 +++---
>  contrib/vhost-user-gpu/main.c              |  9 ++-
>  contrib/vhost-user-input/main.c            | 11 +++-
>  contrib/vhost-user-scsi/vhost-user-scsi.c  | 21 +++----
>  tests/vhost-user-bridge.c                  | 42 +++++++++-----
>  docs/interop/vhost-user.rst                | 21 +++----
>  10 files changed, 132 insertions(+), 77 deletions(-)
> 
> -- 
> 2.21.0

Ping?