On Tue, Dec 03, 2019 at 06:17:44PM +0100, Peter Krempa wrote:
> Use the helper which cancels all blockjobs to perform the backup job
> cancellation in qemuDomainAbortJob.
>
> Signed-off-by: Peter Krempa <pkrempa@xxxxxxxxxx>
> ---
> src/qemu/qemu_driver.c | 15 ++++++++++-----
> 1 file changed, 10 insertions(+), 5 deletions(-)
>
> diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c
> index 95882d9d14..2408b08106 100644
> --- a/src/qemu/qemu_driver.c
> +++ b/src/qemu/qemu_driver.c
> @@ -14054,11 +14054,16 @@ static int qemuDomainAbortJob(virDomainPtr dom)
> }
>
> VIR_DEBUG("Cancelling job at client request");
> - qemuDomainObjAbortAsyncJob(vm);
> - qemuDomainObjEnterMonitor(driver, vm);
> - ret = qemuMonitorMigrateCancel(priv->mon);
> - if (qemuDomainObjExitMonitor(driver, vm) < 0)
> - ret = -1;
> + if (priv->job.asyncJob == QEMU_ASYNC_JOB_BACKUP) {
> + qemuBackupJobCancelBlockjobs(vm, priv->backup, true);
> + ret = 0;
> + } else {
> + qemuDomainObjAbortAsyncJob(vm);
> + qemuDomainObjEnterMonitor(driver, vm);
> + ret = qemuMonitorMigrateCancel(priv->mon);
> + if (qemuDomainObjExitMonitor(driver, vm) < 0)
> + ret = -1;
> + }
Hmm, this makes me thing we should have had some better error checking
in here already. IIUC, we have other types async job that are not
related to either migration or backups, so should we do
switch (priv->job.asyncJob) {
case QEMU_ASYNC_JOB_BACKUP:
...
case QEMU_ASYNC_JOB_MIGRATE:
...
case QEMU_ASYNC_JOB....
default:
report error
}
Regards,
Daniel
--
|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o- https://fstop138.berrange.com :|
|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|
--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list
