Jason Andryuk writes ("[PATCH v5 20/21] libxl: Kill vchan-socket-proxy when
cleaning up qmp"):
> We need to kill the vchan-socket-proxy so we don't leak the daemonized
> processes. libxl__stubdomain_is_linux_running works against the
> guest_domid, but the xenstore path is beneath the stubdomain. This
> leads to the use of libxl_is_stubdom in addition to
> libxl__stubdomain_is_linux_running so that the stubdomain calls kill for
> the qmp-proxy
In theory maybe this patch should be folded into the one that
introduces the vchan proxy. But since this whole mode of operation is
new, having a point in the history where it leaks these is OK I
think.
Do others agree ?
For my part,
Acked-by: Ian Jackson <ian.jack...@eu.citrix.com>
However, if possible maybe this patch could be moved to right after
the one which spawns the proxy ?
Thanks,
Ian.
