Re: [Intel-gfx] [PATCH] drm/i915: Always load guc by default.

2016-12-03 Thread Chris Wilson
> >g...@lists.freedesktop.org; Mcgee, Jeff > >Subject: Re: [Intel-gfx] [PATCH] drm/i915: Always load guc by default. > > > >On Thu, Nov 24, 2016 at 08:31:22AM +, Tvrtko Ursulin wrote: > >> > >> On 24/11/2016 08:21, Chris Wilson wrote: > >> >On

Re: [Intel-gfx] [PATCH] drm/i915: Always load guc by default.

2016-12-02 Thread Srivatsa, Anusha
>-Original Message- >From: Chris Wilson [mailto:ch...@chris-wilson.co.uk] >Sent: Thursday, November 24, 2016 12:41 AM >To: Tvrtko Ursulin >Cc: Srivatsa, Anusha ; intel- >g...@lists.freedesktop.org; Mcgee, Jeff >Subject: Re: [Intel-gfx] [PATCH] drm/i915: Alway

Re: [Intel-gfx] [PATCH] drm/i915: Always load guc by default.

2016-11-24 Thread Chris Wilson
On Thu, Nov 24, 2016 at 08:31:22AM +, Tvrtko Ursulin wrote: > > On 24/11/2016 08:21, Chris Wilson wrote: > >On Thu, Nov 24, 2016 at 08:15:31AM +, Tvrtko Ursulin wrote: > >> > >>On 24/11/2016 07:13, Chris Wilson wrote: > >>>On Wed, Nov 23, 2016 at 04:52:38PM -0800, Anusha Srivatsa wrote: >

Re: [Intel-gfx] [PATCH] drm/i915: Always load guc by default.

2016-11-24 Thread Jani Nikula
On Thu, 24 Nov 2016, Chris Wilson wrote: > History says we will regret enabling a new piece of hw/fw without a > feature option. And history says we'll regret adding new module parameters for everything. Lose-lose. :( BR, Jani. -- Jani Nikula, Intel Open Source Technology Center _

Re: [Intel-gfx] [PATCH] drm/i915: Always load guc by default.

2016-11-24 Thread Tvrtko Ursulin
On 24/11/2016 08:21, Chris Wilson wrote: On Thu, Nov 24, 2016 at 08:15:31AM +, Tvrtko Ursulin wrote: On 24/11/2016 07:13, Chris Wilson wrote: On Wed, Nov 23, 2016 at 04:52:38PM -0800, Anusha Srivatsa wrote: Remove the enable_guc_loading parameter. Load the GuC on plaforms that have GuC.

Re: [Intel-gfx] [PATCH] drm/i915: Always load guc by default.

2016-11-24 Thread Chris Wilson
On Thu, Nov 24, 2016 at 08:15:31AM +, Tvrtko Ursulin wrote: > > On 24/11/2016 07:13, Chris Wilson wrote: > >On Wed, Nov 23, 2016 at 04:52:38PM -0800, Anusha Srivatsa wrote: > >>Remove the enable_guc_loading parameter. Load the GuC on > >>plaforms that have GuC. All issues we found so far are r

Re: [Intel-gfx] [PATCH] drm/i915: Always load guc by default.

2016-11-24 Thread Tvrtko Ursulin
On 24/11/2016 07:13, Chris Wilson wrote: On Wed, Nov 23, 2016 at 04:52:38PM -0800, Anusha Srivatsa wrote: Remove the enable_guc_loading parameter. Load the GuC on plaforms that have GuC. All issues we found so far are related to GuC features like the command submission, but no bug is related to

Re: [Intel-gfx] [PATCH] drm/i915: Always load guc by default.

2016-11-23 Thread Chris Wilson
On Wed, Nov 23, 2016 at 04:52:38PM -0800, Anusha Srivatsa wrote: > Remove the enable_guc_loading parameter. Load the GuC on > plaforms that have GuC. All issues we found so far are related > to GuC features like the command submission, but no bug is related > to the guc loading itself. > > This ad

[Intel-gfx] [PATCH] drm/i915: Always load guc by default.

2016-11-23 Thread Anusha Srivatsa
Remove the enable_guc_loading parameter. Load the GuC on plaforms that have GuC. All issues we found so far are related to GuC features like the command submission, but no bug is related to the guc loading itself. This addresses the case when we need GuC loaded even with no GuC feature in use, lik