[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 2/2 V7] qemu,qmp: add inject-nmi qmp command
From: |
Gleb Natapov |
Subject: |
Re: [Qemu-devel] [PATCH 2/2 V7] qemu,qmp: add inject-nmi qmp command |
Date: |
Fri, 8 Apr 2011 09:04:57 +0300 |
On Thu, Apr 07, 2011 at 04:41:03PM -0500, Anthony Liguori wrote:
> On 04/07/2011 02:17 PM, Gleb Natapov wrote:
> >On Thu, Apr 07, 2011 at 10:04:00PM +0300, Blue Swirl wrote:
> >>On Thu, Apr 7, 2011 at 9:51 PM, Gleb Natapov<address@hidden> wrote:
> >>
> >>I'd prefer something more generic like these:
> >>raise /address@hidden:l1int
> >>lower /i44FX-pcihost/address@hidden/pinD
> >>
> >>The clumsier syntax shouldn't be a problem, since this would be a
> >>system developer tool.
> >>
> >>Some kind of IRQ registration would be needed for this to work without
> >>lots of changes.
> >True. The ability to trigger any interrupt line is very useful for
> >debugging. I often re-implement it during debug.
>
> And it's a good thing to have, but exposing this as the only API to
> do something as simple as generating a guest crash dump is not the
> friendliest thing in the world to do to users.
>
Well, this is not intended to be used by regular users directly and
management can provide nicer interface for issuing NMI. But really,
my point is that NMI actually generates guest core dump in such rare
cases (only preconfigured Windows guests) that it doesn't warrant to
name command as such. Management is in much better position to implement
functionality with such name since it knows what type of guest it runs
and can tell agent to configure guest accordingly.
--
Gleb.
- Re: [Qemu-devel] [PATCH 2/2 V7] qemu,qmp: add inject-nmi qmp command, (continued)
- Re: [Qemu-devel] [PATCH 2/2 V7] qemu,qmp: add inject-nmi qmp command, Luiz Capitulino, 2011/04/06
- Re: [Qemu-devel] [PATCH 2/2 V7] qemu,qmp: add inject-nmi qmp command, Peter Maydell, 2011/04/06
- Re: [Qemu-devel] [PATCH 2/2 V7] qemu,qmp: add inject-nmi qmp command, Anthony Liguori, 2011/04/06
- Re: [Qemu-devel] [PATCH 2/2 V7] qemu,qmp: add inject-nmi qmp command, Jan Kiszka, 2011/04/07
- Re: [Qemu-devel] [PATCH 2/2 V7] qemu,qmp: add inject-nmi qmp command, Peter Maydell, 2011/04/07
- Re: [Qemu-devel] [PATCH 2/2 V7] qemu,qmp: add inject-nmi qmp command, Anthony Liguori, 2011/04/07
- Re: [Qemu-devel] [PATCH 2/2 V7] qemu,qmp: add inject-nmi qmp command, Gleb Natapov, 2011/04/07
- Re: [Qemu-devel] [PATCH 2/2 V7] qemu,qmp: add inject-nmi qmp command, Blue Swirl, 2011/04/07
- Re: [Qemu-devel] [PATCH 2/2 V7] qemu,qmp: add inject-nmi qmp command, Gleb Natapov, 2011/04/07
- Re: [Qemu-devel] [PATCH 2/2 V7] qemu,qmp: add inject-nmi qmp command, Anthony Liguori, 2011/04/07
- Re: [Qemu-devel] [PATCH 2/2 V7] qemu,qmp: add inject-nmi qmp command,
Gleb Natapov <=
- Re: [Qemu-devel] [PATCH 2/2 V7] qemu,qmp: add inject-nmi qmp command, Blue Swirl, 2011/04/08
- Re: [Qemu-devel] [PATCH 2/2 V7] qemu,qmp: add inject-nmi qmp command, Anthony Liguori, 2011/04/08
- Re: [Qemu-devel] [PATCH 2/2 V7] qemu,qmp: add inject-nmi qmp command, Blue Swirl, 2011/04/08
- Re: [Qemu-devel] [PATCH 2/2 V7] qemu,qmp: add inject-nmi qmp command, Avi Kivity, 2011/04/10
- Re: [Qemu-devel] [PATCH 2/2 V7] qemu,qmp: add inject-nmi qmp command, Markus Armbruster, 2011/04/11
- Re: [Qemu-devel] [PATCH 2/2 V7] qemu, qmp: add inject-nmi qmp command, Blue Swirl, 2011/04/11
- Re: [Qemu-devel] [PATCH 2/2 V7] qemu, qmp: add inject-nmi qmp command, Avi Kivity, 2011/04/12
- Re: [Qemu-devel] [PATCH 2/2 V7] qemu, qmp: add inject-nmi qmp command, Blue Swirl, 2011/04/12
- Re: [Qemu-devel] [PATCH 2/2 V7] qemu, qmp: add inject-nmi qmp command, Luiz Capitulino, 2011/04/13
- Re: [Qemu-devel] [PATCH 2/2 V7] qemu, qmp: add inject-nmi qmp command, Blue Swirl, 2011/04/13