On Mon, May 18, 2015 at 11:24 PM, Dave Airlie <airlied@xxxxxxxxx> wrote: > On 19 May 2015 at 12:27, Michel Dänzer <michel@xxxxxxxxxxx> wrote: >> On 19.05.2015 01:24, Alex Deucher wrote: >>> >>> @@ -96,10 +98,12 @@ static void radeon_dp_work_func(struct work_struct *work) >>> struct drm_connector *connector; >>> >>> /* this should take a mutex */ >>> + mutex_lock(&mode_config->mutex); >> >> This comment can be removed? > > I have vague memories of not doing this, because bad things happened. > > so keep an eye out for lockdep traces. I tested the non-MST handling pretty extensively, but I admit I didn't play with mst. Might be better to split this into two patches. Alex _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/dri-devel
