Re: [Intel-gfx] [PATCH] drm/i915: Enable full PPGTT on gen7

2014-09-09 Thread Chris Wilson
On Tue, Sep 09, 2014 at 02:55:15PM +0100, Chris Wilson wrote: > Oh, wait. It just uses the default-context on each fd, i.e. doesn't > actually perform the context restores. Will check back later... A bit more experimentation (igt/gem_ppgtt/bcs-vs-rcs-ctxN) and I am happy that this was a non-issue.

Re: [Intel-gfx] [PATCH] drm/i915: Enable full PPGTT on gen7

2014-09-09 Thread Chris Wilson
On Tue, Sep 09, 2014 at 05:12:23PM +0200, Daniel Vetter wrote: > On Tue, Sep 09, 2014 at 02:55:15PM +0100, Chris Wilson wrote: > > Oh, wait. It just uses the default-context on each fd, i.e. doesn't > > actually perform the context restores. Will check back later... > > We should throw a ->switch_

Re: [Intel-gfx] [PATCH] drm/i915: Enable full PPGTT on gen7

2014-09-09 Thread Daniel Vetter
On Tue, Sep 09, 2014 at 02:55:15PM +0100, Chris Wilson wrote: > On Tue, Sep 09, 2014 at 04:30:06PM +0300, Ville Syrjälä wrote: > > On Tue, Sep 09, 2014 at 02:07:24PM +0100, Chris Wilson wrote: > > > On Tue, Sep 09, 2014 at 12:41:34PM +, Thierry, Michel wrote: > > > > > > > > > > > > On Tue, S

Re: [Intel-gfx] [PATCH] drm/i915: Enable full PPGTT on gen7

2014-09-09 Thread Chris Wilson
On Tue, Sep 09, 2014 at 04:30:06PM +0300, Ville Syrjälä wrote: > On Tue, Sep 09, 2014 at 02:07:24PM +0100, Chris Wilson wrote: > > On Tue, Sep 09, 2014 at 12:41:34PM +, Thierry, Michel wrote: > > > > > > > > > On Tue, Sep 9, 2014 at 1:34 PM, Ville Syrjälä > > > wrote: > > > > On Tue, Sep 09

Re: [Intel-gfx] [PATCH] drm/i915: Enable full PPGTT on gen7

2014-09-09 Thread Ville Syrjälä
On Tue, Sep 09, 2014 at 02:07:24PM +0100, Chris Wilson wrote: > On Tue, Sep 09, 2014 at 12:41:34PM +, Thierry, Michel wrote: > > > > > > On Tue, Sep 9, 2014 at 1:34 PM, Ville Syrjälä > > wrote: > > > On Tue, Sep 09, 2014 at 12:57:11PM +0100, Chris Wilson wrote: > > > > On Fri, Sep 05, 2014

Re: [Intel-gfx] [PATCH] drm/i915: Enable full PPGTT on gen7

2014-09-09 Thread Chris Wilson
On Tue, Sep 09, 2014 at 12:41:34PM +, Thierry, Michel wrote: > > > On Tue, Sep 9, 2014 at 1:34 PM, Ville Syrjälä > wrote: > > On Tue, Sep 09, 2014 at 12:57:11PM +0100, Chris Wilson wrote: > > > On Fri, Sep 05, 2014 at 02:13:16PM +0100, Michel Thierry wrote: > > > > Use full PPGTT as the def

Re: [Intel-gfx] [PATCH] drm/i915: Enable full PPGTT on gen7

2014-09-09 Thread Thierry, Michel
On Tue, Sep 9, 2014 at 1:34 PM, Ville Syrjälä wrote: > On Tue, Sep 09, 2014 at 12:57:11PM +0100, Chris Wilson wrote: > > On Fri, Sep 05, 2014 at 02:13:16PM +0100, Michel Thierry wrote: > > > Use full PPGTT as the default option in gen7. > > > Note that aliasing PPGTT is the default option for g

Re: [Intel-gfx] [PATCH] drm/i915: Enable full PPGTT on gen7

2014-09-09 Thread Ville Syrjälä
On Tue, Sep 09, 2014 at 12:57:11PM +0100, Chris Wilson wrote: > On Fri, Sep 05, 2014 at 02:13:16PM +0100, Michel Thierry wrote: > > Use full PPGTT as the default option in gen7. > > Note that aliasing PPGTT is the default option for gen8 (see HAS_PPGTT). > > > > This may well come back to bite me

Re: [Intel-gfx] [PATCH] drm/i915: Enable full PPGTT on gen7

2014-09-09 Thread Chris Wilson
On Fri, Sep 05, 2014 at 02:13:16PM +0100, Michel Thierry wrote: > Use full PPGTT as the default option in gen7. > Note that aliasing PPGTT is the default option for gen8 (see HAS_PPGTT). > > This may well come back to bite me later. Indeed. So something I spotted was that bspec mentions that the

Re: [Intel-gfx] [PATCH] drm/i915: Enable full PPGTT on gen7

2014-09-08 Thread Daniel Vetter
On Fri, Sep 05, 2014 at 02:13:16PM +0100, Michel Thierry wrote: > Use full PPGTT as the default option in gen7. > Note that aliasing PPGTT is the default option for gen8 (see HAS_PPGTT). > > This may well come back to bite me later. > > Signed-off-by: Michel Thierry Queued for -next, thanks for

[Intel-gfx] [PATCH] drm/i915: Enable full PPGTT on gen7

2014-09-05 Thread Michel Thierry
Use full PPGTT as the default option in gen7. Note that aliasing PPGTT is the default option for gen8 (see HAS_PPGTT). This may well come back to bite me later. Signed-off-by: Michel Thierry --- drivers/gpu/drm/i915/i915_gem_gtt.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --gi