]> git.baikalelectronics.ru Git - kernel.git/commit
Reinstate "drm/probe-helpers: Drop locking from poll_enable""
authorDave Airlie <airlied@redhat.com>
Fri, 27 Jan 2017 02:04:08 +0000 (12:04 +1000)
committerDave Airlie <airlied@redhat.com>
Fri, 27 Jan 2017 02:04:08 +0000 (12:04 +1000)
commit429a580dac9186758b83601b5cd6a9b648ccfdb5
tree0244663e07ce8575d6060d0f1dae9d8fef30e829
parent191a58a738851dbec8f6d79576885e03e2c69eaf
Reinstate "drm/probe-helpers: Drop locking from poll_enable""

This reverts commit ba47a7809243362d1d2d5f2ec6dd229b20d75685,
and reinstates the original.

[airlied: this might be a bad plan for git].

commit e3cd17da1990753f6b1eab7d184086ed51a99a69
Author: Daniel Vetter <daniel.vetter@ffwll.ch>
Date:   Wed Jan 11 10:01:17 2017 +0100

    drm/probe-helpers: Drop locking from poll_enable

    It was only needed to protect the connector_list walking, see

    commit c6b5b6b2c3775b220411320c53ed9a2bfee64c8f
    Author: Daniel Vetter <daniel.vetter@ffwll.ch>
    Date:   Thu Jul 9 23:44:26 2015 +0200

        drm/probe-helper: Grab mode_config.mutex in poll_init/enable

    Unfortunately the commit message of that patch fails to mention that
    the new locking check was for the connector_list.

    But that requirement disappeared in

    commit 32822e46b451eea41e61308ffe060210a8fc8d89
    Author: Daniel Vetter <daniel.vetter@ffwll.ch>
    Date:   Thu Dec 15 16:58:43 2016 +0100

        drm: Convert all helpers to drm_connector_list_iter

    and so we can drop this again.

    This fixes a locking inversion on nouveau, where the rpm code needs to
    re-enable. But in other places the rpm_get() calls are nested within
    the big modeset locks.

    While at it, also improve the kerneldoc for these two functions a
    notch.

    v2: Update the kerneldoc even more to explain that these functions
    can't be called concurrently, or bad things happen (Chris).
drivers/gpu/drm/drm_probe_helper.c
drivers/gpu/drm/i915/intel_hotplug.c
include/drm/drm_crtc_helper.h