[PATCH v3 00/23] vhost refactoring and fixes

Vladimir Sementsov-Ogievskiy posted 23 patches 1 month ago
Failed in applying to current master (apply log)
Maintainers: "Michael S. Tsirkin" <mst@redhat.com>, Stefano Garzarella <sgarzare@redhat.com>, "Gonglei (Arei)" <arei.gonglei@huawei.com>, Zhenwei Pi <pizhenwei@bytedance.com>, Kevin Wolf <kwolf@redhat.com>, Hanna Reitz <hreitz@redhat.com>, Raphael Norwitz <raphael@enfabrica.net>, "Marc-André Lureau" <marcandre.lureau@redhat.com>, Jason Wang <jasowang@redhat.com>, Paolo Bonzini <pbonzini@redhat.com>, Fam Zheng <fam@euphon.net>, "Alex Bennée" <alex.bennee@linaro.org>, Eric Blake <eblake@redhat.com>, Markus Armbruster <armbru@redhat.com>
backends/cryptodev-vhost.c     |   9 +-
hw/block/trace-events          |  10 ++
hw/block/vhost-user-blk.c      |  20 ++-
hw/display/vhost-user-gpu.c    |  11 +-
hw/net/vhost_net.c             |  35 ++--
hw/scsi/vhost-scsi.c           |   1 -
hw/scsi/vhost-user-scsi.c      |   1 -
hw/virtio/trace-events         |  16 +-
hw/virtio/vdpa-dev.c           |   3 +-
hw/virtio/vhost-user-base.c    |   8 +-
hw/virtio/vhost-user.c         | 285 ++++++++++++++++++++++---------
hw/virtio/vhost.c              | 300 +++++++++++++++++----------------
hw/virtio/virtio-bus.c         |  18 +-
hw/virtio/virtio-hmp-cmds.c    |   2 -
hw/virtio/virtio-mmio.c        |  41 +----
hw/virtio/virtio-pci.c         |  34 +---
hw/virtio/virtio-qmp.c         |  12 +-
hw/virtio/virtio.c             |  48 +++---
include/hw/virtio/vhost-user.h |   3 +
include/hw/virtio/vhost.h      |  63 +++++--
include/hw/virtio/virtio-pci.h |   3 -
include/hw/virtio/virtio.h     |  16 +-
net/vhost-vdpa.c               |   7 +-
qapi/virtio.json               |   3 -
24 files changed, 537 insertions(+), 412 deletions(-)
[PATCH v3 00/23] vhost refactoring and fixes
Posted by Vladimir Sementsov-Ogievskiy 1 month ago
Hi all. That's rebased and updated first part of
  [PATCH 00/33] vhost-user-blk: live-backend local migration

v3:
01-05,07-23: add r-b by Daniil
01,03,04,11-12,15,19,22,23: add r-b by Raphael
02: - add a-b by Markus and Raphael
    - touch-up commit message
06: carefully follow existing pattern
    of split into features / features_ex

Vladimir Sementsov-Ogievskiy (23):
  vhost-user: rework enabling vrings
  vhost: drop backend_features field
  vhost-user: introduce vhost_user_has_protocol_feature() helper
  vhost: move protocol_features to vhost_user
  vhost-user-gpu: drop code duplication
  vhost: make vhost_dev.features private
  virtio: move common part of _set_guest_notifier to generic code
  virtio: drop *_set_guest_notifier_fd_handler() helpers
  vhost-user: keep QIOChannelSocket for backend channel
  vhost: vhost_virtqueue_start(): fix failure path
  vhost: make vhost_memory_unmap() null-safe
  vhost: simplify calls to vhost_memory_unmap()
  vhost: move vrings mapping to the top of vhost_virtqueue_start()
  vhost: vhost_virtqueue_start(): drop extra local variables
  vhost: final refactoring of vhost vrings map/unmap
  vhost: simplify vhost_dev_init() error-path
  vhost: move busyloop timeout initialization to vhost_virtqueue_init()
  vhost: introduce check_memslots() helper
  vhost: vhost_dev_init(): simplify features initialization
  hw/virtio/virtio-bus: refactor virtio_bus_set_host_notifier()
  vhost-user: make trace events more readable
  vhost-user-blk: add some useful trace-points
  vhost: add some useful trace-points

 backends/cryptodev-vhost.c     |   9 +-
 hw/block/trace-events          |  10 ++
 hw/block/vhost-user-blk.c      |  20 ++-
 hw/display/vhost-user-gpu.c    |  11 +-
 hw/net/vhost_net.c             |  35 ++--
 hw/scsi/vhost-scsi.c           |   1 -
 hw/scsi/vhost-user-scsi.c      |   1 -
 hw/virtio/trace-events         |  16 +-
 hw/virtio/vdpa-dev.c           |   3 +-
 hw/virtio/vhost-user-base.c    |   8 +-
 hw/virtio/vhost-user.c         | 285 ++++++++++++++++++++++---------
 hw/virtio/vhost.c              | 300 +++++++++++++++++----------------
 hw/virtio/virtio-bus.c         |  18 +-
 hw/virtio/virtio-hmp-cmds.c    |   2 -
 hw/virtio/virtio-mmio.c        |  41 +----
 hw/virtio/virtio-pci.c         |  34 +---
 hw/virtio/virtio-qmp.c         |  12 +-
 hw/virtio/virtio.c             |  48 +++---
 include/hw/virtio/vhost-user.h |   3 +
 include/hw/virtio/vhost.h      |  63 +++++--
 include/hw/virtio/virtio-pci.h |   3 -
 include/hw/virtio/virtio.h     |  16 +-
 net/vhost-vdpa.c               |   7 +-
 qapi/virtio.json               |   3 -
 24 files changed, 537 insertions(+), 412 deletions(-)

-- 
2.48.1
Re: [PATCH v3 00/23] vhost refactoring and fixes
Posted by Lei Yang 3 weeks, 6 days ago
Tested this series of patches with virtio-net regression tests,
everything works fine.

Tested-by: Lei Yang <leiyang@redhat.com>


On Wed, Oct 15, 2025 at 11:03 PM Vladimir Sementsov-Ogievskiy
<vsementsov@yandex-team.ru> wrote:
>
> Hi all. That's rebased and updated first part of
>   [PATCH 00/33] vhost-user-blk: live-backend local migration
>
> v3:
> 01-05,07-23: add r-b by Daniil
> 01,03,04,11-12,15,19,22,23: add r-b by Raphael
> 02: - add a-b by Markus and Raphael
>     - touch-up commit message
> 06: carefully follow existing pattern
>     of split into features / features_ex
>
> Vladimir Sementsov-Ogievskiy (23):
>   vhost-user: rework enabling vrings
>   vhost: drop backend_features field
>   vhost-user: introduce vhost_user_has_protocol_feature() helper
>   vhost: move protocol_features to vhost_user
>   vhost-user-gpu: drop code duplication
>   vhost: make vhost_dev.features private
>   virtio: move common part of _set_guest_notifier to generic code
>   virtio: drop *_set_guest_notifier_fd_handler() helpers
>   vhost-user: keep QIOChannelSocket for backend channel
>   vhost: vhost_virtqueue_start(): fix failure path
>   vhost: make vhost_memory_unmap() null-safe
>   vhost: simplify calls to vhost_memory_unmap()
>   vhost: move vrings mapping to the top of vhost_virtqueue_start()
>   vhost: vhost_virtqueue_start(): drop extra local variables
>   vhost: final refactoring of vhost vrings map/unmap
>   vhost: simplify vhost_dev_init() error-path
>   vhost: move busyloop timeout initialization to vhost_virtqueue_init()
>   vhost: introduce check_memslots() helper
>   vhost: vhost_dev_init(): simplify features initialization
>   hw/virtio/virtio-bus: refactor virtio_bus_set_host_notifier()
>   vhost-user: make trace events more readable
>   vhost-user-blk: add some useful trace-points
>   vhost: add some useful trace-points
>
>  backends/cryptodev-vhost.c     |   9 +-
>  hw/block/trace-events          |  10 ++
>  hw/block/vhost-user-blk.c      |  20 ++-
>  hw/display/vhost-user-gpu.c    |  11 +-
>  hw/net/vhost_net.c             |  35 ++--
>  hw/scsi/vhost-scsi.c           |   1 -
>  hw/scsi/vhost-user-scsi.c      |   1 -
>  hw/virtio/trace-events         |  16 +-
>  hw/virtio/vdpa-dev.c           |   3 +-
>  hw/virtio/vhost-user-base.c    |   8 +-
>  hw/virtio/vhost-user.c         | 285 ++++++++++++++++++++++---------
>  hw/virtio/vhost.c              | 300 +++++++++++++++++----------------
>  hw/virtio/virtio-bus.c         |  18 +-
>  hw/virtio/virtio-hmp-cmds.c    |   2 -
>  hw/virtio/virtio-mmio.c        |  41 +----
>  hw/virtio/virtio-pci.c         |  34 +---
>  hw/virtio/virtio-qmp.c         |  12 +-
>  hw/virtio/virtio.c             |  48 +++---
>  include/hw/virtio/vhost-user.h |   3 +
>  include/hw/virtio/vhost.h      |  63 +++++--
>  include/hw/virtio/virtio-pci.h |   3 -
>  include/hw/virtio/virtio.h     |  16 +-
>  net/vhost-vdpa.c               |   7 +-
>  qapi/virtio.json               |   3 -
>  24 files changed, 537 insertions(+), 412 deletions(-)
>
> --
> 2.48.1
>
>