[libvirt] [PATCH v2] qemu: fix pr-helper0 remain
wangjie (P)
wangjie88 at huawei.com
Thu Jun 13 12:59:10 UTC 2019
Ping
On 2019/5/31 15:33, Jie Wang wrote:
> If libvirt receive DISCONNECTED event and set prDaemonRunning to false,
> and qemuDomainRemoveDiskDevice is performing in the meantime.
> qemuDomainRemoveDiskDevice will return directly by prDaemonRunning
> check, so the pr-helper0 object will remain.
> In that case we should try harder and also kill the pr-helper process,
> so qemuProcessKillManagedPRDaemon could use the same treatment.
>
> Signed-off-by: Jie Wang <wangjie88 at huawei.com>
> ---
> src/qemu/qemu_hotplug.c | 3 +--
> src/qemu/qemu_process.c | 3 ---
> 2 files changed, 1 insertion(+), 5 deletions(-)
>
> diff --git a/src/qemu/qemu_hotplug.c b/src/qemu/qemu_hotplug.c
> index 34249bd..5e4a929 100644
> --- a/src/qemu/qemu_hotplug.c
> +++ b/src/qemu/qemu_hotplug.c
> @@ -465,8 +465,7 @@ qemuHotplugRemoveManagedPR(virQEMUDriverPtr driver,
> virErrorPtr orig_err;
> int ret = -1;
>
> - if (!priv->prDaemonRunning ||
> - virDomainDefHasManagedPR(vm->def))
> + if (virDomainDefHasManagedPR(vm->def))
> return 0;
>
> virErrorPreserveLast(&orig_err);
> diff --git a/src/qemu/qemu_process.c b/src/qemu/qemu_process.c
> index 47d8ca2..8070a09 100644
> --- a/src/qemu/qemu_process.c
> +++ b/src/qemu/qemu_process.c
> @@ -2746,9 +2746,6 @@ qemuProcessKillManagedPRDaemon(virDomainObjPtr vm)
> virErrorPtr orig_err;
> char *pidfile;
>
> - if (!priv->prDaemonRunning)
> - return;
> -
> if (!(pidfile = qemuProcessBuildPRHelperPidfilePath(vm))) {
> VIR_WARN("Unable to construct pr-helper pidfile path");
> return;
More information about the libvir-list
mailing list