mirror of
https://github.com/torvalds/linux.git
synced 2026-07-27 09:36:22 +02:00
A hotplug or link-loss event can tear down the MST topology
(setting mgr->mst_state = false and mgr->mst_primary = NULL) concurrently
with a caller invoking drm_dp_mst_topology_queue_probe(). Since the check
is already performed under mgr->lock, the condition is not a programming
error but a valid race -- the topology was valid when the caller decided
to call this function, but was torn down before the lock was acquired.
Replace the drm_WARN_ON() with a graceful early return. This eliminates
spurious kernel warnings and the resulting compositor crashes observed
when connecting/disconnecting DP MST monitors, while keeping the correct
behavior of doing nothing when MST is not active. A drm_dbg_mst() trace
is added so the skipped probe remains observable under MST debug logging.
The existing WARN_ON(mgr->mst_primary) in drm_dp_mst_topology_mgr_set_mst()
already catches the case where the topology is initialized twice, so no
diagnostic coverage is lost.
Fixes:
|
||
|---|---|---|
| .. | ||
| drm_bridge_connector.c | ||
| drm_display_helper_mod.c | ||
| drm_dp_aux_bus.c | ||
| drm_dp_aux_dev.c | ||
| drm_dp_cec.c | ||
| drm_dp_dual_mode_helper.c | ||
| drm_dp_helper_internal.h | ||
| drm_dp_helper.c | ||
| drm_dp_mst_topology_internal.h | ||
| drm_dp_mst_topology.c | ||
| drm_dp_tunnel.c | ||
| drm_dsc_helper.c | ||
| drm_hdcp_helper.c | ||
| drm_hdmi_audio_helper.c | ||
| drm_hdmi_cec_helper.c | ||
| drm_hdmi_cec_notifier_helper.c | ||
| drm_hdmi_helper.c | ||
| drm_hdmi_state_helper.c | ||
| drm_scdc_helper.c | ||
| Kconfig | ||
| Makefile | ||