On Thu, Jul 03, 2025 at 03:47:08PM +0300, Vladimir Sementsov-Ogievskiy wrote: > This field is mostly unused and sometimes confusing (we even have > a TODO-like comment to drop it). Let's finally do.
Breaks make check with UBSAN enabled: 32/109 /arm/virt/virtio-mmio/virtio-bus/virtio-net-device/virtio-net/virtio-net-tests/vhost-user/flags-mismatch - ERROR:../tests/qtest/qos-test.c:189:subprocess_run_one_test: child process (/arm/virt/virtio-mmio/virtio-bus/virtio-net-device/virtio-net/virtio-net-tests/vhost-user/flags-mismatch/subprocess [9701]) failed unexpectedly FAIL https://gitlab.com/mstredhat/qemu/-/jobs/10668177755 To trigger, configure with: ./configure '--cc=clang' '--cxx=clang++' '--enable-ubsan' '--extra-cflags=-fno-sanitize-recover=undefined -fno-sanitize=pointer-overflow' '--target-list=arm-softmmu' make make check > Vladimir Sementsov-Ogievskiy (4): > vhost: introduce vhost_ops->vhost_set_vring_enable_supported method > vhost-user: stop use backend_features > vhost_net: stop use backend_features > hw/vhost: finally drop vhost_dev.backend_features field > > hw/block/vhost-user-blk.c | 1 - > hw/net/vhost_net.c | 14 ++++++-------- > hw/scsi/vhost-scsi.c | 1 - > hw/scsi/vhost-user-scsi.c | 1 - > hw/virtio/vdpa-dev.c | 1 - > hw/virtio/vhost-user.c | 25 ++++++++++++++++--------- > hw/virtio/vhost.c | 15 ++++++--------- > hw/virtio/virtio-qmp.c | 2 -- > include/hw/virtio/vhost-backend.h | 2 ++ > include/hw/virtio/vhost.h | 7 ------- > 10 files changed, 30 insertions(+), 39 deletions(-) > > -- > 2.48.1
