[Intel-gfx] [PATCH] drm/i915/ppgtt: Load address space after mi_set_context

2014-09-12 Thread Michel Thierry
From: Ben Widawsky benjamin.widaw...@intel.com The simple explanation is, the docs say to do this for GEN8. Perhaps we want to do this for GEN7 too, I am not certain. PDPs are saved and restored with context. Contexts (without execlists) only exist on the render ring. The docs say that PDPs are

Re: [Intel-gfx] [PATCH] drm/i915/ppgtt: Load address space after mi_set_context

2014-09-12 Thread Ville Syrjälä
On Fri, Sep 12, 2014 at 04:10:01PM +0100, Michel Thierry wrote: From: Ben Widawsky benjamin.widaw...@intel.com The simple explanation is, the docs say to do this for GEN8. Perhaps we want to do this for GEN7 too, I am not certain. PDPs are saved and restored with context. Contexts (without

Re: [Intel-gfx] [PATCH] drm/i915/ppgtt: Load address space after mi_set_context

2014-09-12 Thread Chris Wilson
On Fri, Sep 12, 2014 at 06:35:10PM +0300, Ville Syrjälä wrote: On Fri, Sep 12, 2014 at 04:10:01PM +0100, Michel Thierry wrote: From: Ben Widawsky benjamin.widaw...@intel.com The simple explanation is, the docs say to do this for GEN8. Perhaps we want to do this for GEN7 too, I am not

Re: [Intel-gfx] [PATCH] drm/i915/ppgtt: Load address space after mi_set_context

2014-09-12 Thread Michel Thierry
On 9/12/2014 4:40 PM, Chris Wilson wrote: On Fri, Sep 12, 2014 at 06:35:10PM +0300, Ville Syrjälä wrote: On Fri, Sep 12, 2014 at 04:10:01PM +0100, Michel Thierry wrote: From: Ben Widawsky benjamin.widaw...@intel.com The simple explanation is, the docs say to do this for GEN8. Perhaps we want

Re: [Intel-gfx] [PATCH] drm/i915/ppgtt: Load address space after mi_set_context

2014-09-12 Thread Ville Syrjälä
On Fri, Sep 12, 2014 at 05:56:30PM +0100, Michel Thierry wrote: On 9/12/2014 4:40 PM, Chris Wilson wrote: On Fri, Sep 12, 2014 at 06:35:10PM +0300, Ville Syrjälä wrote: On Fri, Sep 12, 2014 at 04:10:01PM +0100, Michel Thierry wrote: From: Ben Widawsky benjamin.widaw...@intel.com The