Mesa (master): intel/perf: fix EuThreadsCount value in performance equations

[GitLab Mirror]

Module: Mesa
Branch: master
Commit: 0430c6d18a90549b1dd40ea29a0c28d1a8e3a47d
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=0430c6d18a90549b1dd40ea29a0c28d1a8e3a47d

Author: Lionel Landwerlin <lionel.g.landwerlin at intel.com>
Date:   Wed Jun  5 11:49:06 2019 +0300

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 <lionel.g.landwerlin at intel.com>
Fixes: 1fc7b951278428 ("i965: Add Gen8+ INTEL_performance_query support")
Reviewed-by: Kenneth Graunke <kenneth at whitecape.org>

---

 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