[Mesa-dev] [PATCH] i965/gen9: Use raw PS invocation count for queries

[Ben Widawsky]

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:
> arb_pipeline_statistics_query-frag
> 
> 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;
>        break;
>  
> -- 
> 2.4.2
>