Re: [Mesa-dev] [PATCH] i965/hsw: Initialize SLM index in state register

2016-03-06 Thread Kenneth Graunke
On Saturday, March 5, 2016 4:02:17 PM PST Jordan Justen wrote: > On 2016-03-04 16:07:08, Matt Turner wrote: > > Alternatively, if overwriting FFTID is bad, instead of emitting > > > >shr(1) sr0.1:UD g0.0:UD 16UD > > > > you could write only the low word: > > > >mov(1) sr0.2:UW g0.1:

Re: [Mesa-dev] [PATCH] i965/hsw: Initialize SLM index in state register

2016-03-05 Thread Matt Turner
On Sat, Mar 5, 2016 at 4:02 PM, Jordan Justen wrote: > On 2016-03-04 16:07:08, Matt Turner wrote: >> Alternatively, if overwriting FFTID is bad, instead of emitting >> >>shr(1) sr0.1:UD g0.0:UD 16UD >> >> you could write only the low word: >> >>mov(1) sr0.2:UW g0.1:UW >> >> or maybe u

Re: [Mesa-dev] [PATCH] i965/hsw: Initialize SLM index in state register

2016-03-05 Thread Jordan Justen
On 2016-03-04 16:07:08, Matt Turner wrote: > Alternatively, if overwriting FFTID is bad, instead of emitting > >shr(1) sr0.1:UD g0.0:UD 16UD > > you could write only the low word: > >mov(1) sr0.2:UW g0.1:UW > > or maybe use an AND to ensure you're writing zeros to the MBZ bits of >

Re: [Mesa-dev] [PATCH] i965/hsw: Initialize SLM index in state register

2016-03-05 Thread Kenneth Graunke
On Friday, March 4, 2016 4:07:08 PM PST Matt Turner wrote: > On Mon, Feb 22, 2016 at 11:35 AM, Jordan Justen > wrote: > > For Haswell, we need to initialize the SLM index in the state > > register. This can be copied out of the CS header dword 0. > > > > Fixes piglit: arb_compute_shader/execution/

Re: [Mesa-dev] [PATCH] i965/hsw: Initialize SLM index in state register

2016-03-04 Thread Matt Turner
On Mon, Feb 22, 2016 at 11:35 AM, Jordan Justen wrote: > For Haswell, we need to initialize the SLM index in the state > register. This can be copied out of the CS header dword 0. > > Fixes piglit: arb_compute_shader/execution/shared-atomics.shader_test > > Signed-off-by: Jordan Justen > Cc: Ilia

Re: [Mesa-dev] [PATCH] i965/hsw: Initialize SLM index in state register

2016-02-22 Thread Ilia Mirkin
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=94081 Tested-by: Ilia Mirkin Seems to fix all the shared_var fails in dEQP here! On Mon, Feb 22, 2016 at 2:35 PM, Jordan Justen wrote: > For Haswell, we need to initialize the SLM index in the state > register. This can be copied out of the

[Mesa-dev] [PATCH] i965/hsw: Initialize SLM index in state register

2016-02-22 Thread Jordan Justen
For Haswell, we need to initialize the SLM index in the state register. This can be copied out of the CS header dword 0. Fixes piglit: arb_compute_shader/execution/shared-atomics.shader_test Signed-off-by: Jordan Justen Cc: Ilia Mirkin Cc: "11.2" --- src/mesa/drivers/dri/i965/brw_fs.cpp | 7