From: Dave Airlie airlied@redhat.com
Since we might have queued work but it hasn't executed yet this could cause an oops in drm_dp_check_and_send_link_address.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=89366
Signed-off-by: Dave Airlie airlied@redhat.com --- drivers/gpu/drm/drm_dp_mst_topology.c | 2 ++ 1 file changed, 2 insertions(+)
diff --git a/drivers/gpu/drm/drm_dp_mst_topology.c b/drivers/gpu/drm/drm_dp_mst_topology.c index fb65f5d..7678cb5 100644 --- a/drivers/gpu/drm/drm_dp_mst_topology.c +++ b/drivers/gpu/drm/drm_dp_mst_topology.c @@ -1936,6 +1936,8 @@ int drm_dp_mst_topology_mgr_set_mst(struct drm_dp_mst_topology_mgr *mgr, bool ms
ret = 0; } else { + /* flush any pending work */ + cancel_work_sync(&mgr->work); /* disable MST on the device */ mstb = mgr->mst_primary; mgr->mst_primary = NULL;
On 19 June 2015 at 10:20, Dave Airlie airlied@gmail.com wrote:
From: Dave Airlie airlied@redhat.com
Since we might have queued work but it hasn't executed yet this could cause an oops in drm_dp_check_and_send_link_address.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=89366
Ignore this, it was lockdeppy.
the other fix is better.
Dave.
dri-devel@lists.freedesktop.org