From 0430c6d18a90549b1dd40ea29a0c28d1a8e3a47d Mon Sep 17 00:00:00 2001 From: Lionel Landwerlin Date: Wed, 5 Jun 2019 11:49:06 +0300 Subject: [PATCH] intel/perf: fix EuThreadsCount value in performance equations EuThreadsCount is supposed to be the number of threads per EU, not the total number of threads in the whole device. Signed-off-by: Lionel Landwerlin Fixes: 1fc7b951278428 ("i965: Add Gen8+ INTEL_performance_query support") Reviewed-by: Kenneth Graunke --- src/intel/perf/gen_perf.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/intel/perf/gen_perf.c b/src/intel/perf/gen_perf.c index 66497e457d9..cd440f7690d 100644 --- a/src/intel/perf/gen_perf.c +++ b/src/intel/perf/gen_perf.c @@ -298,8 +298,7 @@ compute_topology_builtins(struct gen_perf *perf, for (int i = 0; i < sizeof(devinfo->eu_masks); i++) perf->sys_vars.n_eus += __builtin_popcount(devinfo->eu_masks[i]); - perf->sys_vars.eu_threads_count = - perf->sys_vars.n_eus * devinfo->num_thread_per_eu; + perf->sys_vars.eu_threads_count = devinfo->num_thread_per_eu; /* The subslice mask builtin contains bits for all slices. Prior to Gen11 * it had groups of 3bits for each slice, on Gen11 it's 8bits for each -- 2.30.2