Despite of the subject (1/4), this is just a single patch (sorry).

> On 27 October 2019 08:24 Dietmar Maurer <diet...@proxmox.com> wrote:
> 
>  
> The backup loop will automatically abort if we return an error.
> ---
>  blockdev.c | 6 ++----
>  1 file changed, 2 insertions(+), 4 deletions(-)
> 
> diff --git a/blockdev.c b/blockdev.c
> index 07561b6f96..3343388978 100644
> --- a/blockdev.c
> +++ b/blockdev.c
> @@ -3254,10 +3254,8 @@ static int coroutine_fn pvebackup_co_dump_cb(void 
> *opaque, BlockBackend *target,
>                  if (!backup_state.error) {
>                      vma_writer_error_propagate(backup_state.vmaw, 
> &backup_state.error);
>                  }
> -                if (di->bs && di->bs->job) {
> -                    job_cancel(&di->bs->job->job, true);
> -                }
> -                break;
> +                qemu_co_mutex_unlock(&backup_state.backup_mutex);
> +                return ret;
>              } else {
>                  backup_state.zero_bytes += zero_bytes;
>                  if (remaining >= VMA_CLUSTER_SIZE) {
> -- 
> 2.20.1

_______________________________________________
pve-devel mailing list
pve-devel@pve.proxmox.com
https://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

Reply via email to