Commit 162607ea authored by Hemant Kumar's avatar Hemant Kumar Committed by Arnaldo Carvalho de Melo

perf kvm/{x86,s390}: Remove dependency on uapi/kvm_perf.h

Its better to remove the dependency on uapi/kvm_perf.h to allow dynamic
discovery of kvm events (if its needed). To do this, some extern
variables have been introduced with which we can keep the generic
functions generic.
Signed-off-by: default avatarHemant Kumar <hemant@linux.vnet.ibm.com>
Acked-by: default avatarAlexander Yarygin <yarygin@linux.vnet.ibm.com>
Acked-by: default avatarDavid Ahern <dsahern@gmail.com>
Cc: Michael Ellerman <mpe@ellerman.id.au>
Cc: Naveen N. Rao <naveen.n.rao@linux.vnet.ibm.com>
Cc: Paul Mackerras <paulus@samba.org>
Cc: Scott  Wood <scottwood@freescale.com>
Cc: Srikar Dronamraju <srikar@linux.vnet.ibm.com>
Cc: linuxppc-dev@lists.ozlabs.org
Link: http://lkml.kernel.org/r/1453962787-15376-1-git-send-email-hemant@linux.vnet.ibm.comSigned-off-by: default avatarArnaldo Carvalho de Melo <acme@redhat.com>
parent d2db9a98
...@@ -10,7 +10,7 @@ ...@@ -10,7 +10,7 @@
*/ */
#include "../../util/kvm-stat.h" #include "../../util/kvm-stat.h"
#include <asm/kvm_perf.h> #include <asm/sie.h>
define_exit_reasons_table(sie_exit_reasons, sie_intercept_code); define_exit_reasons_table(sie_exit_reasons, sie_intercept_code);
define_exit_reasons_table(sie_icpt_insn_codes, icpt_insn_codes); define_exit_reasons_table(sie_icpt_insn_codes, icpt_insn_codes);
...@@ -18,6 +18,12 @@ define_exit_reasons_table(sie_sigp_order_codes, sigp_order_codes); ...@@ -18,6 +18,12 @@ define_exit_reasons_table(sie_sigp_order_codes, sigp_order_codes);
define_exit_reasons_table(sie_diagnose_codes, diagnose_codes); define_exit_reasons_table(sie_diagnose_codes, diagnose_codes);
define_exit_reasons_table(sie_icpt_prog_codes, icpt_prog_codes); define_exit_reasons_table(sie_icpt_prog_codes, icpt_prog_codes);
const char *vcpu_id_str = "id";
const int decode_str_len = 40;
const char *kvm_exit_reason = "icptcode";
const char *kvm_entry_trace = "kvm:kvm_s390_sie_enter";
const char *kvm_exit_trace = "kvm:kvm_s390_sie_exit";
static void event_icpt_insn_get_key(struct perf_evsel *evsel, static void event_icpt_insn_get_key(struct perf_evsel *evsel,
struct perf_sample *sample, struct perf_sample *sample,
struct event_key *key) struct event_key *key)
......
#include "../../util/kvm-stat.h" #include "../../util/kvm-stat.h"
#include <asm/kvm_perf.h> #include <asm/svm.h>
#include <asm/vmx.h>
#include <asm/kvm.h>
define_exit_reasons_table(vmx_exit_reasons, VMX_EXIT_REASONS); define_exit_reasons_table(vmx_exit_reasons, VMX_EXIT_REASONS);
define_exit_reasons_table(svm_exit_reasons, SVM_EXIT_REASONS); define_exit_reasons_table(svm_exit_reasons, SVM_EXIT_REASONS);
...@@ -11,6 +13,12 @@ static struct kvm_events_ops exit_events = { ...@@ -11,6 +13,12 @@ static struct kvm_events_ops exit_events = {
.name = "VM-EXIT" .name = "VM-EXIT"
}; };
const char *vcpu_id_str = "vcpu_id";
const int decode_str_len = 20;
const char *kvm_exit_reason = "exit_reason";
const char *kvm_entry_trace = "kvm:kvm_entry";
const char *kvm_exit_trace = "kvm:kvm_exit";
/* /*
* For the mmio events, we treat: * For the mmio events, we treat:
* the time of MMIO write: kvm_mmio(KVM_TRACE_MMIO_WRITE...) -> kvm_entry * the time of MMIO write: kvm_mmio(KVM_TRACE_MMIO_WRITE...) -> kvm_entry
...@@ -65,7 +73,7 @@ static void mmio_event_decode_key(struct perf_kvm_stat *kvm __maybe_unused, ...@@ -65,7 +73,7 @@ static void mmio_event_decode_key(struct perf_kvm_stat *kvm __maybe_unused,
struct event_key *key, struct event_key *key,
char *decode) char *decode)
{ {
scnprintf(decode, DECODE_STR_LEN, "%#lx:%s", scnprintf(decode, decode_str_len, "%#lx:%s",
(unsigned long)key->key, (unsigned long)key->key,
key->info == KVM_TRACE_MMIO_WRITE ? "W" : "R"); key->info == KVM_TRACE_MMIO_WRITE ? "W" : "R");
} }
...@@ -109,7 +117,7 @@ static void ioport_event_decode_key(struct perf_kvm_stat *kvm __maybe_unused, ...@@ -109,7 +117,7 @@ static void ioport_event_decode_key(struct perf_kvm_stat *kvm __maybe_unused,
struct event_key *key, struct event_key *key,
char *decode) char *decode)
{ {
scnprintf(decode, DECODE_STR_LEN, "%#llx:%s", scnprintf(decode, decode_str_len, "%#llx:%s",
(unsigned long long)key->key, (unsigned long long)key->key,
key->info ? "POUT" : "PIN"); key->info ? "POUT" : "PIN");
} }
......
...@@ -30,7 +30,6 @@ ...@@ -30,7 +30,6 @@
#include <math.h> #include <math.h>
#ifdef HAVE_KVM_STAT_SUPPORT #ifdef HAVE_KVM_STAT_SUPPORT
#include <asm/kvm_perf.h>
#include "util/kvm-stat.h" #include "util/kvm-stat.h"
void exit_event_get_key(struct perf_evsel *evsel, void exit_event_get_key(struct perf_evsel *evsel,
...@@ -38,12 +37,12 @@ void exit_event_get_key(struct perf_evsel *evsel, ...@@ -38,12 +37,12 @@ void exit_event_get_key(struct perf_evsel *evsel,
struct event_key *key) struct event_key *key)
{ {
key->info = 0; key->info = 0;
key->key = perf_evsel__intval(evsel, sample, KVM_EXIT_REASON); key->key = perf_evsel__intval(evsel, sample, kvm_exit_reason);
} }
bool kvm_exit_event(struct perf_evsel *evsel) bool kvm_exit_event(struct perf_evsel *evsel)
{ {
return !strcmp(evsel->name, KVM_EXIT_TRACE); return !strcmp(evsel->name, kvm_exit_trace);
} }
bool exit_event_begin(struct perf_evsel *evsel, bool exit_event_begin(struct perf_evsel *evsel,
...@@ -59,7 +58,7 @@ bool exit_event_begin(struct perf_evsel *evsel, ...@@ -59,7 +58,7 @@ bool exit_event_begin(struct perf_evsel *evsel,
bool kvm_entry_event(struct perf_evsel *evsel) bool kvm_entry_event(struct perf_evsel *evsel)
{ {
return !strcmp(evsel->name, KVM_ENTRY_TRACE); return !strcmp(evsel->name, kvm_entry_trace);
} }
bool exit_event_end(struct perf_evsel *evsel, bool exit_event_end(struct perf_evsel *evsel,
...@@ -91,7 +90,7 @@ void exit_event_decode_key(struct perf_kvm_stat *kvm, ...@@ -91,7 +90,7 @@ void exit_event_decode_key(struct perf_kvm_stat *kvm,
const char *exit_reason = get_exit_reason(kvm, key->exit_reasons, const char *exit_reason = get_exit_reason(kvm, key->exit_reasons,
key->key); key->key);
scnprintf(decode, DECODE_STR_LEN, "%s", exit_reason); scnprintf(decode, decode_str_len, "%s", exit_reason);
} }
static bool register_kvm_events_ops(struct perf_kvm_stat *kvm) static bool register_kvm_events_ops(struct perf_kvm_stat *kvm)
...@@ -357,7 +356,7 @@ static bool handle_end_event(struct perf_kvm_stat *kvm, ...@@ -357,7 +356,7 @@ static bool handle_end_event(struct perf_kvm_stat *kvm,
time_diff = sample->time - time_begin; time_diff = sample->time - time_begin;
if (kvm->duration && time_diff > kvm->duration) { if (kvm->duration && time_diff > kvm->duration) {
char decode[DECODE_STR_LEN]; char decode[decode_str_len];
kvm->events_ops->decode_key(kvm, &event->key, decode); kvm->events_ops->decode_key(kvm, &event->key, decode);
if (!skip_event(decode)) { if (!skip_event(decode)) {
...@@ -385,7 +384,8 @@ struct vcpu_event_record *per_vcpu_record(struct thread *thread, ...@@ -385,7 +384,8 @@ struct vcpu_event_record *per_vcpu_record(struct thread *thread,
return NULL; return NULL;
} }
vcpu_record->vcpu_id = perf_evsel__intval(evsel, sample, VCPU_ID); vcpu_record->vcpu_id = perf_evsel__intval(evsel, sample,
vcpu_id_str);
thread__set_priv(thread, vcpu_record); thread__set_priv(thread, vcpu_record);
} }
...@@ -574,7 +574,7 @@ static void show_timeofday(void) ...@@ -574,7 +574,7 @@ static void show_timeofday(void)
static void print_result(struct perf_kvm_stat *kvm) static void print_result(struct perf_kvm_stat *kvm)
{ {
char decode[DECODE_STR_LEN]; char decode[decode_str_len];
struct kvm_event *event; struct kvm_event *event;
int vcpu = kvm->trace_vcpu; int vcpu = kvm->trace_vcpu;
...@@ -585,7 +585,7 @@ static void print_result(struct perf_kvm_stat *kvm) ...@@ -585,7 +585,7 @@ static void print_result(struct perf_kvm_stat *kvm)
pr_info("\n\n"); pr_info("\n\n");
print_vcpu_info(kvm); print_vcpu_info(kvm);
pr_info("%*s ", DECODE_STR_LEN, kvm->events_ops->name); pr_info("%*s ", decode_str_len, kvm->events_ops->name);
pr_info("%10s ", "Samples"); pr_info("%10s ", "Samples");
pr_info("%9s ", "Samples%"); pr_info("%9s ", "Samples%");
...@@ -604,7 +604,7 @@ static void print_result(struct perf_kvm_stat *kvm) ...@@ -604,7 +604,7 @@ static void print_result(struct perf_kvm_stat *kvm)
min = get_event_min(event, vcpu); min = get_event_min(event, vcpu);
kvm->events_ops->decode_key(kvm, &event->key, decode); kvm->events_ops->decode_key(kvm, &event->key, decode);
pr_info("%*s ", DECODE_STR_LEN, decode); pr_info("%*s ", decode_str_len, decode);
pr_info("%10llu ", (unsigned long long)ecount); pr_info("%10llu ", (unsigned long long)ecount);
pr_info("%8.2f%% ", (double)ecount / kvm->total_count * 100); pr_info("%8.2f%% ", (double)ecount / kvm->total_count * 100);
pr_info("%8.2f%% ", (double)etime / kvm->total_time * 100); pr_info("%8.2f%% ", (double)etime / kvm->total_time * 100);
......
...@@ -136,5 +136,10 @@ int cpu_isa_init(struct perf_kvm_stat *kvm, const char *cpuid); ...@@ -136,5 +136,10 @@ int cpu_isa_init(struct perf_kvm_stat *kvm, const char *cpuid);
extern const char * const kvm_events_tp[]; extern const char * const kvm_events_tp[];
extern struct kvm_reg_events_ops kvm_reg_events_ops[]; extern struct kvm_reg_events_ops kvm_reg_events_ops[];
extern const char * const kvm_skip_events[]; extern const char * const kvm_skip_events[];
extern const char *vcpu_id_str;
extern const int decode_str_len;
extern const char *kvm_exit_reason;
extern const char *kvm_entry_trace;
extern const char *kvm_exit_trace;
#endif /* __PERF_KVM_STAT_H */ #endif /* __PERF_KVM_STAT_H */
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