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

[GitLab Mirror]

Module: Mesa
Branch: staging/19.0
Commit: 5375846e8d1ea643c13d5809dba55c68f6a9440b
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=5375846e8d1ea643c13d5809dba55c68f6a9440b

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>
(cherry picked from commit 0430c6d18a90549b1dd40ea29a0c28d1a8e3a47d)

---

 src/mesa/drivers/dri/i965/brw_performance_query.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/src/mesa/drivers/dri/i965/brw_performance_query.c b/src/mesa/drivers/dri/i965/brw_performance_query.c
index 10e3d024f17..c10108f38f3 100644
--- a/src/mesa/drivers/dri/i965/brw_performance_query.c
+++ b/src/mesa/drivers/dri/i965/brw_performance_query.c
@@ -1990,8 +1990,7 @@ compute_topology_builtins(struct brw_context *brw)
    for (int i = 0; i < sizeof(devinfo->eu_masks); i++)
       brw->perfquery.sys_vars.n_eus += util_bitcount(devinfo->eu_masks[i]);
 
-   brw->perfquery.sys_vars.eu_threads_count =
-      brw->perfquery.sys_vars.n_eus * devinfo->num_thread_per_eu;
+   brw->perfquery.sys_vars.eu_threads_count = devinfo->num_thread_per_eu;
 
    /* At the moment the subslice mask builtin has groups of 3bits for each
     * slice.