[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] Re: [Xen-devel] [PATCH v2 COLOPre 05/13] tools/libxl: introduce a new API libxl__domain_restore() to load qemu state
On Thu, 2015-06-11 at 16:55 +0800, Yang Hongyang wrote: > > On 06/11/2015 04:43 PM, Ian Campbell wrote: > > On Thu, 2015-06-11 at 10:09 +0800, Yang Hongyang wrote: > >> > >> On 06/10/2015 11:35 PM, Ian Campbell wrote: > >>> On Mon, 2015-06-08 at 11:43 +0800, Yang Hongyang wrote: > >>>> Secondary vm is running in colo mode. So we will do > >>>> the following things again and again: > >>>> 1. suspend both primay vm and secondary vm > >>>> 2. sync the state > >>>> 3. resume both primary vm and secondary vm > >>>> We will send qemu's state each time in step2, and > >>>> slave's qemu should read it each time before resuming > >>>> secondary vm. Introduce a new API libxl__domain_restore() > >>>> to do it. This API should be called before resuming > >>>> secondary vm. > >>> > >>> Is this a preexisting qemu interface or one to be added? > >> > >> We added the qemu interface "xen-load-devices-state", > >> it's not in qemu upstream yet. > > > > OK, please mention this dependency in the commit text since we will want > > to be sure the interface is going to be accepted in this form by QEMU > > upstream before we start using it. Please also CC the QEMU maintainers > > on this patch in the future (by adding Cc: below the S-o-b if you don't > > want to spam them the whole series), I've added them here now. > > > > In particular "devices" seems odd to me, perhaps > > "xen-load-device-state"? > > This api is an invert operation to "xen-save-devices-stat", we used the name > "xen-load-devices-state" in order to follow the existing naming style... OK, I guess it'll do then, thanks. _______________________________________________ Xen-devel mailing list Xen-devel@xxxxxxxxxxxxx http://lists.xen.org/xen-devel
|
Lists.xenproject.org is hosted with RackSpace, monitoring our |