From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from sender4-pp-f112.zoho.com (sender4-pp-f112.zoho.com [136.143.188.112]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id A86C3382F20 for ; Wed, 22 Apr 2026 18:25:00 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=pass smtp.client-ip=136.143.188.112 ARC-Seal:i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1776882302; cv=pass; b=QOa9Fe4uzkwCyfwHR2YUQnF9qQNeDes0DlNTP2dbn5+mssaV1KFjOsMiaKz5XIBX3hNhm1y65/DDse+QZNEs3HJyp0gYQwDO8CU9Dgfzl2InyR0x9hLMC/PQZhJszNJ99Ulsjhe3YDRkDwHrK4A0HTeJlALX2M8d5LbtoGyuSdo= ARC-Message-Signature:i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1776882302; c=relaxed/simple; bh=pN04WnSRfQYtvkTe46tcTv/hDoEDuJUiCOrsuNpY5HI=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=YPqnv5vsJOUeIrhlxITShLPQqjW9IMEBkCkBkaDK/VOYWD+6t51GMcGSp4Jmo14IDOphxJs376yMd39jFxH+jnbbRjY09XVf4PDPh5KP81cc2RS+h1tD0dgs878BHtKqCSTrjDMEcmwVREK9ivE6Q+xgNHUZmNMm5B2dXJaisT8= ARC-Authentication-Results:i=2; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=collabora.com; spf=pass smtp.mailfrom=collabora.com; dkim=pass (1024-bit key) header.d=collabora.com header.i=nicolas.frattaroli@collabora.com header.b=NK1qTxyF; arc=pass smtp.client-ip=136.143.188.112 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=collabora.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=collabora.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=collabora.com header.i=nicolas.frattaroli@collabora.com header.b="NK1qTxyF" ARC-Seal: i=1; a=rsa-sha256; t=1776882278; cv=none; d=zohomail.com; s=zohoarc; b=dTNLwfr5AhcmBC6f6skqYCvDOmTe7opvTR4pIs3g9cgPumBbd2jy+fQVk/RblQE2xicox2wopb0/+kBuTNp4MX0fJUgzKHN3CP0Bj5p620aFmGj3SgorWbIaBxAvb5BtmFKDB+XXFp65Bk7ckJDsKsX3wxTXC0oBG6AoWdvoo7U= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1776882278; h=Content-Type:Content-Transfer-Encoding:Cc:Cc:Date:Date:From:From:In-Reply-To:MIME-Version:Message-ID:References:Subject:Subject:To:To:Message-Id:Reply-To; bh=PHT6T41t2TqVIBPQsCMXPMGDztXJ0gS62OVUZXhxw9c=; b=g0czw8wiUHyuio0SMyKkAU8co0wzV8wP717JB3CGPs5JMhyP8U7Bv/7Do7t7mZ0aQFqigUwA5Yj7X1cgaqKBByKE45h0hHjr0nj+JdL7ONr4b5RAT95VAHJb3yU5Gz00DC3o57yACuVjG1X4dzwpyFYAbq8mYGUBYRGjwAMMAZk= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass header.i=collabora.com; spf=pass smtp.mailfrom=nicolas.frattaroli@collabora.com; dmarc=pass header.from= DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; t=1776882278; s=zohomail; d=collabora.com; i=nicolas.frattaroli@collabora.com; h=From:From:Date:Date:Subject:Subject:MIME-Version:Content-Type:Content-Transfer-Encoding:Message-Id:Message-Id:References:In-Reply-To:To:To:Cc:Cc:Reply-To; bh=PHT6T41t2TqVIBPQsCMXPMGDztXJ0gS62OVUZXhxw9c=; b=NK1qTxyFsT2XX3Z/0eSdJFHe+V6ds0yHGYlKbvbgR6FBYkmuZvGyvuL51hfYl/+J 1o+P9aGSQBz7hCE8P1yObyYwcpg6CVFJkOJELOJ8K8wGL9lzXD/gCI6OPi1TM5rAQXP 8aZmWm+hWx1FLkH7+WVhEk1p8BDmnJ7h9i1STIb8= Received: by mx.zohomail.com with SMTPS id 1776882277358176.8374044602018; Wed, 22 Apr 2026 11:24:37 -0700 (PDT) From: Nicolas Frattaroli Date: Wed, 22 Apr 2026 20:24:08 +0200 Subject: [PATCH v9 2/2] drm: Send per-connector hotplug events Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit Message-Id: <20260422-hot-plug-passup-v9-2-aef804255986@collabora.com> References: <20260422-hot-plug-passup-v9-0-aef804255986@collabora.com> In-Reply-To: <20260422-hot-plug-passup-v9-0-aef804255986@collabora.com> To: Maarten Lankhorst , Maxime Ripard , Thomas Zimmermann , David Airlie , Simona Vetter , Stanislav Lisovskiy , =?utf-8?q?Ville_Syrj=C3=A4l=C3=A4?= , Louis Chauvet , Haneen Mohammed , Melissa Wen , Daniel Stone , Ian Forbes , Dmitry Baryshkov Cc: linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org, kernel@collabora.com, wayland-devel@lists.freedesktop.org, Nicolas Frattaroli , Marius Vlad X-Mailer: b4 0.15.2 Try to send per-connector hotplug events as often as possible, rather than connector-less global hotplug events. This does result in more hotplug events if multiple connectors changed at the same time, but give userspace more actionable information. Since the hotplug event needs to be sent outside of the mode_config mutex to avoid a deadlock, the drm_client_dev_hotplug() call is split off from the drm_sysfs_(connector_)?hotplug_event calls. Co-developed-by: Marius Vlad Signed-off-by: Marius Vlad Signed-off-by: Nicolas Frattaroli --- drivers/gpu/drm/drm_probe_helper.c | 37 ++++++++++++++++--------------------- 1 file changed, 16 insertions(+), 21 deletions(-) diff --git a/drivers/gpu/drm/drm_probe_helper.c b/drivers/gpu/drm/drm_probe_helper.c index d4dc8cb45bce..2daa4242ce7e 100644 --- a/drivers/gpu/drm/drm_probe_helper.c +++ b/drivers/gpu/drm/drm_probe_helper.c @@ -760,16 +760,12 @@ static void output_poll_execute(struct work_struct *work) struct drm_connector *connector; struct drm_connector_list_iter conn_iter; enum drm_connector_status old_status; - bool repoll = false, changed; + bool repoll = false, changed = false; u64 old_epoch_counter; if (!dev->mode_config.poll_enabled) return; - /* Pick up any changes detected by the probe functions. */ - changed = dev->mode_config.delayed_event; - dev->mode_config.delayed_event = false; - if (!drm_kms_helper_poll) { if (dev->mode_config.poll_running) { drm_kms_helper_disable_hpd(dev); @@ -836,6 +832,7 @@ static void output_poll_execute(struct work_struct *work) connector->base.id, connector->name, old_epoch_counter, connector->epoch_counter); + drm_sysfs_connector_hotplug_event(connector); changed = true; } } @@ -844,8 +841,15 @@ static void output_poll_execute(struct work_struct *work) mutex_unlock(&dev->mode_config.mutex); out: + /* Pick up any changes detected by the probe functions. */ + if (dev->mode_config.delayed_event) { + dev->mode_config.delayed_event = false; + changed = true; + drm_sysfs_hotplug_event(dev); + } + if (changed) - drm_kms_helper_hotplug_event(dev); + drm_client_dev_hotplug(dev); if (repoll) schedule_delayed_work(delayed_work, DRM_OUTPUT_POLL_PERIOD); @@ -1081,9 +1085,9 @@ EXPORT_SYMBOL(drm_connector_helper_hpd_irq_event); */ bool drm_helper_hpd_irq_event(struct drm_device *dev) { - struct drm_connector *connector, *first_changed_connector = NULL; struct drm_connector_list_iter conn_iter; - int changed = 0; + struct drm_connector *connector; + bool changed = false; if (!dev->mode_config.poll_enabled) return false; @@ -1096,24 +1100,15 @@ bool drm_helper_hpd_irq_event(struct drm_device *dev) continue; if (check_connector_changed(connector)) { - if (!first_changed_connector) { - drm_connector_get(connector); - first_changed_connector = connector; - } - - changed++; + changed = true; + drm_sysfs_connector_hotplug_event(connector); } } drm_connector_list_iter_end(&conn_iter); mutex_unlock(&dev->mode_config.mutex); - if (changed == 1) - drm_kms_helper_connector_hotplug_event(first_changed_connector); - else if (changed > 0) - drm_kms_helper_hotplug_event(dev); - - if (first_changed_connector) - drm_connector_put(first_changed_connector); + if (changed) + drm_client_dev_hotplug(dev); return changed; } -- 2.53.0