Commit 1171903d authored by Ulrich Obergfell's avatar Ulrich Obergfell Committed by Paolo Bonzini

KVM: x86: improve the usability of the 'kvm_pio' tracepoint

This patch moves the 'kvm_pio' tracepoint to emulator_pio_in_emulated()
and emulator_pio_out_emulated(), and it adds an argument (a pointer to
the 'pio_data'). A single 8-bit or 16-bit or 32-bit data item is fetched
from 'pio_data' (depending on 'size'), and the value is included in the
trace record ('val'). If 'count' is greater than one, this is indicated
by the string "(...)" in the trace output.
Signed-off-by: default avatarUlrich Obergfell <uobergfe@redhat.com>
Reviewed-by: default avatarXiao Guangrong <xiaoguangrong@linux.vnet.ibm.com>
Signed-off-by: default avatarPaolo Bonzini <pbonzini@redhat.com>
parent 719d93cd
...@@ -91,16 +91,21 @@ TRACE_EVENT(kvm_hv_hypercall, ...@@ -91,16 +91,21 @@ TRACE_EVENT(kvm_hv_hypercall,
/* /*
* Tracepoint for PIO. * Tracepoint for PIO.
*/ */
#define KVM_PIO_IN 0
#define KVM_PIO_OUT 1
TRACE_EVENT(kvm_pio, TRACE_EVENT(kvm_pio,
TP_PROTO(unsigned int rw, unsigned int port, unsigned int size, TP_PROTO(unsigned int rw, unsigned int port, unsigned int size,
unsigned int count), unsigned int count, void *data),
TP_ARGS(rw, port, size, count), TP_ARGS(rw, port, size, count, data),
TP_STRUCT__entry( TP_STRUCT__entry(
__field( unsigned int, rw ) __field( unsigned int, rw )
__field( unsigned int, port ) __field( unsigned int, port )
__field( unsigned int, size ) __field( unsigned int, size )
__field( unsigned int, count ) __field( unsigned int, count )
__field( unsigned int, val )
), ),
TP_fast_assign( TP_fast_assign(
...@@ -108,11 +113,18 @@ TRACE_EVENT(kvm_pio, ...@@ -108,11 +113,18 @@ TRACE_EVENT(kvm_pio,
__entry->port = port; __entry->port = port;
__entry->size = size; __entry->size = size;
__entry->count = count; __entry->count = count;
if (size == 1)
__entry->val = *(unsigned char *)data;
else if (size == 2)
__entry->val = *(unsigned short *)data;
else
__entry->val = *(unsigned int *)data;
), ),
TP_printk("pio_%s at 0x%x size %d count %d", TP_printk("pio_%s at 0x%x size %d count %d val 0x%x %s",
__entry->rw ? "write" : "read", __entry->rw ? "write" : "read",
__entry->port, __entry->size, __entry->count) __entry->port, __entry->size, __entry->count, __entry->val,
__entry->count > 1 ? "(...)" : "")
); );
/* /*
......
...@@ -4480,8 +4480,6 @@ static int emulator_pio_in_out(struct kvm_vcpu *vcpu, int size, ...@@ -4480,8 +4480,6 @@ static int emulator_pio_in_out(struct kvm_vcpu *vcpu, int size,
unsigned short port, void *val, unsigned short port, void *val,
unsigned int count, bool in) unsigned int count, bool in)
{ {
trace_kvm_pio(!in, port, size, count);
vcpu->arch.pio.port = port; vcpu->arch.pio.port = port;
vcpu->arch.pio.in = in; vcpu->arch.pio.in = in;
vcpu->arch.pio.count = count; vcpu->arch.pio.count = count;
...@@ -4516,6 +4514,7 @@ static int emulator_pio_in_emulated(struct x86_emulate_ctxt *ctxt, ...@@ -4516,6 +4514,7 @@ static int emulator_pio_in_emulated(struct x86_emulate_ctxt *ctxt,
if (ret) { if (ret) {
data_avail: data_avail:
memcpy(val, vcpu->arch.pio_data, size * count); memcpy(val, vcpu->arch.pio_data, size * count);
trace_kvm_pio(KVM_PIO_IN, port, size, count, vcpu->arch.pio_data);
vcpu->arch.pio.count = 0; vcpu->arch.pio.count = 0;
return 1; return 1;
} }
...@@ -4530,6 +4529,7 @@ static int emulator_pio_out_emulated(struct x86_emulate_ctxt *ctxt, ...@@ -4530,6 +4529,7 @@ static int emulator_pio_out_emulated(struct x86_emulate_ctxt *ctxt,
struct kvm_vcpu *vcpu = emul_to_vcpu(ctxt); struct kvm_vcpu *vcpu = emul_to_vcpu(ctxt);
memcpy(vcpu->arch.pio_data, val, size * count); memcpy(vcpu->arch.pio_data, val, size * count);
trace_kvm_pio(KVM_PIO_OUT, port, size, count, vcpu->arch.pio_data);
return emulator_pio_in_out(vcpu, size, port, (void *)val, count, false); return emulator_pio_in_out(vcpu, size, port, (void *)val, count, false);
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment