From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:56525) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WsC90-0006WU-TA for qemu-devel@nongnu.org; Wed, 04 Jun 2014 10:26:31 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1WsC8s-0001SG-1W for qemu-devel@nongnu.org; Wed, 04 Jun 2014 10:26:22 -0400 Received: from e23smtp09.au.ibm.com ([202.81.31.142]:60467) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WsC8r-0001RS-A9 for qemu-devel@nongnu.org; Wed, 04 Jun 2014 10:26:13 -0400 Received: from /spool/local by e23smtp09.au.ibm.com with IBM ESMTP SMTP Gateway: Authorized Use Only! Violators will be prosecuted for from ; Thu, 5 Jun 2014 00:26:08 +1000 From: Alexey Kardashevskiy Date: Thu, 5 Jun 2014 00:25:58 +1000 Message-Id: <1401891961-17292-2-git-send-email-aik@ozlabs.ru> In-Reply-To: <1401891961-17292-1-git-send-email-aik@ozlabs.ru> References: <1401891961-17292-1-git-send-email-aik@ozlabs.ru> Subject: [Qemu-devel] [PATCH v4 1/4] cpus: Define callback for QEMU "nmi" command List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: Alexey Kardashevskiy , Peter Maydell , qemu-ppc@nongnu.org, Alexander Graf , Paolo Bonzini This introduces an NMI (non maskable interrupt) nmi_monitor_handler() callback to the CPU class. It is called from QMP's "nmi" command and performs an action required to cause debug crash dump on in-kernel debugger invocation. This adds support for it in qmp_inject_nmi(). Since no architecture supports it at the moment, there is no change in behaviour. This changes inject-nmi command description for HMP and QMP. Signed-off-by: Alexey Kardashevskiy --- Changes: v4: * s/\/nmi_monitor_handler/ v3: * actual nmi() enablement moved from last patch to first patch * changed description for QMP command too --- cpus.c | 11 ++++++++++- hmp-commands.hx | 6 ++---- include/qom/cpu.h | 1 + qapi-schema.json | 4 +--- qmp-commands.hx | 3 +-- 5 files changed, 15 insertions(+), 10 deletions(-) diff --git a/cpus.c b/cpus.c index dd7ac13..c0b8918 100644 --- a/cpus.c +++ b/cpus.c @@ -1495,6 +1495,15 @@ void qmp_inject_nmi(Error **errp) } } #else - error_set(errp, QERR_UNSUPPORTED); + CPUState *cs = qemu_get_cpu(monitor_get_cpu_index()); + CPUClass *cc = CPU_GET_CLASS(cs); + int ret = -1; + + if (cs && cc->nmi_monitor_handler) { + ret = cc->nmi_monitor_handler(cs); + } + if (ret) { + error_set(errp, QERR_UNSUPPORTED); + } #endif } diff --git a/hmp-commands.hx b/hmp-commands.hx index 2e462c0..e97b5ec 100644 --- a/hmp-commands.hx +++ b/hmp-commands.hx @@ -830,19 +830,17 @@ The values that can be specified here depend on the machine type, but are the same that can be specified in the @code{-boot} command line option. ETEXI -#if defined(TARGET_I386) || defined(TARGET_S390X) { .name = "nmi", .args_type = "", .params = "", - .help = "inject an NMI on all guest's CPUs", + .help = "inject an NMI on the given guest's CPU", .mhandler.cmd = hmp_inject_nmi, }, -#endif STEXI @item nmi @var{cpu} @findex nmi -Inject an NMI (x86) or RESTART (s390x) on the given CPU. +Inject an NMI on the given CPU. ETEXI diff --git a/include/qom/cpu.h b/include/qom/cpu.h index df977c8..d2547bc 100644 --- a/include/qom/cpu.h +++ b/include/qom/cpu.h @@ -108,6 +108,7 @@ typedef struct CPUClass { void (*parse_features)(CPUState *cpu, char *str, Error **errp); void (*reset)(CPUState *cpu); + int (*nmi_monitor_handler)(CPUState *cs); int reset_dump_flags; bool (*has_work)(CPUState *cpu); void (*do_interrupt)(CPUState *cpu); diff --git a/qapi-schema.json b/qapi-schema.json index 7bc33ea..dcf6642 100644 --- a/qapi-schema.json +++ b/qapi-schema.json @@ -1748,13 +1748,11 @@ ## # @inject-nmi: # -# Injects an Non-Maskable Interrupt into all guest's VCPUs. +# Injects an Non-Maskable Interrupt into the given guest's VCPU. # # Returns: If successful, nothing # # Since: 0.14.0 -# -# Notes: Only x86 Virtual Machines support this command. ## { 'command': 'inject-nmi' } diff --git a/qmp-commands.hx b/qmp-commands.hx index d8aa4ed..553375b 100644 --- a/qmp-commands.hx +++ b/qmp-commands.hx @@ -477,7 +477,7 @@ SQMP inject-nmi ---------- -Inject an NMI on guest's CPUs. +Inject an NMI on the given guest's CPU. Arguments: None. @@ -487,7 +487,6 @@ Example: <- { "return": {} } Note: inject-nmi fails when the guest doesn't support injecting. - Currently, only x86 (NMI) and s390x (RESTART) guests do. EQMP -- 2.0.0