diff options
author | Adrian Hunter <adrian.hunter@intel.com> | 2018-05-31 06:23:44 -0400 |
---|---|---|
committer | Arnaldo Carvalho de Melo <acme@redhat.com> | 2018-06-06 11:52:08 -0400 |
commit | dd27b87ab5fcf3ea1c060b5e3ab5d31cc78e9f4c (patch) | |
tree | 15014b1ee29eef54263f7b69d510290ce948e07e /tools/perf | |
parent | bd2e49ec48feb1855f7624198849eea4610e2286 (diff) |
perf intel-pt: Fix MTC timing after overflow
On some platforms, overflows will clear before MTC wraparound, and there
is no following TSC/TMA packet. In that case the previous TMA is valid.
Since there will be a valid TMA either way, stop setting 'have_tma' to
false upon overflow.
Signed-off-by: Adrian Hunter <adrian.hunter@intel.com>
Cc: stable@vger.kernel.org
Link: http://lkml.kernel.org/r/1527762225-26024-4-git-send-email-adrian.hunter@intel.com
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Diffstat (limited to 'tools/perf')
-rw-r--r-- | tools/perf/util/intel-pt-decoder/intel-pt-decoder.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/tools/perf/util/intel-pt-decoder/intel-pt-decoder.c b/tools/perf/util/intel-pt-decoder/intel-pt-decoder.c index e5eb91777383..881d7c5e5e2a 100644 --- a/tools/perf/util/intel-pt-decoder/intel-pt-decoder.c +++ b/tools/perf/util/intel-pt-decoder/intel-pt-decoder.c | |||
@@ -1376,7 +1376,6 @@ static int intel_pt_overflow(struct intel_pt_decoder *decoder) | |||
1376 | { | 1376 | { |
1377 | intel_pt_log("ERROR: Buffer overflow\n"); | 1377 | intel_pt_log("ERROR: Buffer overflow\n"); |
1378 | intel_pt_clear_tx_flags(decoder); | 1378 | intel_pt_clear_tx_flags(decoder); |
1379 | decoder->have_tma = false; | ||
1380 | decoder->cbr = 0; | 1379 | decoder->cbr = 0; |
1381 | decoder->timestamp_insn_cnt = 0; | 1380 | decoder->timestamp_insn_cnt = 0; |
1382 | decoder->pkt_state = INTEL_PT_STATE_ERR_RESYNC; | 1381 | decoder->pkt_state = INTEL_PT_STATE_ERR_RESYNC; |