Re: [Qemu-devel] [PATCHv3] qxl: abort on panic instead of exit

2011-06-29 Thread Gerd Hoffmann
Hi, Agreed. So keep PANIC_ON but remove a lot of it's users, or add some wrapper that depends on a "guest_panic" parameter that defaults off. Maybe keep PANIC_ON in case there are any which can trigger on qemu bugs only. Add a new one where the name makes clear this triggers on case the g

Re: [Qemu-devel] [PATCHv3] qxl: abort on panic instead of exit

2011-06-29 Thread Alon Levy
On Wed, Jun 29, 2011 at 02:44:11PM +0200, Gerd Hoffmann wrote: > On 06/29/11 13:57, Alon Levy wrote: > >--- > > hw/qxl.h |2 +- > > 1 files changed, 1 insertions(+), 1 deletions(-) > > > >diff --git a/hw/qxl.h b/hw/qxl.h > >index 7df594e..bf875a0 100644 > >--- a/hw/qxl.h > >+++ b/hw/qxl.h > >@

Re: [Qemu-devel] [PATCHv3] qxl: abort on panic instead of exit

2011-06-29 Thread Gerd Hoffmann
On 06/29/11 13:57, Alon Levy wrote: --- hw/qxl.h |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/hw/qxl.h b/hw/qxl.h index 7df594e..bf875a0 100644 --- a/hw/qxl.h +++ b/hw/qxl.h @@ -89,7 +89,7 @@ typedef struct PCIQXLDevice { #define PANIC_ON(x) if ((x)) {

[Qemu-devel] [PATCHv3] qxl: abort on panic instead of exit

2011-06-29 Thread Alon Levy
--- hw/qxl.h |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/hw/qxl.h b/hw/qxl.h index 7df594e..bf875a0 100644 --- a/hw/qxl.h +++ b/hw/qxl.h @@ -89,7 +89,7 @@ typedef struct PCIQXLDevice { #define PANIC_ON(x) if ((x)) { \ printf("%s: PANIC