[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <4EDF3E2A.3060208@redhat.com>
Date: Wed, 07 Dec 2011 12:21:30 +0200
From: Avi Kivity <avi@...hat.com>
To: Sasha Levin <levinsasha928@...il.com>
CC: linux-kernel@...r.kernel.org,
Marcelo Tosatti <mtosatti@...hat.com>, kvm@...r.kernel.org
Subject: Re: [PATCH] KVM: Document KVM_NMI
On 12/06/2011 10:50 PM, Sasha Levin wrote:
> Cc: Avi Kivity <avi@...hat.com>
> Cc: Marcelo Tosatti <mtosatti@...hat.com>
> Cc: kvm@...r.kernel.org
Somehow, the changelog became the place where people manage their cc lists.
>
> +4.64 KVM_NMI
> +
> +Capability: basic
> +Architectures: none
> +Type: vcpu ioctl
> +Parameters: none
> +Returns: 0 on success, -1 on error
> +
> +Trigger an NMI on the specified VCPU.
> +
> +This can be used to cause a panic in the guest, which in turn would result
> +in a backtrace/dump.
> +
I'm nitpicking, but this is too strongly worded. It's completely up to
the guest how to interpret the NMI.
It's also subtly broken with kernel irqchip - it injects an NMI
unconditionally even though LINT1 can be configured to mask NMIs, or to
do something other than queue an NMI (for example, queue a regular
interrupt). Please note this.
On second thoughts, I'll just document this instead of telling you what
to write.
--
error compiling committee.c: too many arguments to function
--
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/
Powered by blists - more mailing lists