[Mesa-dev] [PATCH] i965/gen9: Use raw PS invocation count for queries
ben at bwidawsk.net
Mon Jun 8 15:54:48 PDT 2015
On Mon, Jun 08, 2015 at 03:50:19PM -0700, Ben Widawsky wrote:
> 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 this.
I need to reword this, which I've done locally. BSW requires division by 4.
> Fixes piglit test:
> Signed-off-by: Ben Widawsky <ben at bwidawsk.net>
> 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;
More information about the mesa-dev