Mirror of the lttng-dev mailing list
 help / color / mirror / Atom feed
* [ltt-dev] [LTTNG-MODULES PATCH] Connect probes to KVM TRACE_EVENTs
@ 2010-09-10 11:40 Julien Desfossez
  2010-09-11 14:57 ` Mathieu Desnoyers
  0 siblings, 1 reply; 5+ messages in thread
From: Julien Desfossez @ 2010-09-10 11:40 UTC (permalink / raw)


Signed-off-by: Julien Desfossez <julien.desfossez at polymtl.ca>
---
 probes/Makefile    |    6 ++
 probes/kvm-trace.c |  211 ++++++++++++++++++++++++++++++++++++++++++++++++++++
 2 files changed, 217 insertions(+), 0 deletions(-)
 create mode 100644 probes/kvm-trace.c

diff --git a/probes/Makefile b/probes/Makefile
index d8f1c40..bb3aca7 100644
--- a/probes/Makefile
+++ b/probes/Makefile
@@ -44,4 +44,10 @@ endif
 obj-m += block-trace.o
 endif
 
+ifdef CONFIG_KVM
+ifdef CONFIG_FTRACE
+CFLAGS_REMOVE_kvm-trace.o = -pg
+endif
+obj-m += kvm-trace.o
+endif
 
diff --git a/probes/kvm-trace.c b/probes/kvm-trace.c
new file mode 100644
index 0000000..f8144fa
--- /dev/null
+++ b/probes/kvm-trace.c
@@ -0,0 +1,211 @@
+/*
+ * ltt/probes/kvm-trace.c
+ *
+ * KVM tracepoint probes.
+ *
+ * (C) Copyright 2010 - Julien Desfossez <julien.desfossez at polymtl.ca>
+ * Dual LGPL v2.1/GPL v2 license.
+ *
+ */
+
+#include <linux/module.h>
+#include <asm/vmx.h>
+#include <linux/kvm_host.h>
+#include <kvm-trace.h>
+
+void probe_kvm_entry(void *_data, unsigned int vcpu_id)
+{
+	trace_mark_tp(kvm, kvm_entry, kvm_entry,
+		probe_kvm_entry,
+		"vcpu %u",
+		vcpu_id);
+}
+
+void probe_kvm_hypercall(void *_data, unsigned long nr, unsigned long a0, 
+		unsigned long a1, unsigned long a2, unsigned long a3)
+{
+	trace_mark_tp(kvm, kvm_hypercall, kvm_hypercall,
+		probe_kvm_hypercall,
+		"nr 0x%lx a0 0x%lx a1 0x%lx a2 0x%lx a3 0x%lx",
+		nr, a0, a1, a2, a3);
+}
+
+void probe_kvm_pio(void *_data, unsigned int rw, unsigned int port, 
+		unsigned int size, unsigned int count)
+{
+	trace_mark_tp(kvm, kvm_pio, kvm_pio, 
+		probe_kvm_pio,
+		"pio_%s at 0x%x size %d count %d",
+		rw ? "write" : "read",
+		port, size, count);
+}
+
+void probe_kvm_cpuid(void *_data, unsigned int function, unsigned long rax, 
+		unsigned long rbx, unsigned long rcx, unsigned long rdx)
+{
+	trace_mark_tp(kvm, kvm_cpuid, kvm_cpuid,
+		probe_kvm_cpuid,
+		"func %x rax %lx rbx %lx rcx %lx rdx %lx",
+		function, rax, rbx, rcx, rdx);
+}
+
+void probe_kvm_apic (void *_data, unsigned int rw, unsigned int reg, 
+		unsigned int val)
+{
+	trace_mark_tp(kvm, kvm_apic, kvm_apic,
+		probe_kvm_apic,
+		"apic_%s  reg %d = 0x%x",
+		rw ? "write" : "read",
+		reg, val);
+}
+
+void probe_kvm_exit(void *_data, unsigned int exit_reason, struct kvm_vcpu *vcpu)
+{
+	trace_mark_tp(kvm, kvm_exit, kvm_exit,
+		probe_kvm_exit,
+		"reason %d",
+		/* FIXME : kvm_rip_read(vcpu) */
+		exit_reason);
+}
+
+void probe_kvm_inj_virq(void *_data, unsigned int irq)
+{
+	trace_mark_tp(kvm, kvm_inj_virq, kvm_inj_virq,
+		probe_kvm_inj_virq,
+		"irq %u", irq);
+}
+
+void probe_kvm_page_fault(void *_data, unsigned long fault_address, 
+		unsigned int error_code)
+{
+	trace_mark_tp(kvm, kvm_page_fault, kvm_page_fault,
+		probe_kvm_page_fault,
+		"address %lx error_code %x",
+		fault_address, error_code);
+}
+
+void probe_kvm_msr(void *_data, unsigned int rw, __u32 ecx, __u64 data, bool exception)
+{
+	trace_mark_tp(kvm, kvm_msr, kvm_msr,
+		probe_kvm_msr,
+		"msr_%s %x = 0x%llx",
+		rw ? "read" : "write",
+		ecx, data);
+}
+
+void probe_kvm_cr(void *_data, unsigned int rw, unsigned int cr, unsigned long val)
+{
+	trace_mark_tp(kvm, kvm_cr, kvm_cr,
+		probe_kvm_cr,
+		"cr_%s %x = 0x%lx",
+		rw ? "read" : "write",
+		cr, val);
+}
+
+void probe_kvm_pic_set_irq(void *_data, __u8 chip, __u8 pin, __u8 elcr, 
+		__u8 imr, bool coalesced)
+{
+	trace_mark_tp(kvm, kvm_pic_set_irq, kvm_pic_set_irq,
+		probe_kvm_pic_set_irq,
+		"chip %u pin %u (%s%s)%s",
+		chip, pin,
+		(elcr & (1 << pin)) ? "level":"edge",
+		(imr & (1 << pin)) ? "|masked":"",
+		coalesced ? " (coalesced)" : "");
+}
+
+void probe_kvm_apic_ipi(void *_data, __u32 icr_low, __u32 dest_id)
+{
+	trace_mark_tp(kvm, kvm_apic_ipi, kvm_apic_ipi,
+		probe_kvm_apic_ipi,
+		"dst %x vec %u (%x|%s|%s|%s|%x)",
+		dest_id, icr_low,
+		icr_low >> 8 & 0x7,
+		(icr_low & (1<<11)) ? "logical" : "physical",
+		(icr_low & (1<<14)) ? "assert" : "de-assert",
+		(icr_low & (1<<15)) ? "level" : "edge",
+		icr_low >> 18 & 0x3);
+}
+
+void probe_kvm_apic_accept_irq(void *_data, __u32 apicid, __u16 dm, __u8 tm, 
+		__u8 vec, bool coalesced)
+{
+	trace_mark_tp(kvm, kvm_apic_accept_irq, kvm_apic_accept_irq,
+		probe_kvm_apic_accept_irq,
+		"apicid %x vec %u (%x|%s)%s",
+		apicid, vec, dm >> 8 & 0x7,
+		tm ? "level" : "edge",
+		coalesced ? " (coalesced)" : "");
+}
+
+void probe_kvm_nested_vmrun(void *_data, __u64 rip, __u64 vmcb, __u64 nested_rip, 
+		__u32 int_ctl, __u32 event_inj, bool npt)
+{
+	trace_mark_tp(kvm, kvm_nested_vmrun, kvm_nested_vmrun,
+		probe_kvm_nested_vmrun,
+		"rip 0x%Lx vmcb 0x%Lx nrip 0x%Lx int_ctl 0x%x "
+		    "event_inj 0x%x npt %s",
+		(unsigned long long)rip, 
+		(unsigned long long)vmcb, 
+		(unsigned long long)nested_rip, 
+		int_ctl, event_inj,
+		npt ? "on" : "off");
+}
+
+void probe_kvm_nested_vmexit(void *_data, __u64 rip, __u32 exit_code,
+		__u64 exit_info1, __u64 exit_info2,
+		__u32 exit_int_info, __u32 exit_int_info_err)
+{
+	trace_mark_tp(kvm, kvm_nested_vmexit, kvm_nested_vmexit,
+		probe_kvm_nested_vmexit,
+		"rip 0x%Lx reason %d ext_inf1 0x%Lx "
+	        "ext_inf2 0x%Lx ext_int 0x%x ext_int_err 0x%x",
+		(unsigned long long)rip,
+		exit_code,
+		(unsigned long long)exit_info1, 
+		(unsigned long long)exit_info2, 
+		exit_int_info, exit_int_info_err);
+}
+
+void probe_kvm_nested_vmexit_inject(void *_data, __u32 exit_code,
+		__u64 exit_info1, __u64 exit_info2,
+		__u32 exit_int_info, __u32 exit_int_info_err)
+{
+	trace_mark_tp(kvm, kvm_nested_vmexit_inject, kvm_nested_vmexit_inject,
+		probe_kvm_nested_vmexit_inject,
+		"reason %d ext_inf1 0x%Lx "
+			"ext_inf2 0x%Lx ext_int 0x%x ext_int_err 0x%x",
+		exit_code,
+		(unsigned long long)exit_info1, 
+		(unsigned long long)exit_info2, 
+		exit_int_info, exit_int_info_err);
+}
+
+void probe_kvm_nested_intr_vmexit(void *_data, __u64 rip)
+{
+	trace_mark_tp(kvm, kvm_nested_intr_vmexit, kvm_nested_intr_vmexit,
+		probe_kvm_nested_intr_vmexit,
+		"rip 0x%Lx", (unsigned long long)rip );
+}
+
+void probe_kvm_invlpga(void *_data, __u64 rip, int asid, u64 address)
+{
+	trace_mark_tp(kvm, kvm_invlpga, kvm_invlpga,
+		probe_kvm_invlpga,
+		"rip 0x%Lx asid %d address 0x%Lx",
+		(unsigned long long)rip, 
+		asid, 
+		(unsigned long long)address);
+}
+
+void probe_kvm_skinit(void *_data, __u64 rip, __u32 slb)
+{
+	trace_mark_tp(kvm, kvm_skinit, kvm_skinit,
+		probe_kvm_skinit,
+		"rip 0x%Lx slb 0x%x",
+		(unsigned long long)rip, slb);
+}
+
+MODULE_LICENSE("GPL and additional rights");
+MODULE_AUTHOR("Julien Desfossez");
+MODULE_DESCRIPTION("KVM Tracepoint Probes");
-- 
1.7.0.4




^ permalink raw reply	[flat|nested] 5+ messages in thread

* [ltt-dev] [LTTNG-MODULES PATCH] Connect probes to KVM TRACE_EVENTs
  2010-09-10 11:40 [ltt-dev] [LTTNG-MODULES PATCH] Connect probes to KVM TRACE_EVENTs Julien Desfossez
@ 2010-09-11 14:57 ` Mathieu Desnoyers
  2010-09-14 20:59   ` Julien Desfossez
  0 siblings, 1 reply; 5+ messages in thread
From: Mathieu Desnoyers @ 2010-09-11 14:57 UTC (permalink / raw)


* Julien Desfossez (julien.desfossez at polymtl.ca) wrote:
> Signed-off-by: Julien Desfossez <julien.desfossez at polymtl.ca>
> ---
>  probes/Makefile    |    6 ++
>  probes/kvm-trace.c |  211 ++++++++++++++++++++++++++++++++++++++++++++++++++++
>  2 files changed, 217 insertions(+), 0 deletions(-)
>  create mode 100644 probes/kvm-trace.c
> 
> diff --git a/probes/Makefile b/probes/Makefile
> index d8f1c40..bb3aca7 100644
> --- a/probes/Makefile
> +++ b/probes/Makefile
> @@ -44,4 +44,10 @@ endif
>  obj-m += block-trace.o
>  endif
>  
> +ifdef CONFIG_KVM
> +ifdef CONFIG_FTRACE
> +CFLAGS_REMOVE_kvm-trace.o = -pg
> +endif
> +obj-m += kvm-trace.o
> +endif
>  
> diff --git a/probes/kvm-trace.c b/probes/kvm-trace.c
> new file mode 100644
> index 0000000..f8144fa
> --- /dev/null
> +++ b/probes/kvm-trace.c
> @@ -0,0 +1,211 @@
> +/*
> + * ltt/probes/kvm-trace.c
> + *
> + * KVM tracepoint probes.
> + *
> + * (C) Copyright 2010 - Julien Desfossez <julien.desfossez at polymtl.ca>
> + * Dual LGPL v2.1/GPL v2 license.
> + *
> + */
> +
> +#include <linux/module.h>
> +#include <asm/vmx.h>
> +#include <linux/kvm_host.h>
> +#include <kvm-trace.h>
> +
> +void probe_kvm_entry(void *_data, unsigned int vcpu_id)
> +{
> +	trace_mark_tp(kvm, kvm_entry, kvm_entry,
> +		probe_kvm_entry,
> +		"vcpu %u",
> +		vcpu_id);
> +}
> +
> +void probe_kvm_hypercall(void *_data, unsigned long nr, unsigned long a0, 
> +		unsigned long a1, unsigned long a2, unsigned long a3)
> +{
> +	trace_mark_tp(kvm, kvm_hypercall, kvm_hypercall,
> +		probe_kvm_hypercall,
> +		"nr 0x%lx a0 0x%lx a1 0x%lx a2 0x%lx a3 0x%lx",
> +		nr, a0, a1, a2, a3);
> +}
> +
> +void probe_kvm_pio(void *_data, unsigned int rw, unsigned int port, 
> +		unsigned int size, unsigned int count)
> +{
> +	trace_mark_tp(kvm, kvm_pio, kvm_pio, 
> +		probe_kvm_pio,
> +		"pio_%s at 0x%x size %d count %d",
> +		rw ? "write" : "read",
> +		port, size, count);
> +}
> +
> +void probe_kvm_cpuid(void *_data, unsigned int function, unsigned long rax, 
> +		unsigned long rbx, unsigned long rcx, unsigned long rdx)
> +{
> +	trace_mark_tp(kvm, kvm_cpuid, kvm_cpuid,
> +		probe_kvm_cpuid,
> +		"func %x rax %lx rbx %lx rcx %lx rdx %lx",
> +		function, rax, rbx, rcx, rdx);
> +}
> +
> +void probe_kvm_apic (void *_data, unsigned int rw, unsigned int reg, 
> +		unsigned int val)
> +{
> +	trace_mark_tp(kvm, kvm_apic, kvm_apic,
> +		probe_kvm_apic,
> +		"apic_%s  reg %d = 0x%x",
> +		rw ? "write" : "read",
> +		reg, val);
> +}
> +
> +void probe_kvm_exit(void *_data, unsigned int exit_reason, struct kvm_vcpu *vcpu)
> +{
> +	trace_mark_tp(kvm, kvm_exit, kvm_exit,
> +		probe_kvm_exit,
> +		"reason %d",
> +		/* FIXME : kvm_rip_read(vcpu) */

What is the issue here ?

Mathieu

> +		exit_reason);
> +}
> +
> +void probe_kvm_inj_virq(void *_data, unsigned int irq)
> +{
> +	trace_mark_tp(kvm, kvm_inj_virq, kvm_inj_virq,
> +		probe_kvm_inj_virq,
> +		"irq %u", irq);
> +}
> +
> +void probe_kvm_page_fault(void *_data, unsigned long fault_address, 
> +		unsigned int error_code)
> +{
> +	trace_mark_tp(kvm, kvm_page_fault, kvm_page_fault,
> +		probe_kvm_page_fault,
> +		"address %lx error_code %x",
> +		fault_address, error_code);
> +}
> +
> +void probe_kvm_msr(void *_data, unsigned int rw, __u32 ecx, __u64 data, bool exception)
> +{
> +	trace_mark_tp(kvm, kvm_msr, kvm_msr,
> +		probe_kvm_msr,
> +		"msr_%s %x = 0x%llx",
> +		rw ? "read" : "write",
> +		ecx, data);
> +}
> +
> +void probe_kvm_cr(void *_data, unsigned int rw, unsigned int cr, unsigned long val)
> +{
> +	trace_mark_tp(kvm, kvm_cr, kvm_cr,
> +		probe_kvm_cr,
> +		"cr_%s %x = 0x%lx",
> +		rw ? "read" : "write",
> +		cr, val);
> +}
> +
> +void probe_kvm_pic_set_irq(void *_data, __u8 chip, __u8 pin, __u8 elcr, 
> +		__u8 imr, bool coalesced)
> +{
> +	trace_mark_tp(kvm, kvm_pic_set_irq, kvm_pic_set_irq,
> +		probe_kvm_pic_set_irq,
> +		"chip %u pin %u (%s%s)%s",
> +		chip, pin,
> +		(elcr & (1 << pin)) ? "level":"edge",
> +		(imr & (1 << pin)) ? "|masked":"",
> +		coalesced ? " (coalesced)" : "");
> +}
> +
> +void probe_kvm_apic_ipi(void *_data, __u32 icr_low, __u32 dest_id)
> +{
> +	trace_mark_tp(kvm, kvm_apic_ipi, kvm_apic_ipi,
> +		probe_kvm_apic_ipi,
> +		"dst %x vec %u (%x|%s|%s|%s|%x)",
> +		dest_id, icr_low,
> +		icr_low >> 8 & 0x7,
> +		(icr_low & (1<<11)) ? "logical" : "physical",
> +		(icr_low & (1<<14)) ? "assert" : "de-assert",
> +		(icr_low & (1<<15)) ? "level" : "edge",
> +		icr_low >> 18 & 0x3);
> +}
> +
> +void probe_kvm_apic_accept_irq(void *_data, __u32 apicid, __u16 dm, __u8 tm, 
> +		__u8 vec, bool coalesced)
> +{
> +	trace_mark_tp(kvm, kvm_apic_accept_irq, kvm_apic_accept_irq,
> +		probe_kvm_apic_accept_irq,
> +		"apicid %x vec %u (%x|%s)%s",
> +		apicid, vec, dm >> 8 & 0x7,
> +		tm ? "level" : "edge",
> +		coalesced ? " (coalesced)" : "");
> +}
> +
> +void probe_kvm_nested_vmrun(void *_data, __u64 rip, __u64 vmcb, __u64 nested_rip, 
> +		__u32 int_ctl, __u32 event_inj, bool npt)
> +{
> +	trace_mark_tp(kvm, kvm_nested_vmrun, kvm_nested_vmrun,
> +		probe_kvm_nested_vmrun,
> +		"rip 0x%Lx vmcb 0x%Lx nrip 0x%Lx int_ctl 0x%x "
> +		    "event_inj 0x%x npt %s",
> +		(unsigned long long)rip, 
> +		(unsigned long long)vmcb, 
> +		(unsigned long long)nested_rip, 
> +		int_ctl, event_inj,
> +		npt ? "on" : "off");
> +}
> +
> +void probe_kvm_nested_vmexit(void *_data, __u64 rip, __u32 exit_code,
> +		__u64 exit_info1, __u64 exit_info2,
> +		__u32 exit_int_info, __u32 exit_int_info_err)
> +{
> +	trace_mark_tp(kvm, kvm_nested_vmexit, kvm_nested_vmexit,
> +		probe_kvm_nested_vmexit,
> +		"rip 0x%Lx reason %d ext_inf1 0x%Lx "
> +	        "ext_inf2 0x%Lx ext_int 0x%x ext_int_err 0x%x",
> +		(unsigned long long)rip,
> +		exit_code,
> +		(unsigned long long)exit_info1, 
> +		(unsigned long long)exit_info2, 
> +		exit_int_info, exit_int_info_err);
> +}
> +
> +void probe_kvm_nested_vmexit_inject(void *_data, __u32 exit_code,
> +		__u64 exit_info1, __u64 exit_info2,
> +		__u32 exit_int_info, __u32 exit_int_info_err)
> +{
> +	trace_mark_tp(kvm, kvm_nested_vmexit_inject, kvm_nested_vmexit_inject,
> +		probe_kvm_nested_vmexit_inject,
> +		"reason %d ext_inf1 0x%Lx "
> +			"ext_inf2 0x%Lx ext_int 0x%x ext_int_err 0x%x",
> +		exit_code,
> +		(unsigned long long)exit_info1, 
> +		(unsigned long long)exit_info2, 
> +		exit_int_info, exit_int_info_err);
> +}
> +
> +void probe_kvm_nested_intr_vmexit(void *_data, __u64 rip)
> +{
> +	trace_mark_tp(kvm, kvm_nested_intr_vmexit, kvm_nested_intr_vmexit,
> +		probe_kvm_nested_intr_vmexit,
> +		"rip 0x%Lx", (unsigned long long)rip );
> +}
> +
> +void probe_kvm_invlpga(void *_data, __u64 rip, int asid, u64 address)
> +{
> +	trace_mark_tp(kvm, kvm_invlpga, kvm_invlpga,
> +		probe_kvm_invlpga,
> +		"rip 0x%Lx asid %d address 0x%Lx",
> +		(unsigned long long)rip, 
> +		asid, 
> +		(unsigned long long)address);
> +}
> +
> +void probe_kvm_skinit(void *_data, __u64 rip, __u32 slb)
> +{
> +	trace_mark_tp(kvm, kvm_skinit, kvm_skinit,
> +		probe_kvm_skinit,
> +		"rip 0x%Lx slb 0x%x",
> +		(unsigned long long)rip, slb);
> +}
> +
> +MODULE_LICENSE("GPL and additional rights");
> +MODULE_AUTHOR("Julien Desfossez");
> +MODULE_DESCRIPTION("KVM Tracepoint Probes");
> -- 
> 1.7.0.4
> 
> 
> _______________________________________________
> ltt-dev mailing list
> ltt-dev at lists.casi.polymtl.ca
> http://lists.casi.polymtl.ca/cgi-bin/mailman/listinfo/ltt-dev
> 

-- 
Mathieu Desnoyers
Operating System Efficiency R&D Consultant
EfficiOS Inc.
http://www.efficios.com




^ permalink raw reply	[flat|nested] 5+ messages in thread

* [ltt-dev] [LTTNG-MODULES PATCH] Connect probes to KVM TRACE_EVENTs
  2010-09-11 14:57 ` Mathieu Desnoyers
@ 2010-09-14 20:59   ` Julien Desfossez
  0 siblings, 0 replies; 5+ messages in thread
From: Julien Desfossez @ 2010-09-14 20:59 UTC (permalink / raw)


On 10-09-11 10:57 AM, Mathieu Desnoyers wrote:
> * Julien Desfossez (julien.desfossez at polymtl.ca) wrote:
>> Signed-off-by: Julien Desfossez <julien.desfossez at polymtl.ca>
>> ---
>>  probes/Makefile    |    6 ++
>>  probes/kvm-trace.c |  211 ++++++++++++++++++++++++++++++++++++++++++++++++++++
>>  2 files changed, 217 insertions(+), 0 deletions(-)
>>  create mode 100644 probes/kvm-trace.c
>>
>> diff --git a/probes/Makefile b/probes/Makefile
>> index d8f1c40..bb3aca7 100644
>> --- a/probes/Makefile
>> +++ b/probes/Makefile
>> @@ -44,4 +44,10 @@ endif
>>  obj-m += block-trace.o
>>  endif
>>  
>> +ifdef CONFIG_KVM
>> +ifdef CONFIG_FTRACE
>> +CFLAGS_REMOVE_kvm-trace.o = -pg
>> +endif
>> +obj-m += kvm-trace.o
>> +endif
>>  
>> diff --git a/probes/kvm-trace.c b/probes/kvm-trace.c
>> new file mode 100644
>> index 0000000..f8144fa
>> --- /dev/null
>> +++ b/probes/kvm-trace.c
>> @@ -0,0 +1,211 @@
>> +/*
>> + * ltt/probes/kvm-trace.c
>> + *
>> + * KVM tracepoint probes.
>> + *
>> + * (C) Copyright 2010 - Julien Desfossez <julien.desfossez at polymtl.ca>
>> + * Dual LGPL v2.1/GPL v2 license.
>> + *
>> + */
>> +
>> +#include <linux/module.h>
>> +#include <asm/vmx.h>
>> +#include <linux/kvm_host.h>
>> +#include <kvm-trace.h>
>> +
>> +void probe_kvm_entry(void *_data, unsigned int vcpu_id)
>> +{
>> +	trace_mark_tp(kvm, kvm_entry, kvm_entry,
>> +		probe_kvm_entry,
>> +		"vcpu %u",
>> +		vcpu_id);
>> +}
>> +
>> +void probe_kvm_hypercall(void *_data, unsigned long nr, unsigned long a0, 
>> +		unsigned long a1, unsigned long a2, unsigned long a3)
>> +{
>> +	trace_mark_tp(kvm, kvm_hypercall, kvm_hypercall,
>> +		probe_kvm_hypercall,
>> +		"nr 0x%lx a0 0x%lx a1 0x%lx a2 0x%lx a3 0x%lx",
>> +		nr, a0, a1, a2, a3);
>> +}
>> +
>> +void probe_kvm_pio(void *_data, unsigned int rw, unsigned int port, 
>> +		unsigned int size, unsigned int count)
>> +{
>> +	trace_mark_tp(kvm, kvm_pio, kvm_pio, 
>> +		probe_kvm_pio,
>> +		"pio_%s at 0x%x size %d count %d",
>> +		rw ? "write" : "read",
>> +		port, size, count);
>> +}
>> +
>> +void probe_kvm_cpuid(void *_data, unsigned int function, unsigned long rax, 
>> +		unsigned long rbx, unsigned long rcx, unsigned long rdx)
>> +{
>> +	trace_mark_tp(kvm, kvm_cpuid, kvm_cpuid,
>> +		probe_kvm_cpuid,
>> +		"func %x rax %lx rbx %lx rcx %lx rdx %lx",
>> +		function, rax, rbx, rcx, rdx);
>> +}
>> +
>> +void probe_kvm_apic (void *_data, unsigned int rw, unsigned int reg, 
>> +		unsigned int val)
>> +{
>> +	trace_mark_tp(kvm, kvm_apic, kvm_apic,
>> +		probe_kvm_apic,
>> +		"apic_%s  reg %d = 0x%x",
>> +		rw ? "write" : "read",
>> +		reg, val);
>> +}
>> +
>> +void probe_kvm_exit(void *_data, unsigned int exit_reason, struct kvm_vcpu *vcpu)
>> +{
>> +	trace_mark_tp(kvm, kvm_exit, kvm_exit,
>> +		probe_kvm_exit,
>> +		"reason %d",
>> +		/* FIXME : kvm_rip_read(vcpu) */
> 
> What is the issue here ?
The kvm_rip_read is only defined in arch/x86/kvm/kvm_cache_regs.h and
not exported via the kvm_x86_ops. Maybe I could redefine it in this file
(it's only 2 lines) but it doesn't seem clean.
Would you have a recommandation ?

Thanks,

Julien

>> +		exit_reason);
>> +}
>> +
>> +void probe_kvm_inj_virq(void *_data, unsigned int irq)
>> +{
>> +	trace_mark_tp(kvm, kvm_inj_virq, kvm_inj_virq,
>> +		probe_kvm_inj_virq,
>> +		"irq %u", irq);
>> +}
>> +
>> +void probe_kvm_page_fault(void *_data, unsigned long fault_address, 
>> +		unsigned int error_code)
>> +{
>> +	trace_mark_tp(kvm, kvm_page_fault, kvm_page_fault,
>> +		probe_kvm_page_fault,
>> +		"address %lx error_code %x",
>> +		fault_address, error_code);
>> +}
>> +
>> +void probe_kvm_msr(void *_data, unsigned int rw, __u32 ecx, __u64 data, bool exception)
>> +{
>> +	trace_mark_tp(kvm, kvm_msr, kvm_msr,
>> +		probe_kvm_msr,
>> +		"msr_%s %x = 0x%llx",
>> +		rw ? "read" : "write",
>> +		ecx, data);
>> +}
>> +
>> +void probe_kvm_cr(void *_data, unsigned int rw, unsigned int cr, unsigned long val)
>> +{
>> +	trace_mark_tp(kvm, kvm_cr, kvm_cr,
>> +		probe_kvm_cr,
>> +		"cr_%s %x = 0x%lx",
>> +		rw ? "read" : "write",
>> +		cr, val);
>> +}
>> +
>> +void probe_kvm_pic_set_irq(void *_data, __u8 chip, __u8 pin, __u8 elcr, 
>> +		__u8 imr, bool coalesced)
>> +{
>> +	trace_mark_tp(kvm, kvm_pic_set_irq, kvm_pic_set_irq,
>> +		probe_kvm_pic_set_irq,
>> +		"chip %u pin %u (%s%s)%s",
>> +		chip, pin,
>> +		(elcr & (1 << pin)) ? "level":"edge",
>> +		(imr & (1 << pin)) ? "|masked":"",
>> +		coalesced ? " (coalesced)" : "");
>> +}
>> +
>> +void probe_kvm_apic_ipi(void *_data, __u32 icr_low, __u32 dest_id)
>> +{
>> +	trace_mark_tp(kvm, kvm_apic_ipi, kvm_apic_ipi,
>> +		probe_kvm_apic_ipi,
>> +		"dst %x vec %u (%x|%s|%s|%s|%x)",
>> +		dest_id, icr_low,
>> +		icr_low >> 8 & 0x7,
>> +		(icr_low & (1<<11)) ? "logical" : "physical",
>> +		(icr_low & (1<<14)) ? "assert" : "de-assert",
>> +		(icr_low & (1<<15)) ? "level" : "edge",
>> +		icr_low >> 18 & 0x3);
>> +}
>> +
>> +void probe_kvm_apic_accept_irq(void *_data, __u32 apicid, __u16 dm, __u8 tm, 
>> +		__u8 vec, bool coalesced)
>> +{
>> +	trace_mark_tp(kvm, kvm_apic_accept_irq, kvm_apic_accept_irq,
>> +		probe_kvm_apic_accept_irq,
>> +		"apicid %x vec %u (%x|%s)%s",
>> +		apicid, vec, dm >> 8 & 0x7,
>> +		tm ? "level" : "edge",
>> +		coalesced ? " (coalesced)" : "");
>> +}
>> +
>> +void probe_kvm_nested_vmrun(void *_data, __u64 rip, __u64 vmcb, __u64 nested_rip, 
>> +		__u32 int_ctl, __u32 event_inj, bool npt)
>> +{
>> +	trace_mark_tp(kvm, kvm_nested_vmrun, kvm_nested_vmrun,
>> +		probe_kvm_nested_vmrun,
>> +		"rip 0x%Lx vmcb 0x%Lx nrip 0x%Lx int_ctl 0x%x "
>> +		    "event_inj 0x%x npt %s",
>> +		(unsigned long long)rip, 
>> +		(unsigned long long)vmcb, 
>> +		(unsigned long long)nested_rip, 
>> +		int_ctl, event_inj,
>> +		npt ? "on" : "off");
>> +}
>> +
>> +void probe_kvm_nested_vmexit(void *_data, __u64 rip, __u32 exit_code,
>> +		__u64 exit_info1, __u64 exit_info2,
>> +		__u32 exit_int_info, __u32 exit_int_info_err)
>> +{
>> +	trace_mark_tp(kvm, kvm_nested_vmexit, kvm_nested_vmexit,
>> +		probe_kvm_nested_vmexit,
>> +		"rip 0x%Lx reason %d ext_inf1 0x%Lx "
>> +	        "ext_inf2 0x%Lx ext_int 0x%x ext_int_err 0x%x",
>> +		(unsigned long long)rip,
>> +		exit_code,
>> +		(unsigned long long)exit_info1, 
>> +		(unsigned long long)exit_info2, 
>> +		exit_int_info, exit_int_info_err);
>> +}
>> +
>> +void probe_kvm_nested_vmexit_inject(void *_data, __u32 exit_code,
>> +		__u64 exit_info1, __u64 exit_info2,
>> +		__u32 exit_int_info, __u32 exit_int_info_err)
>> +{
>> +	trace_mark_tp(kvm, kvm_nested_vmexit_inject, kvm_nested_vmexit_inject,
>> +		probe_kvm_nested_vmexit_inject,
>> +		"reason %d ext_inf1 0x%Lx "
>> +			"ext_inf2 0x%Lx ext_int 0x%x ext_int_err 0x%x",
>> +		exit_code,
>> +		(unsigned long long)exit_info1, 
>> +		(unsigned long long)exit_info2, 
>> +		exit_int_info, exit_int_info_err);
>> +}
>> +
>> +void probe_kvm_nested_intr_vmexit(void *_data, __u64 rip)
>> +{
>> +	trace_mark_tp(kvm, kvm_nested_intr_vmexit, kvm_nested_intr_vmexit,
>> +		probe_kvm_nested_intr_vmexit,
>> +		"rip 0x%Lx", (unsigned long long)rip );
>> +}
>> +
>> +void probe_kvm_invlpga(void *_data, __u64 rip, int asid, u64 address)
>> +{
>> +	trace_mark_tp(kvm, kvm_invlpga, kvm_invlpga,
>> +		probe_kvm_invlpga,
>> +		"rip 0x%Lx asid %d address 0x%Lx",
>> +		(unsigned long long)rip, 
>> +		asid, 
>> +		(unsigned long long)address);
>> +}
>> +
>> +void probe_kvm_skinit(void *_data, __u64 rip, __u32 slb)
>> +{
>> +	trace_mark_tp(kvm, kvm_skinit, kvm_skinit,
>> +		probe_kvm_skinit,
>> +		"rip 0x%Lx slb 0x%x",
>> +		(unsigned long long)rip, slb);
>> +}
>> +
>> +MODULE_LICENSE("GPL and additional rights");
>> +MODULE_AUTHOR("Julien Desfossez");
>> +MODULE_DESCRIPTION("KVM Tracepoint Probes");
>> -- 
>> 1.7.0.4
>>
>>
>> _______________________________________________
>> ltt-dev mailing list
>> ltt-dev at lists.casi.polymtl.ca
>> http://lists.casi.polymtl.ca/cgi-bin/mailman/listinfo/ltt-dev
>>
> 




^ permalink raw reply	[flat|nested] 5+ messages in thread

* [ltt-dev] [LTTNG-MODULES PATCH] Connect probes to KVM TRACE_EVENTS
  2010-11-19 13:56 [ltt-dev] [LTTNG-MODULES PATCH] Connect probes to KVM TRACE_EVENTS Julien Desfossez
@ 2010-11-22 17:03 ` Mathieu Desnoyers
  0 siblings, 0 replies; 5+ messages in thread
From: Mathieu Desnoyers @ 2010-11-22 17:03 UTC (permalink / raw)


* Julien Desfossez (julien.desfossez at polymtl.ca) wrote:
> This patch includes probes for the already defined KVM TRACE_EVENTS.
> Also it exports the VMX exit strings during the statedump, that way
> the analyzer can match the exit reason id with a more explicit
> representation.
> The new ltt_dump_enum_tables function aims to provide a generic way to
> export strings in the traces when necessary.

Merged, thanks !

Pushed into v0.19-stable branch, and released lttng-modules-0.19.3 with it.

I did fix a few trailing whitespaces. You might want to consider using
"checkpatch.pl" from the Linux kernel scripts in your workflow just before
submitting patches. This is what I always do.

Thanks,

Mathieu

> 
> Signed-off-by: Julien Desfossez <julien.desfossez at polymtl.ca>
> ---
>  ltt-statedump.c    |   64 ++++++++++++++++
>  probes/Makefile    |    6 ++
>  probes/kvm-trace.c |  210 ++++++++++++++++++++++++++++++++++++++++++++++++++++
>  3 files changed, 280 insertions(+), 0 deletions(-)
>  create mode 100644 probes/kvm-trace.c
> 
> diff --git a/ltt-statedump.c b/ltt-statedump.c
> index 06ade69..43cc2ed 100644
> --- a/ltt-statedump.c
> +++ b/ltt-statedump.c
> @@ -39,6 +39,10 @@
>  #include <linux/irq.h>
>  #endif
>  
> +#ifdef CONFIG_HAVE_KVM
> +#include <asm/vmx.h>
> +#endif
> +
>  #define NB_PROC_CHUNK 20
>  
>  /*
> @@ -85,6 +89,65 @@ enum lttng_process_status {
>  	LTTNG_DEAD = 7,
>  };
>  
> +struct trace_enum_map {
> +	long id;
> +	const char *symbol;
> +};
> +
> +#ifdef CONFIG_HAVE_KVM
> +static const struct trace_enum_map vmx_kvm_exit_enum[] = {
> +	{ EXIT_REASON_EXCEPTION_NMI,            "exception" },
> +	{ EXIT_REASON_EXTERNAL_INTERRUPT,       "ext_irq" },
> +	{ EXIT_REASON_TRIPLE_FAULT,             "triple_fault" },
> +	{ EXIT_REASON_PENDING_INTERRUPT,        "interrupt_window" },
> +	{ EXIT_REASON_NMI_WINDOW,               "nmi_window" },
> +	{ EXIT_REASON_TASK_SWITCH,              "task_switch" },
> +	{ EXIT_REASON_CPUID,                    "cpuid" },
> +	{ EXIT_REASON_HLT,                      "halt" },
> +	{ EXIT_REASON_INVLPG,                   "invlpg" },
> +	{ EXIT_REASON_RDPMC,                    "rdpmc" },
> +	{ EXIT_REASON_RDTSC,                    "rdtsc" },
> +	{ EXIT_REASON_VMCALL,                   "hypercall" },
> +	{ EXIT_REASON_VMCLEAR,                  "vmclear" },
> +	{ EXIT_REASON_VMLAUNCH,                 "vmlaunch" },
> +	{ EXIT_REASON_VMPTRLD,                  "vmprtld" },
> +	{ EXIT_REASON_VMPTRST,                  "vmptrst" },
> +	{ EXIT_REASON_VMREAD,                   "vmread" },
> +	{ EXIT_REASON_VMRESUME,                 "vmresume" },
> +	{ EXIT_REASON_VMWRITE,                  "vmwrite" },
> +	{ EXIT_REASON_VMOFF,                    "vmoff" },
> +	{ EXIT_REASON_VMON,                     "vmon" },
> +	{ EXIT_REASON_CR_ACCESS,                "cr_access" },
> +	{ EXIT_REASON_DR_ACCESS,                "dr_access" },
> +	{ EXIT_REASON_IO_INSTRUCTION,           "io_instruction" },
> +	{ EXIT_REASON_MSR_READ,                 "rdmsr" },
> +	{ EXIT_REASON_MSR_WRITE,                "wrmsr" },
> +	{ EXIT_REASON_MWAIT_INSTRUCTION,        "mwait_instruction" },
> +	{ EXIT_REASON_MONITOR_INSTRUCTION,      "monitor_instruction" },
> +	{ EXIT_REASON_PAUSE_INSTRUCTION,        "pause_instruction" },
> +	{ EXIT_REASON_MCE_DURING_VMENTRY,       "mce_during_vmentry" },
> +	{ EXIT_REASON_TPR_BELOW_THRESHOLD,      "tpr_below_thres" },
> +	{ EXIT_REASON_APIC_ACCESS,              "apic_access" },
> +	{ EXIT_REASON_EPT_VIOLATION,            "ept_violation" },
> +	{ EXIT_REASON_EPT_MISCONFIG,            "epg_misconfig" },
> +	{ EXIT_REASON_WBINVD,                   "wbinvd" },
> +	{ -1, NULL }
> +};
> +#endif /* CONFIG_HAVE_KVM */
> +
> +static void ltt_dump_enum_tables(struct ltt_probe_private_data *call_data)
> +{
> +#ifdef CONFIG_HAVE_KVM
> +	int i;
> +	/* KVM exit reasons for VMX */
> +	for(i = 0; vmx_kvm_exit_enum[i].symbol; i++) {
> +		__trace_mark(0, enum_tables, vmx_kvm_exit, call_data,
> +				"id %ld symbol %s", vmx_kvm_exit_enum[i].id,
> +				vmx_kvm_exit_enum[i].symbol);
> +	}
> +#endif /* CONFIG_HAVE_KVM */
> +}
> +
>  #ifdef CONFIG_INET
>  static void ltt_enumerate_device(struct ltt_probe_private_data *call_data,
>  				 struct net_device *dev)
> @@ -374,6 +437,7 @@ static int do_ltt_statedump(struct ltt_probe_private_data *call_data)
>  	ltt_dump_sys_call_table(call_data);
>  	ltt_dump_softirq_vec(call_data);
>  	ltt_dump_idt_table(call_data);
> +	ltt_dump_enum_tables(call_data);
>  
>  	mutex_lock(&statedump_cb_mutex);
>  
> diff --git a/probes/Makefile b/probes/Makefile
> index d8f1c40..bb3aca7 100644
> --- a/probes/Makefile
> +++ b/probes/Makefile
> @@ -44,4 +44,10 @@ endif
>  obj-m += block-trace.o
>  endif
>  
> +ifdef CONFIG_KVM
> +ifdef CONFIG_FTRACE
> +CFLAGS_REMOVE_kvm-trace.o = -pg
> +endif
> +obj-m += kvm-trace.o
> +endif
>  
> diff --git a/probes/kvm-trace.c b/probes/kvm-trace.c
> new file mode 100644
> index 0000000..43289ce
> --- /dev/null
> +++ b/probes/kvm-trace.c
> @@ -0,0 +1,210 @@
> +/*
> + * ltt/probes/kvm-trace.c
> + *
> + * KVM tracepoint probes.
> + *
> + * (C) Copyright 2010 - Julien Desfossez <julien.desfossez at polymtl.ca>
> + * Dual LGPL v2.1/GPL v2 license.
> + *
> + */
> +
> +#include <linux/module.h>
> +#include <asm/vmx.h>
> +#include <linux/kvm_host.h>
> +#include <asm/kvm-trace.h>
> +
> +void probe_kvm_entry(void *_data, unsigned int vcpu_id)
> +{
> +	trace_mark_tp(kvm, kvm_entry, kvm_entry,
> +		probe_kvm_entry,
> +		"vcpu %u",
> +		vcpu_id);
> +}
> +
> +void probe_kvm_hypercall(void *_data, unsigned long nr, unsigned long a0, 
> +		unsigned long a1, unsigned long a2, unsigned long a3)
> +{
> +	trace_mark_tp(kvm, kvm_hypercall, kvm_hypercall,
> +		probe_kvm_hypercall,
> +		"nr 0x%lx a0 0x%lx a1 0x%lx a2 0x%lx a3 0x%lx",
> +		nr, a0, a1, a2, a3);
> +}
> +
> +void probe_kvm_pio(void *_data, unsigned int rw, unsigned int port, 
> +		unsigned int size, unsigned int count)
> +{
> +	trace_mark_tp(kvm, kvm_pio, kvm_pio, 
> +		probe_kvm_pio,
> +		"pio_%s at 0x%x size %d count %d",
> +		rw ? "write" : "read",
> +		port, size, count);
> +}
> +
> +void probe_kvm_cpuid(void *_data, unsigned int function, unsigned long rax, 
> +		unsigned long rbx, unsigned long rcx, unsigned long rdx)
> +{
> +	trace_mark_tp(kvm, kvm_cpuid, kvm_cpuid,
> +		probe_kvm_cpuid,
> +		"func %x rax %lx rbx %lx rcx %lx rdx %lx",
> +		function, rax, rbx, rcx, rdx);
> +}
> +
> +void probe_kvm_apic (void *_data, unsigned int rw, unsigned int reg, 
> +		unsigned int val)
> +{
> +	trace_mark_tp(kvm, kvm_apic, kvm_apic,
> +		probe_kvm_apic,
> +		"apic_%s  reg %d = 0x%x",
> +		rw ? "write" : "read",
> +		reg, val);
> +}
> +
> +void probe_kvm_exit(void *_data, unsigned int exit_reason, struct kvm_vcpu *vcpu)
> +{
> +	trace_mark_tp(kvm, kvm_exit, kvm_exit,
> +		probe_kvm_exit,
> +		"reason %d",
> +		exit_reason);
> +}
> +
> +void probe_kvm_inj_virq(void *_data, unsigned int irq)
> +{
> +	trace_mark_tp(kvm, kvm_inj_virq, kvm_inj_virq,
> +		probe_kvm_inj_virq,
> +		"irq %u", irq);
> +}
> +
> +void probe_kvm_page_fault(void *_data, unsigned long fault_address, 
> +		unsigned int error_code)
> +{
> +	trace_mark_tp(kvm, kvm_page_fault, kvm_page_fault,
> +		probe_kvm_page_fault,
> +		"address %lx error_code %x",
> +		fault_address, error_code);
> +}
> +
> +void probe_kvm_msr(void *_data, unsigned int rw, __u32 ecx, __u64 data, bool exception)
> +{
> +	trace_mark_tp(kvm, kvm_msr, kvm_msr,
> +		probe_kvm_msr,
> +		"msr_%s %x = 0x%llx",
> +		rw ? "read" : "write",
> +		ecx, data);
> +}
> +
> +void probe_kvm_cr(void *_data, unsigned int rw, unsigned int cr, unsigned long val)
> +{
> +	trace_mark_tp(kvm, kvm_cr, kvm_cr,
> +		probe_kvm_cr,
> +		"cr_%s %x = 0x%lx",
> +		rw ? "read" : "write",
> +		cr, val);
> +}
> +
> +void probe_kvm_pic_set_irq(void *_data, __u8 chip, __u8 pin, __u8 elcr, 
> +		__u8 imr, bool coalesced)
> +{
> +	trace_mark_tp(kvm, kvm_pic_set_irq, kvm_pic_set_irq,
> +		probe_kvm_pic_set_irq,
> +		"chip %u pin %u (%s%s)%s",
> +		chip, pin,
> +		(elcr & (1 << pin)) ? "level":"edge",
> +		(imr & (1 << pin)) ? "|masked":"",
> +		coalesced ? " (coalesced)" : "");
> +}
> +
> +void probe_kvm_apic_ipi(void *_data, __u32 icr_low, __u32 dest_id)
> +{
> +	trace_mark_tp(kvm, kvm_apic_ipi, kvm_apic_ipi,
> +		probe_kvm_apic_ipi,
> +		"dst %x vec %u (%x|%s|%s|%s|%x)",
> +		dest_id, icr_low,
> +		icr_low >> 8 & 0x7,
> +		(icr_low & (1<<11)) ? "logical" : "physical",
> +		(icr_low & (1<<14)) ? "assert" : "de-assert",
> +		(icr_low & (1<<15)) ? "level" : "edge",
> +		icr_low >> 18 & 0x3);
> +}
> +
> +void probe_kvm_apic_accept_irq(void *_data, __u32 apicid, __u16 dm, __u8 tm, 
> +		__u8 vec, bool coalesced)
> +{
> +	trace_mark_tp(kvm, kvm_apic_accept_irq, kvm_apic_accept_irq,
> +		probe_kvm_apic_accept_irq,
> +		"apicid %x vec %u (%x|%s)%s",
> +		apicid, vec, dm >> 8 & 0x7,
> +		tm ? "level" : "edge",
> +		coalesced ? " (coalesced)" : "");
> +}
> +
> +void probe_kvm_nested_vmrun(void *_data, __u64 rip, __u64 vmcb, __u64 nested_rip, 
> +		__u32 int_ctl, __u32 event_inj, bool npt)
> +{
> +	trace_mark_tp(kvm, kvm_nested_vmrun, kvm_nested_vmrun,
> +		probe_kvm_nested_vmrun,
> +		"rip 0x%Lx vmcb 0x%Lx nrip 0x%Lx int_ctl 0x%x "
> +		    "event_inj 0x%x npt %s",
> +		(unsigned long long)rip, 
> +		(unsigned long long)vmcb, 
> +		(unsigned long long)nested_rip, 
> +		int_ctl, event_inj,
> +		npt ? "on" : "off");
> +}
> +
> +void probe_kvm_nested_vmexit(void *_data, __u64 rip, __u32 exit_code,
> +		__u64 exit_info1, __u64 exit_info2,
> +		__u32 exit_int_info, __u32 exit_int_info_err)
> +{
> +	trace_mark_tp(kvm, kvm_nested_vmexit, kvm_nested_vmexit,
> +		probe_kvm_nested_vmexit,
> +		"rip 0x%Lx reason %d ext_inf1 0x%Lx "
> +	        "ext_inf2 0x%Lx ext_int 0x%x ext_int_err 0x%x",
> +		(unsigned long long)rip,
> +		exit_code,
> +		(unsigned long long)exit_info1, 
> +		(unsigned long long)exit_info2, 
> +		exit_int_info, exit_int_info_err);
> +}
> +
> +void probe_kvm_nested_vmexit_inject(void *_data, __u32 exit_code,
> +		__u64 exit_info1, __u64 exit_info2,
> +		__u32 exit_int_info, __u32 exit_int_info_err)
> +{
> +	trace_mark_tp(kvm, kvm_nested_vmexit_inject, kvm_nested_vmexit_inject,
> +		probe_kvm_nested_vmexit_inject,
> +		"reason %d ext_inf1 0x%Lx "
> +			"ext_inf2 0x%Lx ext_int 0x%x ext_int_err 0x%x",
> +		exit_code,
> +		(unsigned long long)exit_info1, 
> +		(unsigned long long)exit_info2, 
> +		exit_int_info, exit_int_info_err);
> +}
> +
> +void probe_kvm_nested_intr_vmexit(void *_data, __u64 rip)
> +{
> +	trace_mark_tp(kvm, kvm_nested_intr_vmexit, kvm_nested_intr_vmexit,
> +		probe_kvm_nested_intr_vmexit,
> +		"rip 0x%Lx", (unsigned long long)rip );
> +}
> +
> +void probe_kvm_invlpga(void *_data, __u64 rip, int asid, u64 address)
> +{
> +	trace_mark_tp(kvm, kvm_invlpga, kvm_invlpga,
> +		probe_kvm_invlpga,
> +		"rip 0x%Lx asid %d address 0x%Lx",
> +		(unsigned long long)rip, 
> +		asid, 
> +		(unsigned long long)address);
> +}
> +
> +void probe_kvm_skinit(void *_data, __u64 rip, __u32 slb)
> +{
> +	trace_mark_tp(kvm, kvm_skinit, kvm_skinit,
> +		probe_kvm_skinit,
> +		"rip 0x%Lx slb 0x%x",
> +		(unsigned long long)rip, slb);
> +}
> +
> +MODULE_LICENSE("GPL and additional rights");
> +MODULE_AUTHOR("Julien Desfossez");
> +MODULE_DESCRIPTION("KVM Tracepoint Probes");
> -- 
> 1.7.0.4
> 

-- 
Mathieu Desnoyers
Operating System Efficiency R&D Consultant
EfficiOS Inc.
http://www.efficios.com




^ permalink raw reply	[flat|nested] 5+ messages in thread

* [ltt-dev] [LTTNG-MODULES PATCH] Connect probes to KVM TRACE_EVENTS
@ 2010-11-19 13:56 Julien Desfossez
  2010-11-22 17:03 ` Mathieu Desnoyers
  0 siblings, 1 reply; 5+ messages in thread
From: Julien Desfossez @ 2010-11-19 13:56 UTC (permalink / raw)


This patch includes probes for the already defined KVM TRACE_EVENTS.
Also it exports the VMX exit strings during the statedump, that way
the analyzer can match the exit reason id with a more explicit
representation.
The new ltt_dump_enum_tables function aims to provide a generic way to
export strings in the traces when necessary.

Signed-off-by: Julien Desfossez <julien.desfossez at polymtl.ca>
---
 ltt-statedump.c    |   64 ++++++++++++++++
 probes/Makefile    |    6 ++
 probes/kvm-trace.c |  210 ++++++++++++++++++++++++++++++++++++++++++++++++++++
 3 files changed, 280 insertions(+), 0 deletions(-)
 create mode 100644 probes/kvm-trace.c

diff --git a/ltt-statedump.c b/ltt-statedump.c
index 06ade69..43cc2ed 100644
--- a/ltt-statedump.c
+++ b/ltt-statedump.c
@@ -39,6 +39,10 @@
 #include <linux/irq.h>
 #endif
 
+#ifdef CONFIG_HAVE_KVM
+#include <asm/vmx.h>
+#endif
+
 #define NB_PROC_CHUNK 20
 
 /*
@@ -85,6 +89,65 @@ enum lttng_process_status {
 	LTTNG_DEAD = 7,
 };
 
+struct trace_enum_map {
+	long id;
+	const char *symbol;
+};
+
+#ifdef CONFIG_HAVE_KVM
+static const struct trace_enum_map vmx_kvm_exit_enum[] = {
+	{ EXIT_REASON_EXCEPTION_NMI,            "exception" },
+	{ EXIT_REASON_EXTERNAL_INTERRUPT,       "ext_irq" },
+	{ EXIT_REASON_TRIPLE_FAULT,             "triple_fault" },
+	{ EXIT_REASON_PENDING_INTERRUPT,        "interrupt_window" },
+	{ EXIT_REASON_NMI_WINDOW,               "nmi_window" },
+	{ EXIT_REASON_TASK_SWITCH,              "task_switch" },
+	{ EXIT_REASON_CPUID,                    "cpuid" },
+	{ EXIT_REASON_HLT,                      "halt" },
+	{ EXIT_REASON_INVLPG,                   "invlpg" },
+	{ EXIT_REASON_RDPMC,                    "rdpmc" },
+	{ EXIT_REASON_RDTSC,                    "rdtsc" },
+	{ EXIT_REASON_VMCALL,                   "hypercall" },
+	{ EXIT_REASON_VMCLEAR,                  "vmclear" },
+	{ EXIT_REASON_VMLAUNCH,                 "vmlaunch" },
+	{ EXIT_REASON_VMPTRLD,                  "vmprtld" },
+	{ EXIT_REASON_VMPTRST,                  "vmptrst" },
+	{ EXIT_REASON_VMREAD,                   "vmread" },
+	{ EXIT_REASON_VMRESUME,                 "vmresume" },
+	{ EXIT_REASON_VMWRITE,                  "vmwrite" },
+	{ EXIT_REASON_VMOFF,                    "vmoff" },
+	{ EXIT_REASON_VMON,                     "vmon" },
+	{ EXIT_REASON_CR_ACCESS,                "cr_access" },
+	{ EXIT_REASON_DR_ACCESS,                "dr_access" },
+	{ EXIT_REASON_IO_INSTRUCTION,           "io_instruction" },
+	{ EXIT_REASON_MSR_READ,                 "rdmsr" },
+	{ EXIT_REASON_MSR_WRITE,                "wrmsr" },
+	{ EXIT_REASON_MWAIT_INSTRUCTION,        "mwait_instruction" },
+	{ EXIT_REASON_MONITOR_INSTRUCTION,      "monitor_instruction" },
+	{ EXIT_REASON_PAUSE_INSTRUCTION,        "pause_instruction" },
+	{ EXIT_REASON_MCE_DURING_VMENTRY,       "mce_during_vmentry" },
+	{ EXIT_REASON_TPR_BELOW_THRESHOLD,      "tpr_below_thres" },
+	{ EXIT_REASON_APIC_ACCESS,              "apic_access" },
+	{ EXIT_REASON_EPT_VIOLATION,            "ept_violation" },
+	{ EXIT_REASON_EPT_MISCONFIG,            "epg_misconfig" },
+	{ EXIT_REASON_WBINVD,                   "wbinvd" },
+	{ -1, NULL }
+};
+#endif /* CONFIG_HAVE_KVM */
+
+static void ltt_dump_enum_tables(struct ltt_probe_private_data *call_data)
+{
+#ifdef CONFIG_HAVE_KVM
+	int i;
+	/* KVM exit reasons for VMX */
+	for(i = 0; vmx_kvm_exit_enum[i].symbol; i++) {
+		__trace_mark(0, enum_tables, vmx_kvm_exit, call_data,
+				"id %ld symbol %s", vmx_kvm_exit_enum[i].id,
+				vmx_kvm_exit_enum[i].symbol);
+	}
+#endif /* CONFIG_HAVE_KVM */
+}
+
 #ifdef CONFIG_INET
 static void ltt_enumerate_device(struct ltt_probe_private_data *call_data,
 				 struct net_device *dev)
@@ -374,6 +437,7 @@ static int do_ltt_statedump(struct ltt_probe_private_data *call_data)
 	ltt_dump_sys_call_table(call_data);
 	ltt_dump_softirq_vec(call_data);
 	ltt_dump_idt_table(call_data);
+	ltt_dump_enum_tables(call_data);
 
 	mutex_lock(&statedump_cb_mutex);
 
diff --git a/probes/Makefile b/probes/Makefile
index d8f1c40..bb3aca7 100644
--- a/probes/Makefile
+++ b/probes/Makefile
@@ -44,4 +44,10 @@ endif
 obj-m += block-trace.o
 endif
 
+ifdef CONFIG_KVM
+ifdef CONFIG_FTRACE
+CFLAGS_REMOVE_kvm-trace.o = -pg
+endif
+obj-m += kvm-trace.o
+endif
 
diff --git a/probes/kvm-trace.c b/probes/kvm-trace.c
new file mode 100644
index 0000000..43289ce
--- /dev/null
+++ b/probes/kvm-trace.c
@@ -0,0 +1,210 @@
+/*
+ * ltt/probes/kvm-trace.c
+ *
+ * KVM tracepoint probes.
+ *
+ * (C) Copyright 2010 - Julien Desfossez <julien.desfossez at polymtl.ca>
+ * Dual LGPL v2.1/GPL v2 license.
+ *
+ */
+
+#include <linux/module.h>
+#include <asm/vmx.h>
+#include <linux/kvm_host.h>
+#include <asm/kvm-trace.h>
+
+void probe_kvm_entry(void *_data, unsigned int vcpu_id)
+{
+	trace_mark_tp(kvm, kvm_entry, kvm_entry,
+		probe_kvm_entry,
+		"vcpu %u",
+		vcpu_id);
+}
+
+void probe_kvm_hypercall(void *_data, unsigned long nr, unsigned long a0, 
+		unsigned long a1, unsigned long a2, unsigned long a3)
+{
+	trace_mark_tp(kvm, kvm_hypercall, kvm_hypercall,
+		probe_kvm_hypercall,
+		"nr 0x%lx a0 0x%lx a1 0x%lx a2 0x%lx a3 0x%lx",
+		nr, a0, a1, a2, a3);
+}
+
+void probe_kvm_pio(void *_data, unsigned int rw, unsigned int port, 
+		unsigned int size, unsigned int count)
+{
+	trace_mark_tp(kvm, kvm_pio, kvm_pio, 
+		probe_kvm_pio,
+		"pio_%s at 0x%x size %d count %d",
+		rw ? "write" : "read",
+		port, size, count);
+}
+
+void probe_kvm_cpuid(void *_data, unsigned int function, unsigned long rax, 
+		unsigned long rbx, unsigned long rcx, unsigned long rdx)
+{
+	trace_mark_tp(kvm, kvm_cpuid, kvm_cpuid,
+		probe_kvm_cpuid,
+		"func %x rax %lx rbx %lx rcx %lx rdx %lx",
+		function, rax, rbx, rcx, rdx);
+}
+
+void probe_kvm_apic (void *_data, unsigned int rw, unsigned int reg, 
+		unsigned int val)
+{
+	trace_mark_tp(kvm, kvm_apic, kvm_apic,
+		probe_kvm_apic,
+		"apic_%s  reg %d = 0x%x",
+		rw ? "write" : "read",
+		reg, val);
+}
+
+void probe_kvm_exit(void *_data, unsigned int exit_reason, struct kvm_vcpu *vcpu)
+{
+	trace_mark_tp(kvm, kvm_exit, kvm_exit,
+		probe_kvm_exit,
+		"reason %d",
+		exit_reason);
+}
+
+void probe_kvm_inj_virq(void *_data, unsigned int irq)
+{
+	trace_mark_tp(kvm, kvm_inj_virq, kvm_inj_virq,
+		probe_kvm_inj_virq,
+		"irq %u", irq);
+}
+
+void probe_kvm_page_fault(void *_data, unsigned long fault_address, 
+		unsigned int error_code)
+{
+	trace_mark_tp(kvm, kvm_page_fault, kvm_page_fault,
+		probe_kvm_page_fault,
+		"address %lx error_code %x",
+		fault_address, error_code);
+}
+
+void probe_kvm_msr(void *_data, unsigned int rw, __u32 ecx, __u64 data, bool exception)
+{
+	trace_mark_tp(kvm, kvm_msr, kvm_msr,
+		probe_kvm_msr,
+		"msr_%s %x = 0x%llx",
+		rw ? "read" : "write",
+		ecx, data);
+}
+
+void probe_kvm_cr(void *_data, unsigned int rw, unsigned int cr, unsigned long val)
+{
+	trace_mark_tp(kvm, kvm_cr, kvm_cr,
+		probe_kvm_cr,
+		"cr_%s %x = 0x%lx",
+		rw ? "read" : "write",
+		cr, val);
+}
+
+void probe_kvm_pic_set_irq(void *_data, __u8 chip, __u8 pin, __u8 elcr, 
+		__u8 imr, bool coalesced)
+{
+	trace_mark_tp(kvm, kvm_pic_set_irq, kvm_pic_set_irq,
+		probe_kvm_pic_set_irq,
+		"chip %u pin %u (%s%s)%s",
+		chip, pin,
+		(elcr & (1 << pin)) ? "level":"edge",
+		(imr & (1 << pin)) ? "|masked":"",
+		coalesced ? " (coalesced)" : "");
+}
+
+void probe_kvm_apic_ipi(void *_data, __u32 icr_low, __u32 dest_id)
+{
+	trace_mark_tp(kvm, kvm_apic_ipi, kvm_apic_ipi,
+		probe_kvm_apic_ipi,
+		"dst %x vec %u (%x|%s|%s|%s|%x)",
+		dest_id, icr_low,
+		icr_low >> 8 & 0x7,
+		(icr_low & (1<<11)) ? "logical" : "physical",
+		(icr_low & (1<<14)) ? "assert" : "de-assert",
+		(icr_low & (1<<15)) ? "level" : "edge",
+		icr_low >> 18 & 0x3);
+}
+
+void probe_kvm_apic_accept_irq(void *_data, __u32 apicid, __u16 dm, __u8 tm, 
+		__u8 vec, bool coalesced)
+{
+	trace_mark_tp(kvm, kvm_apic_accept_irq, kvm_apic_accept_irq,
+		probe_kvm_apic_accept_irq,
+		"apicid %x vec %u (%x|%s)%s",
+		apicid, vec, dm >> 8 & 0x7,
+		tm ? "level" : "edge",
+		coalesced ? " (coalesced)" : "");
+}
+
+void probe_kvm_nested_vmrun(void *_data, __u64 rip, __u64 vmcb, __u64 nested_rip, 
+		__u32 int_ctl, __u32 event_inj, bool npt)
+{
+	trace_mark_tp(kvm, kvm_nested_vmrun, kvm_nested_vmrun,
+		probe_kvm_nested_vmrun,
+		"rip 0x%Lx vmcb 0x%Lx nrip 0x%Lx int_ctl 0x%x "
+		    "event_inj 0x%x npt %s",
+		(unsigned long long)rip, 
+		(unsigned long long)vmcb, 
+		(unsigned long long)nested_rip, 
+		int_ctl, event_inj,
+		npt ? "on" : "off");
+}
+
+void probe_kvm_nested_vmexit(void *_data, __u64 rip, __u32 exit_code,
+		__u64 exit_info1, __u64 exit_info2,
+		__u32 exit_int_info, __u32 exit_int_info_err)
+{
+	trace_mark_tp(kvm, kvm_nested_vmexit, kvm_nested_vmexit,
+		probe_kvm_nested_vmexit,
+		"rip 0x%Lx reason %d ext_inf1 0x%Lx "
+	        "ext_inf2 0x%Lx ext_int 0x%x ext_int_err 0x%x",
+		(unsigned long long)rip,
+		exit_code,
+		(unsigned long long)exit_info1, 
+		(unsigned long long)exit_info2, 
+		exit_int_info, exit_int_info_err);
+}
+
+void probe_kvm_nested_vmexit_inject(void *_data, __u32 exit_code,
+		__u64 exit_info1, __u64 exit_info2,
+		__u32 exit_int_info, __u32 exit_int_info_err)
+{
+	trace_mark_tp(kvm, kvm_nested_vmexit_inject, kvm_nested_vmexit_inject,
+		probe_kvm_nested_vmexit_inject,
+		"reason %d ext_inf1 0x%Lx "
+			"ext_inf2 0x%Lx ext_int 0x%x ext_int_err 0x%x",
+		exit_code,
+		(unsigned long long)exit_info1, 
+		(unsigned long long)exit_info2, 
+		exit_int_info, exit_int_info_err);
+}
+
+void probe_kvm_nested_intr_vmexit(void *_data, __u64 rip)
+{
+	trace_mark_tp(kvm, kvm_nested_intr_vmexit, kvm_nested_intr_vmexit,
+		probe_kvm_nested_intr_vmexit,
+		"rip 0x%Lx", (unsigned long long)rip );
+}
+
+void probe_kvm_invlpga(void *_data, __u64 rip, int asid, u64 address)
+{
+	trace_mark_tp(kvm, kvm_invlpga, kvm_invlpga,
+		probe_kvm_invlpga,
+		"rip 0x%Lx asid %d address 0x%Lx",
+		(unsigned long long)rip, 
+		asid, 
+		(unsigned long long)address);
+}
+
+void probe_kvm_skinit(void *_data, __u64 rip, __u32 slb)
+{
+	trace_mark_tp(kvm, kvm_skinit, kvm_skinit,
+		probe_kvm_skinit,
+		"rip 0x%Lx slb 0x%x",
+		(unsigned long long)rip, slb);
+}
+
+MODULE_LICENSE("GPL and additional rights");
+MODULE_AUTHOR("Julien Desfossez");
+MODULE_DESCRIPTION("KVM Tracepoint Probes");
-- 
1.7.0.4




^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2010-11-22 17:03 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-09-10 11:40 [ltt-dev] [LTTNG-MODULES PATCH] Connect probes to KVM TRACE_EVENTs Julien Desfossez
2010-09-11 14:57 ` Mathieu Desnoyers
2010-09-14 20:59   ` Julien Desfossez
2010-11-19 13:56 [ltt-dev] [LTTNG-MODULES PATCH] Connect probes to KVM TRACE_EVENTS Julien Desfossez
2010-11-22 17:03 ` Mathieu Desnoyers

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox