drm/atomic: rip out unnecessary locking checks
authorRob Clark <robdclark@gmail.com>
Fri, 14 Nov 2014 22:12:59 +0000 (17:12 -0500)
committerDave Airlie <airlied@redhat.com>
Fri, 14 Nov 2014 23:30:24 +0000 (09:30 +1000)
For async commit, it is *intentional* that those locks are not held.

Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Signed-off-by: Rob Clark <robdclark@gmail.com>
Signed-off-by: Dave Airlie <airlied@redhat.com>
drivers/gpu/drm/drm_atomic_helper.c

index c0a07cb..fad2b93 100644 (file)
@@ -1006,8 +1006,6 @@ void drm_atomic_helper_commit_planes(struct drm_device *dev,
                if (!crtc)
                        continue;
 
-               WARN_ON(!drm_modeset_is_locked(&crtc->mutex));
-
                funcs = crtc->helper_private;
 
                if (!funcs || !funcs->atomic_begin)
@@ -1023,8 +1021,6 @@ void drm_atomic_helper_commit_planes(struct drm_device *dev,
                if (!plane)
                        continue;
 
-               WARN_ON(!drm_modeset_is_locked(&plane->mutex));
-
                funcs = plane->helper_private;
 
                if (!funcs || !funcs->atomic_update)