Re: [Mesa-dev] [PATCH] i965: Fix sampler state pointer adjustment for nonconst samplers

2014-11-05 Thread Chris Forbes
I haven't bisected, but this will have got broken when we changed the execsize guessing. Previously we would have picked a larger execsize here, but now it is and(1). This is the d/u indexing case, so sampler_index is a vec8/vec16 grf. Most of the other cases are immediates, or individual vec1s al

Re: [Mesa-dev] [PATCH] i965: Fix sampler state pointer adjustment for nonconst samplers

2014-11-04 Thread Kenneth Graunke
On Wednesday, November 05, 2014 06:41:13 AM Chris Forbes wrote: > This started hitting an assertion recently. Only affects Haswell > (Ivybridge doesn't support this meddling with the sampler state pointer, > and ARB_gpu_shader5 is not enabled yet on Broadwell) > > 14 Piglits crash->pass. > > Sign

[Mesa-dev] [PATCH] i965: Fix sampler state pointer adjustment for nonconst samplers

2014-11-04 Thread Chris Forbes
This started hitting an assertion recently. Only affects Haswell (Ivybridge doesn't support this meddling with the sampler state pointer, and ARB_gpu_shader5 is not enabled yet on Broadwell) 14 Piglits crash->pass. Signed-off-by: Chris Forbes --- src/mesa/drivers/dri/i965/brw_eu_emit.c | 2 +-