Re: [Intel-gfx] [RFC PATCH] drm/i915/skl: Add DC6 disabling as a power well

2015-09-25 Thread Patrik Jakobsson
On Fri, Sep 25, 2015 at 12:41:42PM +0300, Imre Deak wrote: > On pe, 2015-09-25 at 10:56 +0200, Patrik Jakobsson wrote: > > On Thu, Sep 24, 2015 at 06:20:14PM +0300, Ville Syrjälä wrote: > > > On Thu, Sep 24, 2015 at 02:50:16PM +0200, Patrik Jakobsson wrote: > > > > On Wed, Sep 23, 2015 at 01:18:00P

Re: [Intel-gfx] [RFC PATCH] drm/i915/skl: Add DC6 disabling as a power well

2015-09-25 Thread Ville Syrjälä
On Fri, Sep 25, 2015 at 10:56:31AM +0200, Patrik Jakobsson wrote: > On Thu, Sep 24, 2015 at 06:20:14PM +0300, Ville Syrjälä wrote: > > On Thu, Sep 24, 2015 at 02:50:16PM +0200, Patrik Jakobsson wrote: > > > On Wed, Sep 23, 2015 at 01:18:00PM +0200, Patrik Jakobsson wrote: > > > > On Wed, Sep 23, 20

Re: [Intel-gfx] [RFC PATCH] drm/i915/skl: Add DC6 disabling as a power well

2015-09-25 Thread Imre Deak
On pe, 2015-09-25 at 10:56 +0200, Patrik Jakobsson wrote: > On Thu, Sep 24, 2015 at 06:20:14PM +0300, Ville Syrjälä wrote: > > On Thu, Sep 24, 2015 at 02:50:16PM +0200, Patrik Jakobsson wrote: > > > On Wed, Sep 23, 2015 at 01:18:00PM +0200, Patrik Jakobsson wrote: > > > > On Wed, Sep 23, 2015 at 10

Re: [Intel-gfx] [RFC PATCH] drm/i915/skl: Add DC6 disabling as a power well

2015-09-25 Thread Patrik Jakobsson
On Thu, Sep 24, 2015 at 06:20:14PM +0300, Ville Syrjälä wrote: > On Thu, Sep 24, 2015 at 02:50:16PM +0200, Patrik Jakobsson wrote: > > On Wed, Sep 23, 2015 at 01:18:00PM +0200, Patrik Jakobsson wrote: > > > On Wed, Sep 23, 2015 at 10:43:00AM +0200, Daniel Vetter wrote: > > > > On Mon, Sep 21, 2015

Re: [Intel-gfx] [RFC PATCH] drm/i915/skl: Add DC6 disabling as a power well

2015-09-24 Thread Ville Syrjälä
On Thu, Sep 24, 2015 at 02:50:16PM +0200, Patrik Jakobsson wrote: > On Wed, Sep 23, 2015 at 01:18:00PM +0200, Patrik Jakobsson wrote: > > On Wed, Sep 23, 2015 at 10:43:00AM +0200, Daniel Vetter wrote: > > > On Mon, Sep 21, 2015 at 10:00:45AM +0200, Patrik Jakobsson wrote: > > > > On Wed, Sep 16, 20

Re: [Intel-gfx] [RFC PATCH] drm/i915/skl: Add DC6 disabling as a power well

2015-09-24 Thread Patrik Jakobsson
On Wed, Sep 23, 2015 at 01:18:00PM +0200, Patrik Jakobsson wrote: > On Wed, Sep 23, 2015 at 10:43:00AM +0200, Daniel Vetter wrote: > > On Mon, Sep 21, 2015 at 10:00:45AM +0200, Patrik Jakobsson wrote: > > > On Wed, Sep 16, 2015 at 11:10:07PM +0300, Ville Syrjälä wrote: > > > > On Fri, Sep 11, 2015

Re: [Intel-gfx] [RFC PATCH] drm/i915/skl: Add DC6 disabling as a power well

2015-09-23 Thread Patrik Jakobsson
On Wed, Sep 23, 2015 at 10:43:00AM +0200, Daniel Vetter wrote: > On Mon, Sep 21, 2015 at 10:00:45AM +0200, Patrik Jakobsson wrote: > > On Wed, Sep 16, 2015 at 11:10:07PM +0300, Ville Syrjälä wrote: > > > On Fri, Sep 11, 2015 at 01:55:22PM +0200, Patrik Jakobsson wrote: > > > > We need to be able to

Re: [Intel-gfx] [RFC PATCH] drm/i915/skl: Add DC6 disabling as a power well

2015-09-23 Thread Daniel Vetter
On Mon, Sep 21, 2015 at 10:00:45AM +0200, Patrik Jakobsson wrote: > On Wed, Sep 16, 2015 at 11:10:07PM +0300, Ville Syrjälä wrote: > > On Fri, Sep 11, 2015 at 01:55:22PM +0200, Patrik Jakobsson wrote: > > > We need to be able to control if DC6 is allowed or not. Much like > > > requesting power to

Re: [Intel-gfx] [RFC PATCH] drm/i915/skl: Add DC6 disabling as a power well

2015-09-23 Thread Daniel Vetter
On Thu, Sep 17, 2015 at 02:45:34PM +0300, Ville Syrjälä wrote: > On Thu, Sep 17, 2015 at 02:14:37PM +0300, Ville Syrjälä wrote: > > On Wed, Sep 16, 2015 at 11:10:07PM +0300, Ville Syrjälä wrote: > > > and here we need > > > disable pw2 -> enable dc6 > > > > > > That's symmetric which is great for

Re: [Intel-gfx] [RFC PATCH] drm/i915/skl: Add DC6 disabling as a power well

2015-09-21 Thread Patrik Jakobsson
On Mon, Sep 21, 2015 at 12:43:36PM +0200, Patrik Jakobsson wrote: > On Thu, Sep 17, 2015 at 02:14:37PM +0300, Ville Syrjälä wrote: > > On Wed, Sep 16, 2015 at 11:10:07PM +0300, Ville Syrjälä wrote: > > > On Fri, Sep 11, 2015 at 01:55:22PM +0200, Patrik Jakobsson wrote: > > > > We need to be able to

Re: [Intel-gfx] [RFC PATCH] drm/i915/skl: Add DC6 disabling as a power well

2015-09-21 Thread Patrik Jakobsson
On Thu, Sep 17, 2015 at 02:14:37PM +0300, Ville Syrjälä wrote: > On Wed, Sep 16, 2015 at 11:10:07PM +0300, Ville Syrjälä wrote: > > On Fri, Sep 11, 2015 at 01:55:22PM +0200, Patrik Jakobsson wrote: > > > We need to be able to control if DC6 is allowed or not. Much like > > > requesting power to a s

Re: [Intel-gfx] [RFC PATCH] drm/i915/skl: Add DC6 disabling as a power well

2015-09-21 Thread Patrik Jakobsson
On Mon, Sep 21, 2015 at 11:26:06AM +0300, Jani Nikula wrote: > On Mon, 21 Sep 2015, Patrik Jakobsson > wrote: > > On Wed, Sep 16, 2015 at 11:10:07PM +0300, Ville Syrjälä wrote: > >> On Fri, Sep 11, 2015 at 01:55:22PM +0200, Patrik Jakobsson wrote: > >> > We need to be able to control if DC6 is al

Re: [Intel-gfx] [RFC PATCH] drm/i915/skl: Add DC6 disabling as a power well

2015-09-21 Thread Jani Nikula
On Mon, 21 Sep 2015, Patrik Jakobsson wrote: > On Wed, Sep 16, 2015 at 11:10:07PM +0300, Ville Syrjälä wrote: >> On Fri, Sep 11, 2015 at 01:55:22PM +0200, Patrik Jakobsson wrote: >> > We need to be able to control if DC6 is allowed or not. Much like >> > requesting power to a specific piece of the

Re: [Intel-gfx] [RFC PATCH] drm/i915/skl: Add DC6 disabling as a power well

2015-09-21 Thread Patrik Jakobsson
On Wed, Sep 16, 2015 at 11:10:07PM +0300, Ville Syrjälä wrote: > On Fri, Sep 11, 2015 at 01:55:22PM +0200, Patrik Jakobsson wrote: > > We need to be able to control if DC6 is allowed or not. Much like > > requesting power to a specific piece of the hardware we need to be able > > to request that we

Re: [Intel-gfx] [RFC PATCH] drm/i915/skl: Add DC6 disabling as a power well

2015-09-17 Thread Ville Syrjälä
On Thu, Sep 17, 2015 at 02:14:37PM +0300, Ville Syrjälä wrote: > On Wed, Sep 16, 2015 at 11:10:07PM +0300, Ville Syrjälä wrote: > > On Fri, Sep 11, 2015 at 01:55:22PM +0200, Patrik Jakobsson wrote: > > > We need to be able to control if DC6 is allowed or not. Much like > > > requesting power to a s

Re: [Intel-gfx] [RFC PATCH] drm/i915/skl: Add DC6 disabling as a power well

2015-09-17 Thread Ville Syrjälä
On Wed, Sep 16, 2015 at 11:10:07PM +0300, Ville Syrjälä wrote: > On Fri, Sep 11, 2015 at 01:55:22PM +0200, Patrik Jakobsson wrote: > > We need to be able to control if DC6 is allowed or not. Much like > > requesting power to a specific piece of the hardware we need to be able > > to request that we

Re: [Intel-gfx] [RFC PATCH] drm/i915/skl: Add DC6 disabling as a power well

2015-09-16 Thread Ville Syrjälä
On Fri, Sep 11, 2015 at 01:55:22PM +0200, Patrik Jakobsson wrote: > We need to be able to control if DC6 is allowed or not. Much like > requesting power to a specific piece of the hardware we need to be able > to request that we don't enter DC6 during certain hw access. > > To solve this without i

Re: [Intel-gfx] [RFC PATCH] drm/i915/skl: Add DC6 disabling as a power well

2015-09-14 Thread Patrik Jakobsson
On Mon, Sep 14, 2015 at 11:50:29AM +0200, Daniel Vetter wrote: > On Fri, Sep 11, 2015 at 01:55:22PM +0200, Patrik Jakobsson wrote: > > We need to be able to control if DC6 is allowed or not. Much like > > requesting power to a specific piece of the hardware we need to be able > > to request that we

Re: [Intel-gfx] [RFC PATCH] drm/i915/skl: Add DC6 disabling as a power well

2015-09-14 Thread Daniel Vetter
On Fri, Sep 11, 2015 at 01:55:22PM +0200, Patrik Jakobsson wrote: > We need to be able to control if DC6 is allowed or not. Much like > requesting power to a specific piece of the hardware we need to be able > to request that we don't enter DC6 during certain hw access. > > To solve this without i

[Intel-gfx] [RFC PATCH] drm/i915/skl: Add DC6 disabling as a power well

2015-09-11 Thread Patrik Jakobsson
We need to be able to control if DC6 is allowed or not. Much like requesting power to a specific piece of the hardware we need to be able to request that we don't enter DC6 during certain hw access. To solve this without introducing too much infrastructure I'm hooking into the power well / power d