On Tue, Aug 5, 2014 at 2:40 AM, Dave Airlie <airlied@xxxxxxxxx> wrote: > @@ -3502,10 +3505,13 @@ go_again: > void > intel_dp_check_link_status(struct intel_dp *intel_dp) > { > + struct drm_device *dev = intel_dp_to_dev(intel_dp); > struct intel_encoder *intel_encoder = &dp_to_dig_port(intel_dp)->base; > u8 sink_irq_vector; > u8 link_status[DP_LINK_STATUS_SIZE]; > > + WARN_ON(!drm_modeset_is_locked(&dev->mode_config.connection_mutex)); > + > /* FIXME: This access isn't protected by any locks. */ If you drop the FIXME here (since this patch address it) this is Reviewed-by: Daniel Vetter <daniel.vetter@xxxxxxxx> I guess you'll pick this up directly to drm-next like that other mst fixes? -Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx