From mboxrd@z Thu Jan 1 00:00:00 1970 From: "James (song wei)" Subject: [PATCH] fix save or snapshot at kernel loading time with HVM vm Date: Sun, 16 May 2010 22:57:23 -0700 (PDT) Message-ID: <28579732.post@talk.nabble.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 7bit Return-path: List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xensource.com Errors-To: xen-devel-bounces@lists.xensource.com To: xen-devel@lists.xensource.com List-Id: xen-devel@lists.xenproject.org If doing save or snapshot at the time of kernel loading of a HVN vm, event_channel haven't been initialized, so that make this saving with xenstore. a string named "suspend" not "suspended" will be sent to xend from checkpoint thread. But in the shutdown function with parameter "suspend" think this request should shutdown this hvm guest, but obviously, this is not the expectation of save or snapshot. -James (Song Wei) Signed-off-by:James (Song Wei) diff -r baccadfd9418 tools/python/xen/xend/XendDomainInfo.py --- a/tools/python/xen/xend/XendDomainInfo.py Fri May 14 08:05:05 2010 +0100 +++ b/tools/python/xen/xend/XendDomainInfo.py Mon May 17 13:56:15 2010 +0800 @@ -530,7 +530,7 @@ self.storeDom("control/shutdown", reason) # HVM domain shuts itself down only if it has PV drivers - if self.info.is_hvm(): + if self.info.is_hvm() and reason != 'suspend': hvm_pvdrv = xc.hvm_get_param(self.domid, HVM_PARAM_CALLBACK_IRQ) hvm_s_state = xc.hvm_get_param(self.domid, HVM_PARAM_ACPI_S_STATE) if not hvm_pvdrv or hvm_s_state != 0: -- View this message in context: http://old.nabble.com/-PATCH--fix-save-or-snapshot-at-kernel-loading-time-with-HVM-vm-tp28579732p28579732.html Sent from the Xen - Dev mailing list archive at Nabble.com.