Re: [libvirt PATCH] src: ensure GSource background unref happens in correct event loop

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Mar 16, 2021 at 04:30:32PM +0000, Daniel P. Berrangé wrote:
The g_idle_add function	adds a callback	to the primary GMainContext.

To workaround the GSource unref	bugs, we need to add our callbacks
to the GMainContext that is associated with the	GSource	being
unref'd. Thus code using the per-VM virEventThread must use its
private	GMainContext.

Signed-off-by: Daniel P. Berrangé <berrange@xxxxxxxxxx>
---
meson.build                            |  3 ---
src/libvirt_glib_crash_workaround.syms | 11 -----------
src/libvirt_private.syms               |  1 +
src/meson.build                        |  7 -------
src/qemu/qemu_agent.c                  |  2 +-
src/qemu/qemu_monitor.c                |  2 +-
src/util/glibcompat.c                  | 20 ++++++++++++++++++--
src/util/glibcompat.h                  | 11 +----------
src/util/vireventglib.c                | 12 ++++++------
9 files changed, 28 insertions(+), 41 deletions(-)
delete mode 100644 src/libvirt_glib_crash_workaround.syms


This fixes it for me and it makes sense.  Thanks!

Reviewed-by: Martin Kletzander <mkletzan@xxxxxxxxxx>

Attachment: signature.asc
Description: PGP signature


[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux