[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH v11 01/27] tools/libxl: introduction of libxl__qmp_restore to load qemu state



On 03/05/2016 12:30 AM, Ian Jackson wrote:
Changlong Xie writes ("[PATCH v11 01/27] tools/libxl: introduction of 
libxl__qmp_restore to load qemu state"):
From: Wen Congyang <wency@xxxxxxxxxxxxxx>

In normal migration, the qemu state is passed to qemu as a parameter.
With COLO, secondary vm is running. So we will do the following steps
at every checkpoint:
1. suspend both primary vm and secondary vm
2. sync the state
3. resume both primary vm and secondary vm
Primary will send qemu's state in step2, and secondary's qemu should
read it and restore the state before it is resumed. We can not pass the
state to qemu as a parameter because secondary QEMU is already started
at this point, so we introduce libxl__qmp_restore() to do it.

We should accept both the Xen part and the Qemu part, or neither.  So
I think this proposal needs (in the Xen commit message) a reference to
the Qemu upstream commit, or Qemu upstream patches, so that we can see
that they've got the appropriate Qemu upstream acks.

But, subject to that, from a Xen point of view this approach looks
good to me.

Acked-by: Ian Jackson <ian.jackson@xxxxxxxxxxxxx>

Hi Ian

We now push this qemu relevant patch in
http://lists.nongnu.org/archive/html/qemu-devel/2016-03/msg02955.html

Will update the commit messege when it's merged.


Thanks
        -Xie
Ian.


.




_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.