Re: [Qemu-devel] [PATCH 2/2 V7] qemu,qmp: add inject-nmi qmp command

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Apr 13, 2011 at 4:08 PM, Luiz Capitulino <lcapitulino@xxxxxxxxxx> wrote:
> On Tue, 12 Apr 2011 21:31:18 +0300
> Blue Swirl <blauwirbel@xxxxxxxxx> wrote:
>
>> On Tue, Apr 12, 2011 at 10:52 AM, Avi Kivity <avi@xxxxxxxxxx> wrote:
>> > On 04/11/2011 08:15 PM, Blue Swirl wrote:
>> >>
>> >> On Mon, Apr 11, 2011 at 10:01 AM, Markus Armbruster<armbru@xxxxxxxxxx>
>> >> Âwrote:
>> >> > ÂAvi Kivity<avi@xxxxxxxxxx> Âwrites:
>> >> >
>> >> >> ÂOn 04/08/2011 12:41 AM, Anthony Liguori wrote:
>> >> >>>
>> >> >>> Â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.
>> >> >>
>> >> >> Ânmi is a fine name for something that corresponds to a real-life nmi
>> >> >> Âbutton (often labeled "NMI").
>> >> >
>> >> > ÂAgree.
>> >>
>> >> We could also introduce an alias mechanism for user friendly names, so
>> >> nmi could be used in addition of full path. Aliases could be useful
>> >> for device paths as well.
>> >
>> > Yes. ÂPerhaps limited to the human monitor.
>>
>> I'd limit all debugging commands (including NMI) to the human monitor.
>
> Why?

Do they have any real use in production environment? Also, we should
have the freedom to change the debugging facilities (for example, to
improve some internal implementation) as we want without regard to
compatibility to previous versions.
--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [KVM ARM]     [KVM ia64]     [KVM ppc]     [Virtualization Tools]     [Spice Development]     [Libvirt]     [Libvirt Users]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite Questions]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux