From mboxrd@z Thu Jan 1 00:00:00 1970 From: Konrad Rzeszutek Wilk Subject: Re: [PATCH v7 08/18] tools/libxl: introduce libxl__domain_restore_device_model to load qemu state Date: Fri, 29 Jan 2016 11:34:11 -0500 Message-ID: <20160129163411.GI22787@char.us.oracle.com> References: <1454045254-3711-1-git-send-email-wency@cn.fujitsu.com> <1454045254-3711-9-git-send-email-wency@cn.fujitsu.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: <1454045254-3711-9-git-send-email-wency@cn.fujitsu.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Wen Congyang Cc: Lars Kurth , Changlong Xie , Wei Liu , Ian Campbell , Andrew Cooper , Jiang Yunhong , Dong Eddie , xen devel , Anthony Perard , Gui Jianfeng , Shriram Rajagopalan , Ian Jackson , Yang Hongyang List-Id: xen-devel@lists.xenproject.org On Fri, Jan 29, 2016 at 01:27:24PM +0800, Wen Congyang wrote: > 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 already started > at this point, so we introduce libxl__domain_restore_device_model() to > do it. This API MUST be called before resuming secondary vm. > > Signed-off-by: Yang Hongyang > Signed-off-by: Wen Congyang > Cc: Anthony Perard Reviewed-by: Konrad Rzeszutek Wilk