Re: [Intel-gfx] [PATCH 1/3] i915: Remove implied length of 2 from GFX_OP_PIPE_CONTROL #define.

2011-10-11 Thread Daniel Vetter
Switching to PIPE_CONTROL with the snb workaround seems to fix the hangs I see when running cairo-perf-traces. Tested-by: Daniel Vetter daniel.vet...@ffwll.ch -- Daniel Vetter Mail: dan...@ffwll.ch Mobile: +41 (0)79 365 57 48 ___ Intel-gfx mailing list

Re: [Intel-gfx] [PATCH 1/3] i915: Remove implied length of 2 from GFX_OP_PIPE_CONTROL #define.

2011-10-11 Thread Jesse Barnes
On Tue, 11 Oct 2011 13:09:17 +0200 Daniel Vetter dan...@ffwll.ch wrote: Switching to PIPE_CONTROL with the snb workaround seems to fix the hangs I see when running cairo-perf-traces. Tested-by: Daniel Vetter daniel.vet...@ffwll.ch Ah wow, so it actually fixes a bug. Let's get this upstream

Re: [Intel-gfx] [PATCH 1/3] i915: Remove implied length of 2 from GFX_OP_PIPE_CONTROL #define.

2011-10-11 Thread Ben Widawsky
On Tue, 11 Oct 2011 10:20:04 -0700 Jesse Barnes jbar...@virtuousgeek.org wrote: On Tue, 11 Oct 2011 13:09:17 +0200 Daniel Vetter dan...@ffwll.ch wrote: Switching to PIPE_CONTROL with the snb workaround seems to fix the hangs I see when running cairo-perf-traces. Tested-by: Daniel

Re: [Intel-gfx] [PATCH 1/3] i915: Remove implied length of 2 from GFX_OP_PIPE_CONTROL #define.

2011-10-11 Thread Jesse Barnes
On Tue, 11 Oct 2011 11:39:36 -0700 Ben Widawsky b...@bwidawsk.net wrote: On Tue, 11 Oct 2011 10:20:04 -0700 Jesse Barnes jbar...@virtuousgeek.org wrote: On Tue, 11 Oct 2011 13:09:17 +0200 Daniel Vetter dan...@ffwll.ch wrote: Switching to PIPE_CONTROL with the snb workaround seems to

Re: [Intel-gfx] [PATCH 1/3] i915: Remove implied length of 2 from GFX_OP_PIPE_CONTROL #define.

2011-10-11 Thread Keith Packard
On Tue, 11 Oct 2011 11:53:56 -0700, Jesse Barnes jbar...@virtuousgeek.org wrote: Keith can you pick this up? Yup, I'll make it work. -- keith.pack...@intel.com pgpEirXYXMjwW.pgp Description: PGP signature ___ Intel-gfx mailing list

[Intel-gfx] [PATCH 1/3] i915: Remove implied length of 2 from GFX_OP_PIPE_CONTROL #define.

2011-10-03 Thread Kenneth Graunke
Not all PIPE_CONTROLs have a length of 2, so remove it from the #define and make each invocation specify the desired length. Signed-off-by: Kenneth Graunke kenn...@whitecape.org --- drivers/gpu/drm/i915/i915_reg.h |2 +- drivers/gpu/drm/i915/intel_ringbuffer.c |4 ++-- 2 files