KVM: nVMX: add tracepoint for vmwrite

Commit Message

Paolo Bonzini Nov. 4, 2016, 11:18 p.m. UTC

Both the field and the value are in registers, so they are not clear from
existing tracepoints.

Cc: Ladi Prosek <[e-mail]>
Signed-off-by: Paolo Bonzini <[e-mail]>
---
 arch/x86/kvm/trace.h | 23 +++++++++++++++++++++++
 arch/x86/kvm/vmx.c   |  2 ++
 2 files changed, 25 insertions(+)

Comments

kernel test robot Nov. 5, 2016, 10:48 a.m. UTC | #1

Hi Paolo,

[auto build test ERROR on kvm/linux-next]
[also build test ERROR on v4.9-rc3 next-20161028]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]

url:    https://github.com/0day-ci/linux/commits/Paolo-Bonzini/KVM-nVMX-add-tracepoint-for-vmwrite/20161105-081951
base:   https://git.kernel.org/pub/scm/virt/kvm/kvm.git linux-next
config: x86_64-rhel-7.2 (attached as .config)
compiler: gcc-6 (Debian 6.2.0-3) 6.2.0 20160901
reproduce:
        # save the attached .config to linux build tree
        make ARCH=x86_64 

All errors (new ones prefixed by >>):

>> ERROR: "__tracepoint_kvm_vmwrite" [arch/x86/kvm/kvm-intel.ko] undefined!

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

Patch

diff --git a/arch/x86/kvm/trace.h b/arch/x86/kvm/trace.h
index 0a6cc6754ec5..b3d4dec78ae2 100644
--- a/arch/x86/kvm/trace.h
+++ b/arch/x86/kvm/trace.h
@@ -723,6 +723,29 @@  TRACE_EVENT(kvm_skinit,
 	flags;						\
 	})
 
+/*
+ * Tracepoint for nested #vmexit because of vmwrite
+ */
+TRACE_EVENT(kvm_vmwrite,
+	    TP_PROTO(__u64 vmptr, __u16 field, __u64 value),
+	    TP_ARGS(vmptr, field, value),
+
+	TP_STRUCT__entry(
+		__field(	__u64,	vmptr	)
+		__field(	__u16,	field	)
+		__field(	__u64,	value	)
+	),
+
+	TP_fast_assign(
+		__entry->vmptr		=	vmptr;
+		__entry->field		=	field;
+		__entry->value		=	value;
+	),
+
+	TP_printk("vmcs: 0x%016llx field: 0x%04x value: 0x%llx",
+		  __entry->vmptr, __entry->field, __entry->value)
+);
+
 TRACE_EVENT(kvm_emulate_insn,
 	TP_PROTO(struct kvm_vcpu *vcpu, __u8 failed),
 	TP_ARGS(vcpu, failed),
diff --git a/arch/x86/kvm/vmx.c b/arch/x86/kvm/vmx.c
index 5382b82462fc..e7d01d7d350b 100644
--- a/arch/x86/kvm/vmx.c
+++ b/arch/x86/kvm/vmx.c
@@ -7474,6 +7474,7 @@  static int handle_vmwrite(struct kvm_vcpu *vcpu)
 {
 	unsigned long field;
 	gva_t gva;
+	struct vcpu_vmx *vmx = to_vmx(vcpu);
 	unsigned long exit_qualification = vmcs_readl(EXIT_QUALIFICATION);
 	u32 vmx_instruction_info = vmcs_read32(VMX_INSTRUCTION_INFO);
 	/* The value to write might be 32 or 64 bits, depending on L1's long
@@ -7512,6 +7513,7 @@  static int handle_vmwrite(struct kvm_vcpu *vcpu)
 		return 1;
 	}
 
+	trace_kvm_vmwrite(vmx->nested.current_vmptr, field, field_value);
 	if (vmcs12_write_any(vcpu, field, field_value) < 0) {
 		nested_vmx_failValid(vcpu, VMXERR_UNSUPPORTED_VMCS_COMPONENT);
 		skip_emulated_instruction(vcpu);