Re: [Intel-gfx] [PATCH] drm/i915: Remove Braswell GGTT update w/a

2017-02-23 Thread Chris Wilson
On Wed, Feb 22, 2017 at 01:27:41PM +0200, Joonas Lahtinen wrote: > On ma, 2017-02-20 at 12:47 +, Chris Wilson wrote: > > Testing with concurrent GGTT accesses no longer show the coherency > > problems from yonder, commit 5bab6f60cb4d ("drm/i915: Serialise updates > > to GGTT with access through

Re: [Intel-gfx] [PATCH] drm/i915: Remove Braswell GGTT update w/a

2017-02-22 Thread Chris Wilson
On Wed, Feb 22, 2017 at 06:02:46PM +, Chris Wilson wrote: > On Wed, Feb 22, 2017 at 11:39:30AM +, Chris Wilson wrote: > > On Wed, Feb 22, 2017 at 01:27:41PM +0200, Joonas Lahtinen wrote: > > > On ma, 2017-02-20 at 12:47 +, Chris Wilson wrote: > > > > Testing with concurrent GGTT accesse

Re: [Intel-gfx] [PATCH] drm/i915: Remove Braswell GGTT update w/a

2017-02-22 Thread Chris Wilson
On Wed, Feb 22, 2017 at 11:39:30AM +, Chris Wilson wrote: > On Wed, Feb 22, 2017 at 01:27:41PM +0200, Joonas Lahtinen wrote: > > On ma, 2017-02-20 at 12:47 +, Chris Wilson wrote: > > > Testing with concurrent GGTT accesses no longer show the coherency > > > problems from yonder, commit 5bab

Re: [Intel-gfx] [PATCH] drm/i915: Remove Braswell GGTT update w/a

2017-02-22 Thread Chris Wilson
On Wed, Feb 22, 2017 at 01:27:41PM +0200, Joonas Lahtinen wrote: > On ma, 2017-02-20 at 12:47 +, Chris Wilson wrote: > > Testing with concurrent GGTT accesses no longer show the coherency > > problems from yonder, commit 5bab6f60cb4d ("drm/i915: Serialise updates > > to GGTT with access through

Re: [Intel-gfx] [PATCH] drm/i915: Remove Braswell GGTT update w/a

2017-02-22 Thread Joonas Lahtinen
On ma, 2017-02-20 at 12:47 +, Chris Wilson wrote: > Testing with concurrent GGTT accesses no longer show the coherency > problems from yonder, commit 5bab6f60cb4d ("drm/i915: Serialise updates > to GGTT with access through GGTT on Braswell"). My presumption is that > the root cause was more lik

[Intel-gfx] [PATCH] drm/i915: Remove Braswell GGTT update w/a

2017-02-20 Thread Chris Wilson
Testing with concurrent GGTT accesses no longer show the coherency problems from yonder, commit 5bab6f60cb4d ("drm/i915: Serialise updates to GGTT with access through GGTT on Braswell"). My presumption is that the root cause was more likely fixed by commit 3b5724d702ef ("drm/i915: Wait for writes t