Mesa (master): i965/gen9: Use raw PS invocation count for queries

Ben Widawsky bwidawsk at kemper.freedesktop.org
Tue Jun 9 18:17:55 UTC 2015


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

Author: Ben Widawsky <benjamin.widawsky at intel.com>
Date:   Mon Jun  8 14:52:07 2015 -0700

i965/gen9: Use raw PS invocation count for queries

Previously the number needed to be divided by 4 to get the proper results. Now
the hardware does the right thing. Through experimentation it seems Braswell
(CHV) does also need the division by 4.

Fixes piglit test:
arb_pipeline_statistics_query-frag

Signed-off-by: Ben Widawsky <ben at bwidawsk.net>
Reviewed-by: Anuj Phogat <anuj.phogat at gmail.com>

---

 src/mesa/drivers/dri/i965/gen6_queryobj.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/mesa/drivers/dri/i965/gen6_queryobj.c b/src/mesa/drivers/dri/i965/gen6_queryobj.c
index 6431ed5..ba5c944 100644
--- a/src/mesa/drivers/dri/i965/gen6_queryobj.c
+++ b/src/mesa/drivers/dri/i965/gen6_queryobj.c
@@ -246,7 +246,7 @@ gen6_queryobj_get_results(struct gl_context *ctx,
        * and correctly emitted the number of pixel shader invocations, but,
        * whomever forgot to undo the multiply by 4.
        */
-      if (brw->gen >= 8 || brw->is_haswell)
+      if (brw->gen == 8 || brw->is_haswell)
          query->Base.Result /= 4;
       break;
 




More information about the mesa-commit mailing list