From: Lionel Landwerlin Date: Wed, 5 Jun 2019 08:49:06 +0000 (+0300) Subject: intel/perf: fix EuThreadsCount value in performance equations X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=0430c6d18a90549b1dd40ea29a0c28d1a8e3a47d;p=mesa.git 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 --- 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