Re: [libvirt PATCH 62/80] qemu: Simplify cleanup in qemuMigrationDstPrepareFresh

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

 



On Tue, May 10, 2022 at 17:21:23 +0200, Jiri Denemark wrote:
> Since most of the cleanup code was moved out of this function, we don't
> need to preserve the original error here.
> 
> Signed-off-by: Jiri Denemark <jdenemar@xxxxxxxxxx>
> ---
>  src/qemu/qemu_migration.c | 6 +-----
>  1 file changed, 1 insertion(+), 5 deletions(-)

[...]

> @@ -3480,7 +3479,6 @@ qemuMigrationDstPrepareFresh(virQEMUDriver *driver,
>      ret = 0;
>  
>   cleanup:
> -    virErrorPreserveLast(&origErr);
>      if (ret < 0 && priv) {
>          /* priv is set right after vm is added to the list of domains
>           * and there is no 'goto cleanup;' in the middle of those */
> @@ -3489,8 +3487,6 @@ qemuMigrationDstPrepareFresh(virQEMUDriver *driver,
>          qemuDomainRemoveInactive(driver, vm);

I'm not sure how much we care, but qemuDomainRemoveInactive has
theoretical possibility to overwrite the error in at least the
snapshot/checkpoint removal code and also via qemuExtDevicesCleanupHost.

>      }
>      virDomainObjEndAPI(&vm);
> -    qemuMigrationCookieFree(mig);
> -    virErrorRestore(&origErr);
>      return ret;

If you add justification to the commit message outlining why it's okay
not to preserve the error or drop that part:

Reviewed-by: Peter Krempa <pkrempa@xxxxxxxxxx>




[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