Looks like this patch no longer applies cleanly but looks like the same cleanup to drop the local array may be fine?
On Wed, Aug 13, 2025 at 12:51 PM Vladimir Sementsov-Ogievskiy <[email protected]> wrote: > > Signed-off-by: Vladimir Sementsov-Ogievskiy <[email protected]> > --- > hw/virtio/vhost.c | 5 +---- > 1 file changed, 1 insertion(+), 4 deletions(-) > > diff --git a/hw/virtio/vhost.c b/hw/virtio/vhost.c > index f9163ba895..e796ad347d 100644 > --- a/hw/virtio/vhost.c > +++ b/hw/virtio/vhost.c > @@ -1596,7 +1596,6 @@ int vhost_dev_init(struct vhost_dev *hdev, void *opaque, > VhostBackendType backend_type, uint32_t busyloop_timeout, > Error **errp) > { > - uint64_t features; > int i, r, n_initialized_vqs = 0; > > hdev->vdev = NULL; > @@ -1616,7 +1615,7 @@ int vhost_dev_init(struct vhost_dev *hdev, void *opaque, > goto fail; > } > > - r = hdev->vhost_ops->vhost_get_features(hdev, &features); > + r = hdev->vhost_ops->vhost_get_features(hdev, &hdev->_features); > if (r < 0) { > error_setg_errno(errp, -r, "vhost_get_features failed"); > goto fail; > @@ -1631,8 +1630,6 @@ int vhost_dev_init(struct vhost_dev *hdev, void *opaque, > } > } > > - hdev->_features = features; > - > hdev->memory_listener = (MemoryListener) { > .name = "vhost", > .begin = vhost_begin, > -- > 2.48.1 > >
