瀏覽代碼

perf intel-pt: Remove redundant initial_skip checks

'initial_skip' is checked inside the sample synthesis functions which means
it is actually being done twice for 'instructions' and 'transactions'
samples. Remove the redundant checks.

Signed-off-by: Adrian Hunter <adrian.hunter@intel.com>
Cc: Andi Kleen <ak@linux.intel.com>
Link: http://lkml.kernel.org/r/1495786658-18063-17-git-send-email-adrian.hunter@intel.com
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Adrian Hunter 8 年之前
父節點
當前提交
5da3b23b3b
共有 1 個文件被更改,包括 2 次插入6 次删除
  1. 2 6
      tools/perf/util/intel-pt.c

+ 2 - 6
tools/perf/util/intel-pt.c

@@ -1322,18 +1322,14 @@ static int intel_pt_sample(struct intel_pt_queue *ptq)
 	ptq->have_sample = false;
 	ptq->have_sample = false;
 
 
 	if (pt->sample_instructions &&
 	if (pt->sample_instructions &&
-	    (state->type & INTEL_PT_INSTRUCTION) &&
-	    (!pt->synth_opts.initial_skip ||
-	     pt->num_events++ >= pt->synth_opts.initial_skip)) {
+	    (state->type & INTEL_PT_INSTRUCTION)) {
 		err = intel_pt_synth_instruction_sample(ptq);
 		err = intel_pt_synth_instruction_sample(ptq);
 		if (err)
 		if (err)
 			return err;
 			return err;
 	}
 	}
 
 
 	if (pt->sample_transactions &&
 	if (pt->sample_transactions &&
-	    (state->type & INTEL_PT_TRANSACTION) &&
-	    (!pt->synth_opts.initial_skip ||
-	     pt->num_events++ >= pt->synth_opts.initial_skip)) {
+	    (state->type & INTEL_PT_TRANSACTION)) {
 		err = intel_pt_synth_transaction_sample(ptq);
 		err = intel_pt_synth_transaction_sample(ptq);
 		if (err)
 		if (err)
 			return err;
 			return err;