diff options
| author | Laurent Vivier <lvivier@redhat.com> | 2017-07-18 12:16:32 +0200 |
|---|---|---|
| committer | David Gibson <david@gibson.dropbear.id.au> | 2017-07-25 11:14:25 +1000 |
| commit | e8cd4247e96bb2158ef0ae0ff20e72746b9dd32d (patch) | |
| tree | 6ecf1aea2fe690d3f6b11a25840b5cca144092cc /hw/ppc | |
| parent | 18f2330ef548be5f67342c01817f096c34ab90af (diff) | |
| download | focaccia-qemu-e8cd4247e96bb2158ef0ae0ff20e72746b9dd32d.tar.gz focaccia-qemu-e8cd4247e96bb2158ef0ae0ff20e72746b9dd32d.zip | |
spapr/htab: fix savevm
Commit 3a38429 ("spapr: Add a "no HPT" encoding to HTAB migration stream")
allows to migrate an empty HPT, but doesn't mark correctly the
end of the migration stream.
The end condition (value returned by htab_save_iterate())
should be 1, whereas in 3a38429 it returns 0.
The problem can be reproduced with QEMU monitor command "savevm":
the command never stops and the disk image grows without limit.
Fixes: 3a38429748aa4f74abaecf16c4c087e8a325e12a
Signed-off-by: Laurent Vivier <lvivier@redhat.com>
Reviewed-by: Thomas Huth <thuth@redhat.com>
Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
Diffstat (limited to 'hw/ppc')
| -rw-r--r-- | hw/ppc/spapr.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/hw/ppc/spapr.c b/hw/ppc/spapr.c index 970093e6b5..1cb09e7c64 100644 --- a/hw/ppc/spapr.c +++ b/hw/ppc/spapr.c @@ -1827,7 +1827,7 @@ static int htab_save_iterate(QEMUFile *f, void *opaque) /* Iteration header */ if (!spapr->htab_shift) { qemu_put_be32(f, -1); - return 0; + return 1; } else { qemu_put_be32(f, 0); } |