From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756157Ab3ANHqk (ORCPT ); Mon, 14 Jan 2013 02:46:40 -0500 Received: from cn.fujitsu.com ([222.73.24.84]:10233 "EHLO song.cn.fujitsu.com" rhost-flags-OK-FAIL-OK-OK) by vger.kernel.org with ESMTP id S1752518Ab3ANHqj (ORCPT ); Mon, 14 Jan 2013 02:46:39 -0500 X-IronPort-AV: E=Sophos;i="4.84,466,1355068800"; d="scan'208";a="6581986" Date: Mon, 14 Jan 2013 15:46:35 +0800 From: Hu Tao To: Marcelo Tosatti Cc: kvm list , qemu-devel , "linux-kernel@vger.kernel.org" , Avi Kivity , "Daniel P. Berrange" , KAMEZAWA Hiroyuki , Jan Kiszka , Gleb Natapov , Blue Swirl , Eric Blake , Andrew Jones , Sasha Levin , Luiz Capitulino Subject: Re: [Qemu-devel] [PATCH v12 0/8] pv event to notify host when the guest is panicked Message-ID: <20130114074635.GF14958@localhost.localdomain> References: <20121220075359.GF7141@localhost.localdomain> <20121225215420.GB6222@amt.cnet> MIME-Version: 1.0 In-Reply-To: <20121225215420.GB6222@amt.cnet> User-Agent: Mutt/1.5.21 (2010-09-15) X-MIMETrack: Itemize by SMTP Server on mailserver/fnst(Release 8.5.3|September 15, 2011) at 2013/01/14 15:45:56, Serialize by Router on mailserver/fnst(Release 8.5.3|September 15, 2011) at 2013/01/14 15:45:56, Serialize complete at 2013/01/14 15:45:56 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Dec 25, 2012 at 07:54:20PM -0200, Marcelo Tosatti wrote: > On Thu, Dec 20, 2012 at 03:53:59PM +0800, Hu Tao wrote: > > Hi, > > > > Any comments? > > Did you verify possibilities listed at > https://lkml.org/lkml/2012/11/20/653 ? Except the EIO one you mentioned. I don't know how to reproduce it. > > If so, a summary in the patchset would be helpful. Thanks for reminding, I'll add it in the next version. Basically, the run state is saved and loaded again so we can know the state when restoring a vm. This is done by patch #1, and is independent of guest panic. Can you have a review of patch #1 before next version? > > > On Wed, Dec 12, 2012 at 02:13:43PM +0800, Hu Tao wrote: > > > This series implements a new interface, kvm pv event, to notify host when > > > some events happen in guest. Right now there is one supported event: guest > > > panic. > > > > > > changes from v11: > > > > > > - add a new patch 'save/load cpu runstate' > > > - fix a bug of null-dereference when no -machine option is supplied > > > - reserve RUN_STATE_GUEST_PANICKED during migration > > > - add doc of enable_pv_event option > > > - disable reboot-on-panic if pv_event is on > > > > > > v11: http://lists.gnu.org/archive/html/qemu-devel/2012-10/msg04361.html > > > > > > Hu Tao (7): > > > save/load cpu runstate > > > update kernel headers > > > add a new runstate: RUN_STATE_GUEST_PANICKED > > > add a new qevent: QEVENT_GUEST_PANICKED > > > introduce a new qom device to deal with panicked event > > > allower the user to disable pv event support > > > pv event: add document to describe the usage > > > > > > Wen Congyang (1): > > > start vm after resetting it > > > > > > block.h | 2 + > > > docs/pv-event.txt | 17 ++++ > > > hw/kvm/Makefile.objs | 2 +- > > > hw/kvm/pv_event.c | 197 +++++++++++++++++++++++++++++++++++++++ > > > hw/pc_piix.c | 11 +++ > > > kvm-stub.c | 4 + > > > kvm.h | 2 + > > > linux-headers/asm-x86/kvm_para.h | 1 + > > > linux-headers/linux/kvm_para.h | 6 ++ > > > migration.c | 7 +- > > > monitor.c | 6 +- > > > monitor.h | 1 + > > > qapi-schema.json | 6 +- > > > qemu-config.c | 4 + > > > qemu-options.hx | 3 +- > > > qmp.c | 5 +- > > > savevm.c | 1 + > > > sysemu.h | 2 + > > > vl.c | 52 ++++++++++- > > > 19 files changed, 312 insertions(+), 17 deletions(-) > > > create mode 100644 docs/pv-event.txt > > > create mode 100644 hw/kvm/pv_event.c > > > > > > -- > > > 1.8.0.1.240.ge8a1f5a > > -- > > To unsubscribe from this list: send the line "unsubscribe kvm" in > > the body of a message to majordomo@vger.kernel.org > > More majordomo info at http://vger.kernel.org/majordomo-info.html From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:45388) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TufYu-0001fS-Rl for qemu-devel@nongnu.org; Mon, 14 Jan 2013 03:38:38 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1TufYk-0000tn-BQ for qemu-devel@nongnu.org; Mon, 14 Jan 2013 03:38:32 -0500 Received: from [222.73.24.84] (port=44961 helo=song.cn.fujitsu.com) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Tuekh-0005Mf-IL for qemu-devel@nongnu.org; Mon, 14 Jan 2013 02:46:39 -0500 Date: Mon, 14 Jan 2013 15:46:35 +0800 From: Hu Tao Message-ID: <20130114074635.GF14958@localhost.localdomain> References: <20121220075359.GF7141@localhost.localdomain> <20121225215420.GB6222@amt.cnet> MIME-Version: 1.0 In-Reply-To: <20121225215420.GB6222@amt.cnet> Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Subject: Re: [Qemu-devel] [PATCH v12 0/8] pv event to notify host when the guest is panicked List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Marcelo Tosatti Cc: Andrew Jones , Gleb Natapov , kvm list , Jan Kiszka , qemu-devel , "linux-kernel@vger.kernel.org" , Blue Swirl , Avi Kivity , Luiz Capitulino , KAMEZAWA Hiroyuki , Sasha Levin On Tue, Dec 25, 2012 at 07:54:20PM -0200, Marcelo Tosatti wrote: > On Thu, Dec 20, 2012 at 03:53:59PM +0800, Hu Tao wrote: > > Hi, > > > > Any comments? > > Did you verify possibilities listed at > https://lkml.org/lkml/2012/11/20/653 ? Except the EIO one you mentioned. I don't know how to reproduce it. > > If so, a summary in the patchset would be helpful. Thanks for reminding, I'll add it in the next version. Basically, the run state is saved and loaded again so we can know the state when restoring a vm. This is done by patch #1, and is independent of guest panic. Can you have a review of patch #1 before next version? > > > On Wed, Dec 12, 2012 at 02:13:43PM +0800, Hu Tao wrote: > > > This series implements a new interface, kvm pv event, to notify host when > > > some events happen in guest. Right now there is one supported event: guest > > > panic. > > > > > > changes from v11: > > > > > > - add a new patch 'save/load cpu runstate' > > > - fix a bug of null-dereference when no -machine option is supplied > > > - reserve RUN_STATE_GUEST_PANICKED during migration > > > - add doc of enable_pv_event option > > > - disable reboot-on-panic if pv_event is on > > > > > > v11: http://lists.gnu.org/archive/html/qemu-devel/2012-10/msg04361.html > > > > > > Hu Tao (7): > > > save/load cpu runstate > > > update kernel headers > > > add a new runstate: RUN_STATE_GUEST_PANICKED > > > add a new qevent: QEVENT_GUEST_PANICKED > > > introduce a new qom device to deal with panicked event > > > allower the user to disable pv event support > > > pv event: add document to describe the usage > > > > > > Wen Congyang (1): > > > start vm after resetting it > > > > > > block.h | 2 + > > > docs/pv-event.txt | 17 ++++ > > > hw/kvm/Makefile.objs | 2 +- > > > hw/kvm/pv_event.c | 197 +++++++++++++++++++++++++++++++++++++++ > > > hw/pc_piix.c | 11 +++ > > > kvm-stub.c | 4 + > > > kvm.h | 2 + > > > linux-headers/asm-x86/kvm_para.h | 1 + > > > linux-headers/linux/kvm_para.h | 6 ++ > > > migration.c | 7 +- > > > monitor.c | 6 +- > > > monitor.h | 1 + > > > qapi-schema.json | 6 +- > > > qemu-config.c | 4 + > > > qemu-options.hx | 3 +- > > > qmp.c | 5 +- > > > savevm.c | 1 + > > > sysemu.h | 2 + > > > vl.c | 52 ++++++++++- > > > 19 files changed, 312 insertions(+), 17 deletions(-) > > > create mode 100644 docs/pv-event.txt > > > create mode 100644 hw/kvm/pv_event.c > > > > > > -- > > > 1.8.0.1.240.ge8a1f5a > > -- > > To unsubscribe from this list: send the line "unsubscribe kvm" in > > the body of a message to majordomo@vger.kernel.org > > More majordomo info at http://vger.kernel.org/majordomo-info.html