From 61fef1ed726b69b28ece60fb10bd5a65fcefb296 Mon Sep 17 00:00:00 2001 From: Lionel Landwerlin Date: Tue, 28 Jun 2022 14:20:02 +0000 Subject: [PATCH] intel/perf: fix B/C counters accumulation in non query mode When we're not using queries, all the counters from the MI_REPORT_PERF_COUNT are available. This is the case when using perfetto with the global pps datasource that capture global counter values. Signed-off-by: Lionel Landwerlin Fixes: 8750f43a9077 ("intel/perf: add performance query layout using MI_SRM") Reviewed-by: Ivan Briano Part-of: --- src/intel/perf/intel_perf.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/intel/perf/intel_perf.c b/src/intel/perf/intel_perf.c index e1b482c9d04..07e6c0fe512 100644 --- a/src/intel/perf/intel_perf.c +++ b/src/intel/perf/intel_perf.c @@ -1128,7 +1128,8 @@ intel_perf_query_result_accumulate(struct intel_perf_query_result *result, result->accumulator + query->a_offset + 32 + i); } - if (can_use_mi_rpc_bc_counters(&query->perf->devinfo)) { + if (can_use_mi_rpc_bc_counters(&query->perf->devinfo) || + !query->perf->sys_vars.query_mode) { /* 8x 32bit B counters */ for (i = 0; i < 8; i++) { accumulate_uint32(start + 48 + i, end + 48 + i,