Bug 1470035
Summary: | [qmp] Load internal snapshot failed on Power9 | ||
---|---|---|---|
Product: | Red Hat Enterprise Linux 7 | Reporter: | yilzhang |
Component: | qemu-kvm | Assignee: | Laurent Vivier <lvivier> |
Status: | CLOSED ERRATA | QA Contact: | yilzhang |
Severity: | high | Docs Contact: | |
Priority: | high | ||
Version: | 7.4-Alt | CC: | areis, bugproxy, fnovak, gsun, haizhao, hannsj_uhl, knoel, lvivier, mrezanin, qzhang, rbalakri, virt-maint, yilzhang |
Target Milestone: | rc | Keywords: | Patch |
Target Release: | 7.4-Alt | ||
Hardware: | ppc64le | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | qemu-kvm-2.9.0-19.el7a | Doc Type: | If docs needed, set a value |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2017-11-09 11:31:17 UTC | Type: | Bug |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: | |||
Bug Depends On: | |||
Bug Blocks: | 1440030 |
Description
yilzhang
2017-07-12 10:18:21 UTC
According to error message: {"return": "htab_load() bad index 2113929216 (9474+0 entries) in htab stream (htab_shift=0)\r\nerror while loading state for instance 0x0 of device 'spapr/htab'\r\nError -22 while loading VM state\r\n"} This is a duplicate of BZ1456287, please re-test with qemu-kvm-2.9.0-17.el7a After applying qemu-kvm-2.9.0-17.el7a on Power9 host, the testing result is as follows: 1. BZ 1456287 cannot be reproduced 2. But this bug still exists, and its actual result is a little different now: step4 ( Create internal snapshot with command 'savevm' ) hangs there (In reply to yilzhang from comment #3) > After applying qemu-kvm-2.9.0-17.el7a on Power9 host, the testing result is > as follows: > > 1. BZ 1456287 cannot be reproduced > 2. But this bug still exists, and its actual result is a little different > now: > step4 ( Create internal snapshot with command 'savevm' ) hangs there I think there is a bug in the bugfix patch. The end of sequence should be marked by a positive value, not by 0. diff --git a/hw/ppc/spapr.c b/hw/ppc/spapr.c index 970093e..fa01511 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); } @@ -1866,7 +1866,7 @@ static int htab_save_complete(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); } Hi Laurent, Using your latest build, this case passed without error. [host]# rpm -qa| grep qemu-kvm qemu-kvm-tools-rhev-2.9.0-17.el7a.lvivier201707172158.ppc64le qemu-kvm-common-rhev-2.9.0-17.el7a.lvivier201707172158.ppc64le qemu-kvm-rhev-2.9.0-17.el7a.lvivier201707172158.ppc64le qemu-kvm-rhev-debuginfo-2.9.0-17.el7a.lvivier201707172158.ppc64le Patch sent upstream: https://lists.nongnu.org/archive/html/qemu-devel/2017-07/msg05494.html ------- Comment From sthoufee.com 2017-08-01 08:21 EDT------- patch accepted upstream http://git.qemu.org/?p=qemu.git;a=commit;h=e8cd4247e96bb2158ef0ae0ff20e72746b9dd32d Fix included in qemu-kvm-2.9.0-19.el7a *** Bug 1473121 has been marked as a duplicate of this bug. *** This bug has been verified against the following version of components: host: kernel-4.11.0-22.el7a.ppc64le qemu-kvm-2.9.0-19.el7a.ppc64le SLOF-20170303-4.git66d250e.el7.noarch guest kernel: 4.11.0-16.el7a.ppc64le Actual results: load snapshot succeeds without error So, this bug is fixed. Since the problem described in this bug report should be resolved in a recent advisory, it has been closed with a resolution of ERRATA. For information on the advisory, and where to find the updated files, follow the link below. If the solution does not work for you, open a new bug report. https://access.redhat.com/errata/RHEA-2017:3169 |