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 --- opps, hit send too soon, disregard last one

drivers/gpu/drm/drm_atomic_helper.c | 4 ---- 1 file changed, 4 deletions(-)

diff --git a/drivers/gpu/drm/drm_atomic_helper.c b/drivers/gpu/drm/drm_atomic_helper.c index c0a07cb..fad2b93 100644 --- a/drivers/gpu/drm/drm_atomic_helper.c +++ b/drivers/gpu/drm/drm_atomic_helper.c @@ -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)