Op 27-06-17 om 16:59 schreef Daniel Vetter: > From: Thierry Reding <treding@xxxxxxxxxx> > > Move the modeset locking from drivers into FB helpers. > > v2: Also handle intel_connector_add_to_fbdev. > > Tested-by: John Stultz <john.stultz@xxxxxxxxxx> > Signed-off-by: Thierry Reding <treding@xxxxxxxxxx> (v1) > Signed-off-by: Daniel Vetter <daniel.vetter@xxxxxxxx> > --- > drivers/gpu/drm/drm_fb_helper.c | 40 +++++++++++++++++++++++++++++----- > drivers/gpu/drm/i915/intel_dp_mst.c | 6 ----- > drivers/gpu/drm/radeon/radeon_dp_mst.c | 7 ------ > 3 files changed, 34 insertions(+), 19 deletions(-) I know we suck at DP-MST. But I fear the unregister leaves open a race with mst_port being unset without connection_mutex.. best_encoder() and mode_valid() appear to use it. It's probably harmless and I have no good solution, so maybe just annotate it in the patch? Might also affect i915_hpd_poll_init_work, though I think the race in itself is harmless. _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
