commit 18f2971ce403008d5e1c2875b483c9d1778143dc
author Li Feng <fengli@smartx.com> Mon Jul 31 20:10:06 2023 +0800
committer Michael S. Tsirkin <mst@redhat.com> Thu Aug 03 16:06:49 2023 -0400
tree 14167bc0322a9a1eb63aaf536e3a98f7aa729f85
parent 348e354417b64c484877354ee7cc66f29fa6c7df

vhost: fix the fd leak

When the vhost-user reconnect to the backend, the notifer should be
cleanup. Otherwise, the fd resource will be exhausted.

Fixes: f9a09ca3ea ("vhost: add support for configure interrupt")

Signed-off-by: Li Feng <fengli@smartx.com>
Reviewed-by: Raphael Norwitz <raphael.norwitz@nutanix.com>
Message-Id: <20230731121018.2856310-2-fengli@smartx.com>
Reviewed-by: Michael S. Tsirkin <mst@redhat.com>
Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
Tested-by: Fiona Ebner <f.ebner@proxmox.com>

hw/virtio/vhost.c

diff --git a/hw/virtio/vhost.c b/hw/virtio/vhost.c
index abf0d03..e2f6ffb 100644
--- a/hw/virtio/vhost.c
+++ b/hw/virtio/vhost.c
@@ -2044,6 +2044,8 @@
     event_notifier_test_and_clear(
         &hdev->vqs[VHOST_QUEUE_NUM_CONFIG_INR].masked_config_notifier);
     event_notifier_test_and_clear(&vdev->config_notifier);
+    event_notifier_cleanup(
+        &hdev->vqs[VHOST_QUEUE_NUM_CONFIG_INR].masked_config_notifier);
 
     trace_vhost_dev_stop(hdev, vdev->name, vrings);