On 25/04/14 11:12, Christian Borntraeger wrote: > +#if defined(__i386__) || defined(__x86_64__) > else if (!strcmp(kvm->report_event, "mmio")) > kvm->events_ops = &mmio_events; > else if (!strcmp(kvm->report_event, "ioport")) > kvm->events_ops = &ioport_events; > +#endif To address Davids review, the next version will have this hunk as well: diff --git a/tools/perf/Documentation/perf-kvm.txt b/tools/perf/Documentation/perf-kvm.txt index 52276a6..e974749 100644 --- a/tools/perf/Documentation/perf-kvm.txt +++ b/tools/perf/Documentation/perf-kvm.txt @@ -103,8 +103,8 @@ STAT REPORT OPTIONS analyze events which occures on this vcpu. (default: all vcpus) --event=<value>:: - event to be analyzed. Possible values: vmexit, mmio, ioport. - (default: vmexit) + event to be analyzed. Possible values: vmexit, mmio (x86 only), + ioport (x86 only). (default: vmexit) -k:: --key=<value>:: Sorting key. Possible values: sample (default, sort by samples -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html