Re: [PATCH] drm/i915: Fix locking around GuC firmware load

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




On 11/03/2015 07:22 PM, Dave Airlie wrote:
On 4 November 2015 at 07:42, Daniel Stone <daniels@xxxxxxxxxxxxx> wrote:
> The GuC firmware load requires struct_mutex to create a GEM object,
> but this collides badly with request_firmware. Move struct_mutex
> locking down into the loader itself, so we don't hold it across the
> entire load process, including request_firmware.
>
> Signed-off-by: Daniel Stone <daniels@xxxxxxxxxxxxx>

This avoids the lockdep spew on my machine.

Reviewed-by: Dave AIrlie <airlied@xxxxxxxxxx>

I'm going to put this into -next, granted I've still got the other WARN spew
but lockdep working is a good start.



Looks good to me. Thanks for the fix.

Reviewed-by: Alex Dai <yu.dai@xxxxxxxxx>

Alex
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux