From: Hu Tao <hutao@cn.fujitsu.com> To: Marcelo Tosatti <mtosatti@redhat.com> Cc: kvm list <kvm@vger.kernel.org>, qemu-devel <qemu-devel@nongnu.org>, "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>, Avi Kivity <avi@redhat.com>, "Daniel P. Berrange" <berrange@redhat.com>, KAMEZAWA Hiroyuki <kamezawa.hiroyu@jp.fujitsu.com>, Jan Kiszka <jan.kiszka@siemens.com>, Gleb Natapov <gleb@redhat.com>, Blue Swirl <blauwirbel@gmail.com>, Eric Blake <eblake@redhat.com>, Andrew Jones <drjones@redhat.com>, Sasha Levin <levinsasha928@gmail.com>, Luiz Capitulino <lcapitulino@redhat.com> Subject: Re: [Qemu-devel] [PATCH v12 0/8] pv event to notify host when the guest is panicked Date: Mon, 14 Jan 2013 15:46:35 +0800 [thread overview] Message-ID: <20130114074635.GF14958@localhost.localdomain> (raw) In-Reply-To: <20121225215420.GB6222@amt.cnet> 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
WARNING: multiple messages have this Message-ID (diff)
From: Hu Tao <hutao@cn.fujitsu.com> To: Marcelo Tosatti <mtosatti@redhat.com> Cc: Andrew Jones <drjones@redhat.com>, Gleb Natapov <gleb@redhat.com>, kvm list <kvm@vger.kernel.org>, Jan Kiszka <jan.kiszka@siemens.com>, qemu-devel <qemu-devel@nongnu.org>, "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>, Blue Swirl <blauwirbel@gmail.com>, Avi Kivity <avi@redhat.com>, Luiz Capitulino <lcapitulino@redhat.com>, KAMEZAWA Hiroyuki <kamezawa.hiroyu@jp.fujitsu.com>, Sasha Levin <levinsasha928@gmail.com> Subject: Re: [Qemu-devel] [PATCH v12 0/8] pv event to notify host when the guest is panicked Date: Mon, 14 Jan 2013 15:46:35 +0800 [thread overview] Message-ID: <20130114074635.GF14958@localhost.localdomain> (raw) In-Reply-To: <20121225215420.GB6222@amt.cnet> 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
next prev parent reply other threads:[~2013-01-14 7:46 UTC|newest] Thread overview: 32+ messages / expand[flat|nested] mbox.gz Atom feed top 2012-12-12 6:13 [PATCH v12 0/8] pv event to notify host when the guest is panicked Hu Tao 2012-12-12 6:13 ` [Qemu-devel] " Hu Tao 2012-12-12 6:13 ` [PATCH v12 1/8] save/load cpu runstate Hu Tao 2012-12-12 6:13 ` [Qemu-devel] " Hu Tao 2012-12-12 6:13 ` [PATCH v12 2/8] start vm after resetting it Hu Tao 2012-12-12 6:13 ` [Qemu-devel] " Hu Tao 2012-12-12 6:13 ` [PATCH v12 3/8] update kernel headers Hu Tao 2012-12-12 6:13 ` [Qemu-devel] " Hu Tao 2012-12-12 6:13 ` [PATCH v12 4/8] add a new runstate: RUN_STATE_GUEST_PANICKED Hu Tao 2012-12-12 6:13 ` [Qemu-devel] " Hu Tao 2012-12-12 6:13 ` [PATCH v12 5/8] add a new qevent: QEVENT_GUEST_PANICKED Hu Tao 2012-12-12 6:13 ` [Qemu-devel] " Hu Tao 2012-12-12 6:13 ` [PATCH v12 6/8] introduce a new qom device to deal with panicked event Hu Tao 2012-12-12 6:13 ` [Qemu-devel] " Hu Tao 2012-12-12 6:13 ` Hu Tao 2012-12-12 6:13 ` [PATCH v12 7/8] allower the user to disable pv event support Hu Tao 2012-12-12 6:13 ` [Qemu-devel] " Hu Tao 2012-12-12 6:13 ` [PATCH v12 8/8] pv event: add document to describe the usage Hu Tao 2012-12-12 6:13 ` [Qemu-devel] " Hu Tao 2012-12-12 6:15 ` [PATCH] kvm: notify host when the guest is panicked Hu Tao 2012-12-12 6:15 ` [Qemu-devel] " Hu Tao 2012-12-20 7:53 ` [Qemu-devel] [PATCH v12 0/8] pv event to " Hu Tao 2012-12-25 21:52 ` Marcelo Tosatti 2012-12-25 21:52 ` Marcelo Tosatti 2012-12-25 21:52 ` Marcelo Tosatti 2013-01-14 7:38 ` [Qemu-devel] " Hu Tao 2013-01-14 7:38 ` Hu Tao 2012-12-25 21:54 ` Marcelo Tosatti 2012-12-25 21:54 ` Marcelo Tosatti 2012-12-25 21:54 ` Marcelo Tosatti 2013-01-14 7:46 ` Hu Tao [this message] 2013-01-14 7:46 ` [Qemu-devel] " Hu Tao
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20130114074635.GF14958@localhost.localdomain \ --to=hutao@cn.fujitsu.com \ --cc=avi@redhat.com \ --cc=berrange@redhat.com \ --cc=blauwirbel@gmail.com \ --cc=drjones@redhat.com \ --cc=eblake@redhat.com \ --cc=gleb@redhat.com \ --cc=jan.kiszka@siemens.com \ --cc=kamezawa.hiroyu@jp.fujitsu.com \ --cc=kvm@vger.kernel.org \ --cc=lcapitulino@redhat.com \ --cc=levinsasha928@gmail.com \ --cc=linux-kernel@vger.kernel.org \ --cc=mtosatti@redhat.com \ --cc=qemu-devel@nongnu.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.