[Intel-gfx] [PATCH] drm/i915/guc: Always initialize action_lock

[Arkadiusz Hiler]

On Tue, Nov 22, 2016 at 10:25:40PM +0100, Srivatsa, Anusha wrote:
> 
> 
> >-----Original Message-----
> >From: Intel-gfx [mailto:intel-gfx-bounces at lists.freedesktop.org] On Behalf Of
> >Chris Wilson
> >Sent: Tuesday, November 22, 2016 9:06 AM
> >To: Hiler, Arkadiusz <arkadiusz.hiler at intel.com>
> >Cc: intel-gfx at lists.freedesktop.org
> >Subject: Re: [Intel-gfx] [PATCH] drm/i915/guc: Always initialize action_lock
> >
> >On Tue, Nov 22, 2016 at 05:22:47PM +0100, Arkadiusz Hiler wrote:
> >> Action lock is not being initialized if the GuC submission is disabled
> >> (i.e. i915.guc_submission=0).
> >>
> >> host2guc_action(), which uses the action_lock can be used for
> >> non-submission purposes, e.g. triggering HuC authentication.
> >>
> >> Moving action_lock initialization before enablement check will allow
> >> us to use the host2guc_action no matter whether submission is enabled
> >> or not.
> >
> >Seems like you want to split uc_send(), uc_recv() out of i915_guc_submission.c -
> >Chris
>  Hiler, this fixes the guc_loading enabled and guc_submission disabled scenario that I was facing.
> Anusha

As you can see from the discussion in this thread I'll approach this
differently - by doing some code reorganization.

It will obsolete the patch (and will create cosy place for functions
renamed in "Make the GuC fw loading helper functions general").

Feel free to include this patch in your series for now.

PS. Just call me Arek :)

-- 
Cheers,
Arek