* [PATCH] x86/VT-x: Dump VMCS on VMLAUNCH/VMRESUME failure
@ 2016-07-01 17:52 Andrew Cooper
2016-07-04 6:25 ` Tian, Kevin
2016-07-04 8:24 ` Jan Beulich
0 siblings, 2 replies; 3+ messages in thread
From: Andrew Cooper @ 2016-07-01 17:52 UTC (permalink / raw)
To: Xen-devel; +Cc: Andrew Cooper, Kevin Tian, Jun Nakajima, Jan Beulich
If a VMLAUNCH/VMRESUME fails due to invalid control or host state, dump the
VMCS before crashing the domain.
Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com>
---
CC: Jan Beulich <JBeulich@suse.com>
CC: Jun Nakajima <jun.nakajima@intel.com>
CC: Kevin Tian <kevin.tian@intel.com>
---
xen/arch/x86/hvm/vmx/vmcs.c | 5 +++++
xen/include/asm-x86/hvm/vmx/vmcs.h | 4 ++++
2 files changed, 9 insertions(+)
diff --git a/xen/arch/x86/hvm/vmx/vmcs.c b/xen/arch/x86/hvm/vmx/vmcs.c
index 46b63b6..1bd875a 100644
--- a/xen/arch/x86/hvm/vmx/vmcs.c
+++ b/xen/arch/x86/hvm/vmx/vmcs.c
@@ -1637,6 +1637,11 @@ void vmx_vmentry_failure(void)
__vmread(VM_INSTRUCTION_ERROR, &error);
gprintk(XENLOG_ERR, "VM%s error: %#lx\n",
curr->arch.hvm_vmx.launched ? "RESUME" : "LAUNCH", error);
+
+ if ( error == VMX_INSN_INVALID_CONTROL_STATE ||
+ error == VMX_INSN_INVALID_HOST_STATE )
+ vmcs_dump_vcpu(curr);
+
domain_crash_synchronous();
}
diff --git a/xen/include/asm-x86/hvm/vmx/vmcs.h b/xen/include/asm-x86/hvm/vmx/vmcs.h
index 9f1729d..1e33d9c 100644
--- a/xen/include/asm-x86/hvm/vmx/vmcs.h
+++ b/xen/include/asm-x86/hvm/vmx/vmcs.h
@@ -567,6 +567,10 @@ enum vmcs_field {
#define VMX_GUEST_MSR 0
#define VMX_HOST_MSR 1
+/* VM Instruction error numbers. */
+#define VMX_INSN_INVALID_CONTROL_STATE 7
+#define VMX_INSN_INVALID_HOST_STATE 8
+
void vmx_disable_intercept_for_msr(struct vcpu *v, u32 msr, int type);
void vmx_enable_intercept_for_msr(struct vcpu *v, u32 msr, int type);
int vmx_read_guest_msr(u32 msr, u64 *val);
--
2.1.4
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] x86/VT-x: Dump VMCS on VMLAUNCH/VMRESUME failure
2016-07-01 17:52 [PATCH] x86/VT-x: Dump VMCS on VMLAUNCH/VMRESUME failure Andrew Cooper
@ 2016-07-04 6:25 ` Tian, Kevin
2016-07-04 8:24 ` Jan Beulich
1 sibling, 0 replies; 3+ messages in thread
From: Tian, Kevin @ 2016-07-04 6:25 UTC (permalink / raw)
To: Andrew Cooper, Xen-devel; +Cc: Nakajima, Jun, Jan Beulich
> From: Andrew Cooper [mailto:andrew.cooper3@citrix.com]
> Sent: Saturday, July 02, 2016 1:53 AM
>
> If a VMLAUNCH/VMRESUME fails due to invalid control or host state, dump the
> VMCS before crashing the domain.
>
> Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com>
Acked-by: Kevin Tian <kevin.tian@intel.com>
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] x86/VT-x: Dump VMCS on VMLAUNCH/VMRESUME failure
2016-07-01 17:52 [PATCH] x86/VT-x: Dump VMCS on VMLAUNCH/VMRESUME failure Andrew Cooper
2016-07-04 6:25 ` Tian, Kevin
@ 2016-07-04 8:24 ` Jan Beulich
1 sibling, 0 replies; 3+ messages in thread
From: Jan Beulich @ 2016-07-04 8:24 UTC (permalink / raw)
To: Andrew Cooper; +Cc: Kevin Tian, Jun Nakajima, Xen-devel
>>> On 01.07.16 at 19:52, <andrew.cooper3@citrix.com> wrote:
> If a VMLAUNCH/VMRESUME fails due to invalid control or host state, dump the
> VMCS before crashing the domain.
>
> Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com>
Good idea:
Reviewed-by: Jan Beulich <jbeulich@suse.com>
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2016-07-04 8:24 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-07-01 17:52 [PATCH] x86/VT-x: Dump VMCS on VMLAUNCH/VMRESUME failure Andrew Cooper
2016-07-04 6:25 ` Tian, Kevin
2016-07-04 8:24 ` Jan Beulich
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.