linux-kernel - Re: [Qemu-devel] [PATCH v12 0/8] pv event to notify host when the guest is panicked

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20121225215205.GA6222@amt.cnet>
Date:	Tue, 25 Dec 2012 19:52:05 -0200
From:	Marcelo Tosatti <mtosatti@...hat.com>
To:	Hu Tao <hutao@...fujitsu.com>
Cc:	kvm list <kvm@...r.kernel.org>, qemu-devel <qemu-devel@...gnu.org>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	Avi Kivity <avi@...hat.com>,
	"Daniel P. Berrange" <berrange@...hat.com>,
	KAMEZAWA Hiroyuki <kamezawa.hiroyu@...fujitsu.com>,
	Jan Kiszka <jan.kiszka@...mens.com>,
	Gleb Natapov <gleb@...hat.com>,
	Blue Swirl <blauwirbel@...il.com>,
	Eric Blake <eblake@...hat.com>,
	Andrew Jones <drjones@...hat.com>,
	Sasha Levin <levinsasha928@...il.com>,
	Luiz Capitulino <lcapitulino@...hat.com>
Subject: Re: [Qemu-devel] [PATCH v12 0/8] pv event to notify host when the
 guest is panicked

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@...r.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/