[bug report] drm/amd/display: Merge amdgpu_dm_types and amdgpu_dm
Dan Carpenter
dan.carpenter at oracle.com
Sat Nov 4 07:50:29 UTC 2017
Hello Harry Wentland,
The patch e7b07ceef2a6: "drm/amd/display: Merge amdgpu_dm_types and
amdgpu_dm" from Aug 10, 2017, leads to the following static checker
warning:
drivers/gpu/drm/amd/amdgpu/../display/amdgpu_dm/amdgpu_dm.c:3895 amdgpu_dm_commit_planes()
error: double unlock 'spin_lock:&crtc->dev->event_lock'
drivers/gpu/drm/amd/amdgpu/../display/amdgpu_dm/amdgpu_dm.c
3883
3884 pflip_needed = !state->allow_modeset;
3885
3886 spin_lock_irqsave(&crtc->dev->event_lock, flags);
3887 if (acrtc_attach->pflip_status != AMDGPU_FLIP_NONE) {
3888 DRM_ERROR("%s: acrtc %d, already busy\n",
3889 __func__,
3890 acrtc_attach->crtc_id);
3891 spin_unlock_irqrestore(&crtc->dev->event_lock, flags);
^^^^^^^^^^^^^^^^^^^^^^
3892 /* In commit tail framework this cannot happen */
3893 WARN_ON(1);
3894 }
3895 spin_unlock_irqrestore(&crtc->dev->event_lock, flags);
^^^^^^^^^^^^^^^^^^^^^^
Double unlock.
3896
3897 if (!pflip_needed) {
regards,
dan carpenter
More information about the amd-gfx
mailing list