Re: [Qemu-devel] [PATCH 0/2 v3] kvm: notify host when guest panicked

2012-03-22 Thread Gleb Natapov
On Thu, Mar 22, 2012 at 03:44:45PM +0800, Wen Congyang wrote: > At 03/22/2012 03:31 PM, Gleb Natapov Wrote: > > On Thu, Mar 22, 2012 at 09:05:12AM +0800, Wen Congyang wrote: > >> At 03/22/2012 03:19 AM, Anthony Liguori Wrote: > >>> On 03/21/2012 11:25 AM, Avi Kivity wrote: > On 03/21/2012 06:1

Re: [Qemu-devel] [PATCH 0/2 v3] kvm: notify host when guest panicked

2012-03-22 Thread Wen Congyang
At 03/22/2012 03:31 PM, Gleb Natapov Wrote: > On Thu, Mar 22, 2012 at 09:05:12AM +0800, Wen Congyang wrote: >> At 03/22/2012 03:19 AM, Anthony Liguori Wrote: >>> On 03/21/2012 11:25 AM, Avi Kivity wrote: On 03/21/2012 06:18 PM, Corey Minyard wrote: > >> Look at drivers/char/ipmi/ipmi_m

Re: [Qemu-devel] [PATCH 0/2 v3] kvm: notify host when guest panicked

2012-03-22 Thread Wen Congyang
At 03/22/2012 03:12 AM, Anthony Liguori Wrote: > On 03/15/2012 06:46 AM, Avi Kivity wrote: >> On 03/15/2012 01:25 PM, Jan Kiszka wrote: > There was such vm exit (KVM_EXIT_HYPERCALL), but it was deemed to be a bad idea. >>> >>> BTW, this would help a lot in emulating hypercalls of othe

Re: [Qemu-devel] [PATCH 0/2 v3] kvm: notify host when guest panicked

2012-03-22 Thread Wen Congyang
At 03/22/2012 03:28 PM, Gleb Natapov Wrote: > On Wed, Mar 21, 2012 at 02:19:34PM -0500, Anthony Liguori wrote: >> On 03/21/2012 11:25 AM, Avi Kivity wrote: >>> On 03/21/2012 06:18 PM, Corey Minyard wrote: > Look at drivers/char/ipmi/ipmi_msghandler.c. It has code to send panic > event

Re: [Qemu-devel] [PATCH 0/2 v3] kvm: notify host when guest panicked

2012-03-22 Thread Gleb Natapov
On Wed, Mar 21, 2012 at 02:04:34PM -0500, Anthony Liguori wrote: > On 03/13/2012 05:47 AM, Avi Kivity wrote: > >On 03/13/2012 11:18 AM, Daniel P. Berrange wrote: > >>On Mon, Mar 12, 2012 at 12:33:33PM +0200, Avi Kivity wrote: > >>>On 03/12/2012 11:04 AM, Wen Congyang wrote: > Do you have any ot

Re: [Qemu-devel] [PATCH 0/2 v3] kvm: notify host when guest panicked

2012-03-22 Thread Gleb Natapov
On Thu, Mar 22, 2012 at 09:05:12AM +0800, Wen Congyang wrote: > At 03/22/2012 03:19 AM, Anthony Liguori Wrote: > > On 03/21/2012 11:25 AM, Avi Kivity wrote: > >> On 03/21/2012 06:18 PM, Corey Minyard wrote: > >>> > Look at drivers/char/ipmi/ipmi_msghandler.c. It has code to send panic > e

Re: [Qemu-devel] [PATCH 0/2 v3] kvm: notify host when guest panicked

2012-03-22 Thread Gleb Natapov
On Wed, Mar 21, 2012 at 02:19:34PM -0500, Anthony Liguori wrote: > On 03/21/2012 11:25 AM, Avi Kivity wrote: > >On 03/21/2012 06:18 PM, Corey Minyard wrote: > >> > >>>Look at drivers/char/ipmi/ipmi_msghandler.c. It has code to send panic > >>>event over IMPI. The code is pretty complex. Of course i

Re: [Qemu-devel] [PATCH 0/2 v3] kvm: notify host when guest panicked

2012-03-21 Thread Wen Congyang
At 03/22/2012 03:19 AM, Anthony Liguori Wrote: > On 03/21/2012 11:25 AM, Avi Kivity wrote: >> On 03/21/2012 06:18 PM, Corey Minyard wrote: >>> Look at drivers/char/ipmi/ipmi_msghandler.c. It has code to send panic event over IMPI. The code is pretty complex. Of course if we a going to >>>

Re: [Qemu-devel] [PATCH 0/2 v3] kvm: notify host when guest panicked

2012-03-21 Thread Anthony Liguori
On 03/21/2012 11:25 AM, Avi Kivity wrote: On 03/21/2012 06:18 PM, Corey Minyard wrote: Look at drivers/char/ipmi/ipmi_msghandler.c. It has code to send panic event over IMPI. The code is pretty complex. Of course if we a going to implement something more complex than simple hypercall for panic

Re: [Qemu-devel] [PATCH 0/2 v3] kvm: notify host when guest panicked

2012-03-21 Thread Anthony Liguori
On 03/15/2012 06:46 AM, Avi Kivity wrote: On 03/15/2012 01:25 PM, Jan Kiszka wrote: There was such vm exit (KVM_EXIT_HYPERCALL), but it was deemed to be a bad idea. BTW, this would help a lot in emulating hypercalls of other hypervisors (or of KVM's VAPIC in the absence of in-kernel irqchip

Re: [Qemu-devel] [PATCH 0/2 v3] kvm: notify host when guest panicked

2012-03-21 Thread Anthony Liguori
On 03/13/2012 05:47 AM, Avi Kivity wrote: On 03/13/2012 11:18 AM, Daniel P. Berrange wrote: On Mon, Mar 12, 2012 at 12:33:33PM +0200, Avi Kivity wrote: On 03/12/2012 11:04 AM, Wen Congyang wrote: Do you have any other comments about this patch? Not really, but I'm not 100% convinced the pat