On Mon, Jun 8, 2015 at 3:50 PM, Ben Widawsky <benjamin.widaw...@intel.com> 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. > > Fixes piglit test: > arb_pipeline_statistics_query-frag > > Signed-off-by: Ben Widawsky <b...@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 >
LGTM. Tested on Skylake. Reviewed-by: Anuj Phogat <anuj.pho...@gmail.com> _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
