Re: [Mesa-dev] [PATCH] i965: add ARB_clip_control on ivybridge/haswell.

2014-10-30 Thread Mathias Fröhlich
Hi, On Tuesday, October 28, 2014 19:43:23 Neil Roberts wrote: > I think if you change the clip depth mode without changing the clip > origin then only the _NEW_VIEWPORT state would be set. Does that mean we > have to add _NEW_VIEWPORT to the state for the gen7_clip_state atom as > well to make su

Re: [Mesa-dev] [PATCH] i965: add ARB_clip_control on ivybridge/haswell.

2014-10-28 Thread Mathias Fröhlich
Hi, On Tuesday, October 28, 2014 19:43:23 Neil Roberts wrote: > I think if you change the clip depth mode without changing the clip > origin then only the _NEW_VIEWPORT state would be set. Does that mean we > have to add _NEW_VIEWPORT to the state for the gen7_clip_state atom as > well to make su

Re: [Mesa-dev] [PATCH] i965: add ARB_clip_control on ivybridge/haswell.

2014-10-28 Thread Neil Roberts
I think if you change the clip depth mode without changing the clip origin then only the _NEW_VIEWPORT state would be set. Does that mean we have to add _NEW_VIEWPORT to the state for the gen7_clip_state atom as well to make sure that upload_clip_state is called? There is also some state for the f

[Mesa-dev] [PATCH] i965: add ARB_clip_control on ivybridge/haswell.

2014-10-27 Thread Dave Airlie
From: Dave Airlie I've only tested this on IVB so far, but it passes the two piglit tests. Signed-off-by: Dave Airlie --- src/mesa/drivers/dri/i965/gen6_clip_state.c | 2 +- src/mesa/drivers/dri/i965/gen7_sf_state.c| 2 +- src/mesa/drivers/dri/i965/intel_extensions.c | 4 +++- 3 files cha