[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PULL 3/5] spapr: fix spapr-nvram migration
From: |
David Gibson |
Subject: |
[Qemu-devel] [PULL 3/5] spapr: fix spapr-nvram migration |
Date: |
Tue, 26 Jul 2016 13:06:09 +1000 |
From: "address@hidden" <address@hidden>
When spapr-nvram is backed by a file using pflash interface,
migration fails on the destination guest with assert:
bdrv_co_pwritev: Assertion `!(bs->open_flags & 0x0800)' failed.
This avoids the problem by delaying the pflash update until after
the device loads complete.
This fix is similar to the one for the pflash_cfi01 migration:
90c647d Fix pflash migration
Signed-off-by: Laurent Vivier <address@hidden>
Reviewed-by: Paolo Bonzini <address@hidden>
Signed-off-by: David Gibson <address@hidden>
---
hw/nvram/spapr_nvram.c | 23 +++++++++++++++--------
1 file changed, 15 insertions(+), 8 deletions(-)
diff --git a/hw/nvram/spapr_nvram.c b/hw/nvram/spapr_nvram.c
index 019f25d..4de5f70 100644
--- a/hw/nvram/spapr_nvram.c
+++ b/hw/nvram/spapr_nvram.c
@@ -39,6 +39,7 @@ typedef struct sPAPRNVRAM {
uint32_t size;
uint8_t *buf;
BlockBackend *blk;
+ VMChangeStateEntry *vmstate;
} sPAPRNVRAM;
#define TYPE_VIO_SPAPR_NVRAM "spapr-nvram"
@@ -185,19 +186,25 @@ static int spapr_nvram_pre_load(void *opaque)
return 0;
}
+static void postload_update_cb(void *opaque, int running, RunState state)
+{
+ sPAPRNVRAM *nvram = opaque;
+
+ /* This is called after bdrv_invalidate_cache_all. */
+
+ qemu_del_vm_change_state_handler(nvram->vmstate);
+ nvram->vmstate = NULL;
+
+ blk_pwrite(nvram->blk, 0, nvram->buf, nvram->size, 0);
+}
+
static int spapr_nvram_post_load(void *opaque, int version_id)
{
sPAPRNVRAM *nvram = VIO_SPAPR_NVRAM(opaque);
if (nvram->blk) {
- int alen = blk_pwrite(nvram->blk, 0, nvram->buf, nvram->size, 0);
-
- if (alen < 0) {
- return alen;
- }
- if (alen != nvram->size) {
- return -1;
- }
+ nvram->vmstate = qemu_add_vm_change_state_handler(postload_update_cb,
+ nvram);
}
return 0;
--
2.7.4
- [Qemu-devel] [PULL 0/5] ppc-for-2.7 queue 20160726, David Gibson, 2016/07/25
- [Qemu-devel] [PULL 1/5] ppc: Huge page detection mechanism fixes - Episode III, David Gibson, 2016/07/25
- [Qemu-devel] [PULL 2/5] hw/ppc/spapr: Make sure to close the htab_fd when migration is canceled, David Gibson, 2016/07/25
- [Qemu-devel] [PULL 4/5] target-ppc: add PPC_MFTB flag to e500mc and e5500, David Gibson, 2016/07/25
- [Qemu-devel] [PULL 5/5] spapr: disintricate core-id from DT semantics, David Gibson, 2016/07/25
- [Qemu-devel] [PULL 3/5] spapr: fix spapr-nvram migration,
David Gibson <=
- Re: [Qemu-devel] [PULL 0/5] ppc-for-2.7 queue 20160726, Peter Maydell, 2016/07/26