[PATCH] drm: Add locking to resume_force_mode to prevent multiple

Sean Paul seanpaul at chromium.org
Fri Mar 23 05:51:31 PDT 2012


On Thu, Mar 22, 2012 at 6:57 PM, Chris Wilson <chris at chris-wilson.co.uk> wrote:
> On Thu, 22 Mar 2012 18:25:55 -0400, Sean Paul <seanpaul at chromium.org> wrote:
>> Add a mutex to protect resume_force_mode from being called multiple
>> times. This fixes a bug observed on SNB where two wake sources call
>> resume_force_mode and the FDI training fails as a result. The user
>> facing result of this is complete screen corruption.
>
> Looks like a bug in i915_drv.c for calling that function without holding
> the mode_config.lock

Good catch, thanks for the feedback Chris.

I'll submit a new patch to intel-gfx, please drop this one.

Sean


> -Chris
>
> --
> Chris Wilson, Intel Open Source Technology Centre


More information about the dri-devel mailing list