[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v4 1/1] Introduce "xen-load-devices-state"
From: |
Stefano Stabellini |
Subject: |
Re: [Qemu-devel] [PATCH v4 1/1] Introduce "xen-load-devices-state" |
Date: |
Mon, 30 May 2016 16:17:10 +0100 (BST) |
User-agent: |
Alpine 2.10 (DEB 1266 2009-07-14) |
On Fri, 27 May 2016, Anthony PERARD wrote:
> On Mon, Apr 11, 2016 at 11:56:02AM +0800, Changlong Xie wrote:
> > From: Wen Congyang <address@hidden>
> >
> > Introduce a "xen-load-devices-state" QAPI command that can be used to
> > load the state of all devices, but not the RAM or the block devices of
> > the VM.
> >
> > We only have hmp commands savevm/loadvm, and qmp commands
> > xen-save-devices-state.
> >
> > We use this new command for COLO:
> > 1. suspend both primary vm and secondary vm
> > 2. sync the state
> > 3. resume both primary vm and secondary vm
> >
> > In such case, we need to update all devices' state in any time.
> >
> > Signed-off-by: Wen Congyang <address@hidden>
> > Signed-off-by: Changlong Xie <address@hidden>
>
> This patch looks good to me.
>
> Reviewed-by: Anthony PERARD <address@hidden>
It would be nicer (and less problematic) to load the state from a file
descriptor, but given that we still saving the state to file, it would
be unfair to ask to use file descriptors here.
Acked-by: Stefano Stabellini <address@hidden>
Given that this is migration code, it still needs an ack from Juan or
Amit.