Re: [Intel-gfx] [PATCH 1/2] drm/i915: Remove i915.enable_execlists module parameter

2017-10-31 Thread Lionel Landwerlin
I can't speak for the totality of the driver, but the perf part is : Reviewed-by: Lionel Landwerlin On 26/10/17 22:17, Chris Wilson wrote: diff --git a/drivers/gpu/drm/i915/i915_perf.c b/drivers/gpu/drm/i915/i915_perf.c index 59ee808f8fd9..8dfa74bd3f37 100644 --- a/drivers/gpu/drm/i915/i915_pe

Re: [Intel-gfx] [PATCH 1/2] drm/i915: Remove i915.enable_execlists module parameter

2017-10-30 Thread Joonas Lahtinen
On Thu, 2017-10-26 at 22:17 +0100, Chris Wilson wrote: > Execlists and legacy ringbuffer submission are no longer feature > comparable (execlists now offer greater functionality that should > overcome their performance hit) and obsoletes the unsafe module > parameter, i.e. comparing the two modes o

[Intel-gfx] [PATCH 1/2] drm/i915: Remove i915.enable_execlists module parameter

2017-10-26 Thread Chris Wilson
Execlists and legacy ringbuffer submission are no longer feature comparable (execlists now offer greater functionality that should overcome their performance hit) and obsoletes the unsafe module parameter, i.e. comparing the two modes of execution is no longer useful, so remove the debug tool. Sig

Re: [Intel-gfx] [PATCH 1/2] drm/i915: Remove i915.enable_execlists module parameter

2017-03-27 Thread Joonas Lahtinen
On pe, 2017-03-24 at 09:42 +, Chris Wilson wrote: > Execlists and legacy ringbuffer submission are no longer feature > comparable (execlists now offer greater functionality that should > overcome their performance hit) and obsoletes the unsafe module > parameter, i.e. comparing the two modes of

[Intel-gfx] [PATCH 1/2] drm/i915: Remove i915.enable_execlists module parameter

2017-03-24 Thread Chris Wilson
Execlists and legacy ringbuffer submission are no longer feature comparable (execlists now offer greater functionality that should overcome their performance hit) and obsoletes the unsafe module parameter, i.e. comparing the two modes of execution is no longer useful, so remove the debug tool. Sig