On 4/10/25 08:58, Laine Stump via Devel wrote:
> This brings behavior of passt+vhostuser in line with trad-passt - if
> the passt process is somehow terminated, libvirt will start a new
> passt process that qemu will connect to.
>
> You'll notice no change in behavior if your QEMU binary doesn't have
> the patches that add NETDEV_VHOST_USER_DISCONNECTED to qemu, but also
> nothing extra will fail. The requisite QEMU patches are upstream in
> commit 02fd9f8aeeb1 ("net: vhost-user: add QAPI events to report
> connection state"), which will be in upstream qemu-10.0.0.
>
> Resolves: https://issues.redhat.com/browse/RHEL-80169
>
> Laine Stump (5):
> qemu: remove nonsensical sanity check in
> processNetdevStreamDisconnectedEvent()
> qemu: make processNetDevStreamDisconnectedEvent() reusable
> qemu: respond to NETDEV_VHOST_USER_DISCONNECTED event
> qemu: put vhost-user code that's special for passt in a helper
> function
> qemu: make passt+vhostuser reconnect behave identically to passt+user
>
> src/qemu/qemu_domain.c | 1 +
> src/qemu/qemu_domain.h | 1 +
> src/qemu/qemu_driver.c | 55 ++++++++++++-------
> src/qemu/qemu_hotplug.c | 7 +--
> src/qemu/qemu_monitor.c | 11 ++++
> src/qemu/qemu_monitor.h | 6 ++
> src/qemu/qemu_monitor_json.c | 16 ++++++
> src/qemu/qemu_passt.c | 25 +++++++++
> src/qemu/qemu_passt.h | 3 +
> src/qemu/qemu_process.c | 27 ++++++---
> .../net-vhostuser-passt.x86_64-latest.args | 6 +-
> ...rder-domain-subelements.x86_64-latest.args | 2 +-
> 12 files changed, 122 insertions(+), 38 deletions(-)
>
Reviewed-by: Michal Privoznik <mpriv...@redhat.com>
Michal
