Re: [Intel-gfx] [PATCH 3/3] drm/i915: Force sync command ordering

2011-10-11 Thread Chris Wilson
On Tue, 11 Oct 2011 12:33:05 -0700, Ben Widawsky wrote: > On Tue, 11 Oct 2011 12:30:36 -0700 > Ben Widawsky wrote: > > > On Tue, 11 Oct 2011 12:18:15 -0700 > > Kenneth Graunke wrote: > > > > > > I might only enable this on Gen7 for now, unless it actually fixes > > > something on Sandybridge.

Re: [Intel-gfx] [PATCH 3/3] drm/i915: Force sync command ordering

2011-10-11 Thread Ben Widawsky
On Tue, 11 Oct 2011 12:30:36 -0700 Ben Widawsky wrote: > On Tue, 11 Oct 2011 12:18:15 -0700 > Kenneth Graunke wrote: > > > > I might only enable this on Gen7 for now, unless it actually fixes > > something on Sandybridge. It's not listed as required for Gen6. > > I would prefer to keep for ge

Re: [Intel-gfx] [PATCH 3/3] drm/i915: Force sync command ordering

2011-10-11 Thread Ben Widawsky
On Tue, 11 Oct 2011 12:18:15 -0700 Kenneth Graunke wrote: > On 10/11/2011 11:31 AM, Ben Widawsky wrote: > > This will strongly order synchronization commands with respect to 3d > > state and 3d primitive commands. AFAIK, this shouldn't impact anything > > as these sync commands are all for privil

Re: [Intel-gfx] [PATCH 3/3] drm/i915: Force sync command ordering

2011-10-11 Thread Kenneth Graunke
On 10/11/2011 11:31 AM, Ben Widawsky wrote: > This will strongly order synchronization commands with respect to 3d > state and 3d primitive commands. AFAIK, this shouldn't impact anything > as these sync commands are all for privileged (or ppgtt) batches only, > so user space should not be relying

Re: [Intel-gfx] [PATCH 3/3] drm/i915: Force sync command ordering

2011-10-11 Thread Chris Wilson
On Tue, 11 Oct 2011 11:31:56 -0700, Ben Widawsky wrote: > This will strongly order synchronization commands with respect to 3d > state and 3d primitive commands. AFAIK, this shouldn't impact anything > as these sync commands are all for privileged (or ppgtt) batches only, > so user space should no

Re: [Intel-gfx] [PATCH 3/3] drm/i915: Force sync command ordering

2011-10-11 Thread Ben Widawsky
On Tue, 11 Oct 2011 11:31:56 -0700 Ben Widawsky wrote: > This will strongly order synchronization commands with respect to 3d > state and 3d primitive commands. AFAIK, this shouldn't impact anything > as these sync commands are all for privileged (or ppgtt) batches only, > so user space should n

[Intel-gfx] [PATCH 3/3] drm/i915: Force sync command ordering

2011-10-11 Thread Ben Widawsky
This will strongly order synchronization commands with respect to 3d state and 3d primitive commands. AFAIK, this shouldn't impact anything as these sync commands are all for privileged (or ppgtt) batches only, so user space should not be relying on this, and the kernel wouldn't be relying on 3d st