[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 1/2] migration: return -EINVAL directly when ver
From: |
Dr. David Alan Gilbert |
Subject: |
Re: [Qemu-devel] [PATCH 1/2] migration: return -EINVAL directly when version_id mismatch |
Date: |
Tue, 23 Jul 2019 16:44:48 +0100 |
User-agent: |
Mutt/1.12.0 (2019-05-25) |
* Wei Yang (address@hidden) wrote:
> It is not reasonable to continue when version_id mismatch.
>
> Signed-off-by: Wei Yang <address@hidden>
> ---
> migration/ram.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/migration/ram.c b/migration/ram.c
> index 7e34c82a72..6bfdfae16e 100644
> --- a/migration/ram.c
> +++ b/migration/ram.c
> @@ -4216,7 +4216,7 @@ static int ram_load(QEMUFile *f, void *opaque, int
> version_id)
> seq_iter++;
>
> if (version_id != 4) {
> - ret = -EINVAL;
> + return -EINVAL;
> }
Oh yes that's nice; it's before the lock gets taken so we can just
return
Reviewed-by: Dr. David Alan Gilbert <address@hidden>
> if (!migrate_use_compression()) {
> --
> 2.17.1
>
--
Dr. David Alan Gilbert / address@hidden / Manchester, UK