All of lore.kernel.org
 help / color / mirror / Atom feed
* [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.