提交 a10eb530 编写于 作者: A Adrian Hunter 提交者: Arnaldo Carvalho de Melo

perf intel-pt/bts: Do not swap when synthesizing samples

Both 'perf inject' and internal tools consume cpu endian samples, so
there is never a need to do any swapping when synthesizing samples.
Signed-off-by: NAdrian Hunter <adrian.hunter@intel.com>
Acked-by: NJiri Olsa <jolsa@redhat.com>
Link: http://lkml.kernel.org/r/1516108492-21401-2-git-send-email-adrian.hunter@intel.comSigned-off-by: NArnaldo Carvalho de Melo <acme@redhat.com>
上级 a72594ca
...@@ -67,7 +67,6 @@ struct intel_bts { ...@@ -67,7 +67,6 @@ struct intel_bts {
u64 branches_sample_type; u64 branches_sample_type;
u64 branches_id; u64 branches_id;
size_t branches_event_size; size_t branches_event_size;
bool synth_needs_swap;
unsigned long num_events; unsigned long num_events;
}; };
...@@ -303,8 +302,7 @@ static int intel_bts_synth_branch_sample(struct intel_bts_queue *btsq, ...@@ -303,8 +302,7 @@ static int intel_bts_synth_branch_sample(struct intel_bts_queue *btsq,
event.sample.header.size = bts->branches_event_size; event.sample.header.size = bts->branches_event_size;
ret = perf_event__synthesize_sample(&event, ret = perf_event__synthesize_sample(&event,
bts->branches_sample_type, bts->branches_sample_type,
0, &sample, 0, &sample, false);
bts->synth_needs_swap);
if (ret) if (ret)
return ret; return ret;
} }
...@@ -841,8 +839,6 @@ static int intel_bts_synth_events(struct intel_bts *bts, ...@@ -841,8 +839,6 @@ static int intel_bts_synth_events(struct intel_bts *bts,
__perf_evsel__sample_size(attr.sample_type); __perf_evsel__sample_size(attr.sample_type);
} }
bts->synth_needs_swap = evsel->needs_swap;
return 0; return 0;
} }
......
...@@ -104,8 +104,6 @@ struct intel_pt { ...@@ -104,8 +104,6 @@ struct intel_pt {
u64 pwrx_id; u64 pwrx_id;
u64 cbr_id; u64 cbr_id;
bool synth_needs_swap;
u64 tsc_bit; u64 tsc_bit;
u64 mtc_bit; u64 mtc_bit;
u64 mtc_freq_bits; u64 mtc_freq_bits;
...@@ -1101,11 +1099,10 @@ static void intel_pt_prep_b_sample(struct intel_pt *pt, ...@@ -1101,11 +1099,10 @@ static void intel_pt_prep_b_sample(struct intel_pt *pt,
} }
static int intel_pt_inject_event(union perf_event *event, static int intel_pt_inject_event(union perf_event *event,
struct perf_sample *sample, u64 type, struct perf_sample *sample, u64 type)
bool swapped)
{ {
event->header.size = perf_event__sample_event_size(sample, type, 0); event->header.size = perf_event__sample_event_size(sample, type, 0);
return perf_event__synthesize_sample(event, type, 0, sample, swapped); return perf_event__synthesize_sample(event, type, 0, sample, false);
} }
static inline int intel_pt_opt_inject(struct intel_pt *pt, static inline int intel_pt_opt_inject(struct intel_pt *pt,
...@@ -1115,7 +1112,7 @@ static inline int intel_pt_opt_inject(struct intel_pt *pt, ...@@ -1115,7 +1112,7 @@ static inline int intel_pt_opt_inject(struct intel_pt *pt,
if (!pt->synth_opts.inject) if (!pt->synth_opts.inject)
return 0; return 0;
return intel_pt_inject_event(event, sample, type, pt->synth_needs_swap); return intel_pt_inject_event(event, sample, type);
} }
static int intel_pt_deliver_synth_b_event(struct intel_pt *pt, static int intel_pt_deliver_synth_b_event(struct intel_pt *pt,
...@@ -2329,8 +2326,6 @@ static int intel_pt_synth_events(struct intel_pt *pt, ...@@ -2329,8 +2326,6 @@ static int intel_pt_synth_events(struct intel_pt *pt,
id += 1; id += 1;
} }
pt->synth_needs_swap = evsel->needs_swap;
return 0; return 0;
} }
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册