Re: [Intel-gfx] [PATCH v3] drm/i915: Remove unsafe i915.enable_rc6

2017-11-02 Thread Jani Nikula
On Thu, 02 Nov 2017, Joonas Lahtinen wrote: > On Thu, 2017-11-02 at 07:47 -0700, Rodrigo Vivi wrote: >> On Thu, Nov 02, 2017 at 08:06:29AM +, Jani Nikula wrote: >> > On Wed, 01 Nov 2017, Rodrigo Vivi wrote: >> > > On Wed, Nov 01, 2017 at 04:21:08PM +, Ben Widawsky wrote: >> > > > On 17-11

Re: [Intel-gfx] [PATCH v3] drm/i915: Remove unsafe i915.enable_rc6

2017-11-02 Thread Joonas Lahtinen
On Thu, 2017-11-02 at 07:47 -0700, Rodrigo Vivi wrote: > On Thu, Nov 02, 2017 at 08:06:29AM +, Jani Nikula wrote: > > On Wed, 01 Nov 2017, Rodrigo Vivi wrote: > > > On Wed, Nov 01, 2017 at 04:21:08PM +, Ben Widawsky wrote: > > > > On 17-11-01 18:09:47, Joonas Lahtinen wrote: > > > > > + Ki

Re: [Intel-gfx] [PATCH v3] drm/i915: Remove unsafe i915.enable_rc6

2017-11-02 Thread Rodrigo Vivi
On Thu, Nov 02, 2017 at 08:06:29AM +, Jani Nikula wrote: > On Wed, 01 Nov 2017, Rodrigo Vivi wrote: > > On Wed, Nov 01, 2017 at 04:21:08PM +, Ben Widawsky wrote: > >> On 17-11-01 18:09:47, Joonas Lahtinen wrote: > >> > + Kimmo and Paul > >> > > >> > On Wed, 2017-11-01 at 07:43 -0700, Ben

Re: [Intel-gfx] [PATCH v3] drm/i915: Remove unsafe i915.enable_rc6

2017-11-02 Thread Jani Nikula
On Wed, 01 Nov 2017, Rodrigo Vivi wrote: > On Wed, Nov 01, 2017 at 04:21:08PM +, Ben Widawsky wrote: >> On 17-11-01 18:09:47, Joonas Lahtinen wrote: >> > + Kimmo and Paul >> > >> > On Wed, 2017-11-01 at 07:43 -0700, Ben Widawsky wrote: >> > > On 17-11-01 14:07:28, Joonas Lahtinen wrote: >> >

Re: [Intel-gfx] [PATCH v3] drm/i915: Remove unsafe i915.enable_rc6

2017-11-01 Thread Rodrigo Vivi
On Wed, Nov 01, 2017 at 04:21:08PM +, Ben Widawsky wrote: > On 17-11-01 18:09:47, Joonas Lahtinen wrote: > > + Kimmo and Paul > > > > On Wed, 2017-11-01 at 07:43 -0700, Ben Widawsky wrote: > > > On 17-11-01 14:07:28, Joonas Lahtinen wrote: > > > > On Mon, 2017-10-30 at 10:48 -0700, Rodrigo Viv

Re: [Intel-gfx] [PATCH v3] drm/i915: Remove unsafe i915.enable_rc6

2017-11-01 Thread Ben Widawsky
On 17-11-01 18:09:47, Joonas Lahtinen wrote: + Kimmo and Paul On Wed, 2017-11-01 at 07:43 -0700, Ben Widawsky wrote: On 17-11-01 14:07:28, Joonas Lahtinen wrote: > On Mon, 2017-10-30 at 10:48 -0700, Rodrigo Vivi wrote: > > On Mon, Oct 30, 2017 at 01:00:51PM +, David Weinehall wrote: > > > O

Re: [Intel-gfx] [PATCH v3] drm/i915: Remove unsafe i915.enable_rc6

2017-11-01 Thread Joonas Lahtinen
+ Kimmo and Paul On Wed, 2017-11-01 at 07:43 -0700, Ben Widawsky wrote: > On 17-11-01 14:07:28, Joonas Lahtinen wrote: > > On Mon, 2017-10-30 at 10:48 -0700, Rodrigo Vivi wrote: > > > On Mon, Oct 30, 2017 at 01:00:51PM +, David Weinehall wrote: > > > > On Fri, Oct 27, 2017 at 01:57:09PM -0700,

Re: [Intel-gfx] [PATCH v3] drm/i915: Remove unsafe i915.enable_rc6

2017-11-01 Thread Ben Widawsky
On 17-11-01 14:07:28, Joonas Lahtinen wrote: On Mon, 2017-10-30 at 10:48 -0700, Rodrigo Vivi wrote: On Mon, Oct 30, 2017 at 01:00:51PM +, David Weinehall wrote: > On Fri, Oct 27, 2017 at 01:57:09PM -0700, Daniele Ceraolo Spurio wrote: > > > > > > On 26/10/17 03:32, Chris Wilson wrote: > > >

Re: [Intel-gfx] [PATCH v3] drm/i915: Remove unsafe i915.enable_rc6

2017-11-01 Thread Joonas Lahtinen
On Mon, 2017-10-30 at 10:48 -0700, Rodrigo Vivi wrote: > On Mon, Oct 30, 2017 at 01:00:51PM +, David Weinehall wrote: > > On Fri, Oct 27, 2017 at 01:57:09PM -0700, Daniele Ceraolo Spurio wrote: > > > > > > > > > On 26/10/17 03:32, Chris Wilson wrote: > > > > It has been many years since the l

Re: [Intel-gfx] [PATCH v3] drm/i915: Remove unsafe i915.enable_rc6

2017-10-30 Thread Rodrigo Vivi
On Mon, Oct 30, 2017 at 01:00:51PM +, David Weinehall wrote: > On Fri, Oct 27, 2017 at 01:57:09PM -0700, Daniele Ceraolo Spurio wrote: > > > > > > On 26/10/17 03:32, Chris Wilson wrote: > > > It has been many years since the last confirmed sighting (and fix) of an > > > RC6 related bug (usually

Re: [Intel-gfx] [PATCH v3] drm/i915: Remove unsafe i915.enable_rc6

2017-10-30 Thread David Weinehall
On Fri, Oct 27, 2017 at 01:57:09PM -0700, Daniele Ceraolo Spurio wrote: > > > On 26/10/17 03:32, Chris Wilson wrote: > > It has been many years since the last confirmed sighting (and fix) of an > > RC6 related bug (usually a system hang). Remove the parameter to stop > > users from setting danger

Re: [Intel-gfx] [PATCH v3] drm/i915: Remove unsafe i915.enable_rc6

2017-10-27 Thread Daniele Ceraolo Spurio
On 26/10/17 03:32, Chris Wilson wrote: It has been many years since the last confirmed sighting (and fix) of an RC6 related bug (usually a system hang). Remove the parameter to stop users from setting dangerous values, as they often set it during triage and end up disabling the entire runtime p

Re: [Intel-gfx] [PATCH v3] drm/i915: Remove unsafe i915.enable_rc6

2017-10-26 Thread Joonas Lahtinen
On Thu, 2017-10-26 at 11:32 +0100, Chris Wilson wrote: > It has been many years since the last confirmed sighting (and fix) of an > RC6 related bug (usually a system hang). Remove the parameter to stop > users from setting dangerous values, as they often set it during triage > and end up disabling

[Intel-gfx] [PATCH v3] drm/i915: Remove unsafe i915.enable_rc6

2017-10-26 Thread Chris Wilson
It has been many years since the last confirmed sighting (and fix) of an RC6 related bug (usually a system hang). Remove the parameter to stop users from setting dangerous values, as they often set it during triage and end up disabling the entire runtime pm instead (the option is not a fine scalpel