qemu-e2k/contrib/libvhost-user
Johannes Berg a7290a79fa libvhost-user-glib: fix VugDev main fd cleanup
If you try to make a device implementation that can handle multiple
connections and allow disconnections (which requires overriding the
VHOST_USER_NONE handling), then glib will warn that we remove a src
while it's still on the mainloop, and will poll() an FD that doesn't
exist anymore.

Fix this by making vug_source_new() require pairing with the new
vug_source_destroy() so we can keep the GSource referenced in the
meantime.

Note that this requires calling the new API in vhost-user-input.
vhost-user-gpu also uses vug_source_new(), but never seems to free
the result at all, so I haven't changed anything there.

Fixes: 8bb7ddb78a ("libvhost-user: add glib source helper")
Reviewed-by: Marc-André Lureau <marcandre.lureau@redhat.com>
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
Message-Id: <20200123081708.7817-3-johannes@sipsolutions.net>
Reviewed-by: Michael S. Tsirkin <mst@redhat.com>
Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
2020-02-27 03:46:10 -05:00
..
libvhost-user-glib.c libvhost-user-glib: fix VugDev main fd cleanup 2020-02-27 03:46:10 -05:00
libvhost-user-glib.h libvhost-user-glib: fix VugDev main fd cleanup 2020-02-27 03:46:10 -05:00
libvhost-user.c libvhost-user: implement VHOST_USER_PROTOCOL_F_REPLY_ACK 2020-02-27 03:46:10 -05:00
libvhost-user.h libvhost-user: Fix some memtable remap cases 2020-01-23 16:41:37 +00:00
Makefile.objs libvhost-user: add glib source helper 2017-10-10 23:31:09 +02:00