From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754271Ab2LYVwk (ORCPT ); Tue, 25 Dec 2012 16:52:40 -0500 Received: from mx1.redhat.com ([209.132.183.28]:43546 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754213Ab2LYVwh (ORCPT ); Tue, 25 Dec 2012 16:52:37 -0500 Date: Tue, 25 Dec 2012 19:52:05 -0200 From: Marcelo Tosatti To: Hu Tao 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: <20121225215205.GA6222@amt.cnet> References: <20121220075359.GF7141@localhost.localdomain> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20121220075359.GF7141@localhost.localdomain> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, Dec 20, 2012 at 03:53:59PM +0800, Hu Tao wrote: > Hi, > > Any comments? As far as i can see, items 2 and 3 of https://lkml.org/lkml/2012/11/12/588 Have not been addressed. https://lkml.org/lkml/2012/11/20/653 contains discussions on those items. 2) Format of the interface for other architectures (you can choose a different KVM supported architecture and write an example). It was your choice to choose an I/O port, which is x86 specific. 3) Clear/documented management interface for the feature. Note 3 is for management, not the guest<->host interface. > 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 From: Marcelo Tosatti Subject: Re: [PATCH v12 0/8] pv event to notify host when the guest is panicked Date: Tue, 25 Dec 2012 19:52:05 -0200 Message-ID: <20121225215205.GA6222@amt.cnet> References: <20121220075359.GF7141@localhost.localdomain> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii 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 To: Hu Tao Return-path: Content-Disposition: inline In-Reply-To: <20121220075359.GF7141@localhost.localdomain> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: qemu-devel-bounces+gceq-qemu-devel=gmane.org@nongnu.org Sender: qemu-devel-bounces+gceq-qemu-devel=gmane.org@nongnu.org List-Id: kvm.vger.kernel.org On Thu, Dec 20, 2012 at 03:53:59PM +0800, Hu Tao wrote: > Hi, > > Any comments? As far as i can see, items 2 and 3 of https://lkml.org/lkml/2012/11/12/588 Have not been addressed. https://lkml.org/lkml/2012/11/20/653 contains discussions on those items. 2) Format of the interface for other architectures (you can choose a different KVM supported architecture and write an example). It was your choice to choose an I/O port, which is x86 specific. 3) Clear/documented management interface for the feature. Note 3 is for management, not the guest<->host interface. > 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]:52150) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TncQW-0003Cc-NM for qemu-devel@nongnu.org; Tue, 25 Dec 2012 16:52:50 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1TncQN-0001OV-E9 for qemu-devel@nongnu.org; Tue, 25 Dec 2012 16:52:44 -0500 Received: from mx1.redhat.com ([209.132.183.28]:11612) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TncQN-0001OR-67 for qemu-devel@nongnu.org; Tue, 25 Dec 2012 16:52:35 -0500 Date: Tue, 25 Dec 2012 19:52:05 -0200 From: Marcelo Tosatti Message-ID: <20121225215205.GA6222@amt.cnet> References: <20121220075359.GF7141@localhost.localdomain> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20121220075359.GF7141@localhost.localdomain> 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: Hu Tao 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 Thu, Dec 20, 2012 at 03:53:59PM +0800, Hu Tao wrote: > Hi, > > Any comments? As far as i can see, items 2 and 3 of https://lkml.org/lkml/2012/11/12/588 Have not been addressed. https://lkml.org/lkml/2012/11/20/653 contains discussions on those items. 2) Format of the interface for other architectures (you can choose a different KVM supported architecture and write an example). It was your choice to choose an I/O port, which is x86 specific. 3) Clear/documented management interface for the feature. Note 3 is for management, not the guest<->host interface. > 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