[PATCH v3 0/5] vhost-user-blk: fix compatibility with older qemu versions

Alexandr Moshkov posted 5 patches 2 days, 17 hours ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20260330095226.158386-1-dtalexundeer@yandex-team.ru
Maintainers: "Gonglei (Arei)" <arei.gonglei@huawei.com>, zhenwei pi <zhenwei.pi@linux.dev>, "Michael S. Tsirkin" <mst@redhat.com>, Stefano Garzarella <sgarzare@redhat.com>, Pierrick Bouvier <pierrick.bouvier@linaro.org>, Raphael Norwitz <rnorwitz@nvidia.com>, Kevin Wolf <kwolf@redhat.com>, Hanna Reitz <hreitz@redhat.com>, Jason Wang <jasowang@redhat.com>, Paolo Bonzini <pbonzini@redhat.com>, Fam Zheng <fam@euphon.net>, "Alex Bennée" <alex.bennee@linaro.org>, Stefan Hajnoczi <stefanha@redhat.com>, Milan Zamazal <mzamazal@redhat.com>
backends/cryptodev-vhost.c     |  2 +-
backends/vhost-user.c          |  2 +-
docs/interop/vhost-user.rst    |  8 +++----
hw/block/vhost-user-blk.c      | 43 +++++++++++++++++++++++++++-------
hw/net/vhost_net.c             |  9 +++----
hw/scsi/vhost-scsi-common.c    |  2 +-
hw/virtio/vdpa-dev.c           |  2 +-
hw/virtio/vhost-user-base.c    |  2 +-
hw/virtio/vhost-user-fs.c      |  2 +-
hw/virtio/vhost-user-scmi.c    |  2 +-
hw/virtio/vhost-user.c         |  3 +--
hw/virtio/vhost-vsock-common.c |  2 +-
hw/virtio/vhost.c              | 29 ++++++++++++++++-------
include/hw/virtio/vhost-user.h |  1 -
include/hw/virtio/vhost.h      |  7 ++++--
15 files changed, 76 insertions(+), 40 deletions(-)
[PATCH v3 0/5] vhost-user-blk: fix compatibility with older qemu versions
Posted by Alexandr Moshkov 2 days, 17 hours ago
v2 -> v3:
- fix complile problems
- add assert check in do_vhost_virtio_stop
- make inflight-migration property mutable

v1 -> v2:
- reorganize commits: make refactor commits first, then core semantic change
- add additional pre_save check for inflight migration possibility

---

This is a small continuation of my series about inflight migration for vhost-user-blk.

This series is designed to solve the problem of compatibility with older versions of qemu, where this feature has not yet been introduced (for example, if we want to downgrade versions due to some problems).

In the current version for vhost-user-blk, this feature is enabled using the parameter and further migration of the inflight region will depend on whether the vhost-user has accepted the new protocol feature or not. This creates an inconvenient dependency, because there is no way to disable this feature without reconnecting to the backend.
This series slightly changes the semantics of the introduced protocol feature VHOST_USER_PROTOCOL_F_GET_VRING_BASE_INFLIGHT. Enabling this feature adds a new parameter for GET_VRING_BASE message - skip_drain, which allows to control drain in-flight requests on the backend.

Thus, user can enable or disable inflight-migration param for vhost-user-blk to maintain compatibility with older versions of QEMU.

Alexandr Moshkov (5):
  vhost-user.rst: fix typo
  vhost-user-blk: make inflight-migration prop mutable
  vhost-user: add skip_drain param to do_vhost_virtqueue_stop
  vhost-user-blk: move inflight_needed higher
  vhost-user: add skip_drain param to GET_VRING_BASE

 backends/cryptodev-vhost.c     |  2 +-
 backends/vhost-user.c          |  2 +-
 docs/interop/vhost-user.rst    |  8 +++----
 hw/block/vhost-user-blk.c      | 43 +++++++++++++++++++++++++++-------
 hw/net/vhost_net.c             |  9 +++----
 hw/scsi/vhost-scsi-common.c    |  2 +-
 hw/virtio/vdpa-dev.c           |  2 +-
 hw/virtio/vhost-user-base.c    |  2 +-
 hw/virtio/vhost-user-fs.c      |  2 +-
 hw/virtio/vhost-user-scmi.c    |  2 +-
 hw/virtio/vhost-user.c         |  3 +--
 hw/virtio/vhost-vsock-common.c |  2 +-
 hw/virtio/vhost.c              | 29 ++++++++++++++++-------
 include/hw/virtio/vhost-user.h |  1 -
 include/hw/virtio/vhost.h      |  7 ++++--
 15 files changed, 76 insertions(+), 40 deletions(-)

-- 
2.34.1