Intel-GFX Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Imre Deak <imre.deak@intel.com>
To: Alexander Kaplan <alexander.kaplan@sms-medipool.de>
Cc: intel-gfx@lists.freedesktop.org, intel-xe@lists.freedesktop.org,
	jani.nikula@linux.intel.com, rodrigo.vivi@intel.com,
	"Ville Syrjälä" <ville.syrjala@linux.intel.com>
Subject: Re: [PATCH] drm/i915/tc: Disable outputs instead of modesetting them on link reset
Date: Tue, 9 Jun 2026 17:31:17 +0300	[thread overview]
Message-ID: <aigjtXiFqvxTGUnD@ideak-desk.lan> (raw)
In-Reply-To: <20260608223548.2574-1-alexander.kaplan@sms-medipool.de>

On Tue, Jun 09, 2026 at 12:35:48AM +0200, Alexander Kaplan wrote:
> Hi Imre,
>
> [...]
>
> I did the test you asked for. I rebooted latest drm-tip with your
> patch applied and drm.debug=0x15e, and reproduced the problem by
> power-cycling the TV while the output was active.  The full boot log
> up to the wedge is on the first ticket [1], with the reproduction
> steps.

Thanks for the log.

> On PTL this never reaches the -EINVAL / WARN you get on the ADL/MTL
> reports. The recovery modeset computes a valid (degraded) config and
> commits it against the dead link, link training fails, and the pipe is
> re-enabled regardless, leaving the output enabled on a disconnected
> port.  After that the TC PHY ownership stays held,
> intel_tc_port_connected() returns false and AUX is rejected, so the
> reconnect HPD never produces a successful detect.  Keeping the DSC
> caps doesn't help here. The caps we lose are dfp.*/EDID (the PCON FRL
> bandwidth) via intel_dp_unset_edid(), not dsc_dpcd, and the second
> failure mode above is independent of the sink caps anyway.

The actual problem is that userspace does not follow up with a disabling
modeset, as it should after it received a hotplug event and did a
connector probing on the connector where the sink got disconnected. The
connector state is disconnected in this case and userspace must disable
the output accordingly. However it doesn't do this, not sure why. I
provided the corresponding events from the log on [1], the best course
forward would be to find out why userspace (something based on Wayland)
on your host doesn't do disabling modesets.

--Imre

> ...
>
> [1] https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/14807
> 
> Thanks again,
> Alexander

  reply	other threads:[~2026-06-09 14:31 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-06-05 21:28 [PATCH] drm/i915/tc: Disable outputs instead of modesetting them on link reset Alexander Kaplan
2026-06-05 22:11 ` ✗ LGCI.VerificationFailed: failure for " Patchwork
2026-06-08 12:36 ` [PATCH] " Imre Deak
2026-06-08 22:35   ` Alexander Kaplan
2026-06-09 14:31     ` Imre Deak [this message]
2026-06-09 18:48       ` Alexander Kaplan
2026-06-10 11:42         ` Jani Nikula
2026-06-10 17:50           ` Alexander Kaplan

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=aigjtXiFqvxTGUnD@ideak-desk.lan \
    --to=imre.deak@intel.com \
    --cc=alexander.kaplan@sms-medipool.de \
    --cc=intel-gfx@lists.freedesktop.org \
    --cc=intel-xe@lists.freedesktop.org \
    --cc=jani.nikula@linux.intel.com \
    --cc=rodrigo.vivi@intel.com \
    --cc=ville.syrjala@linux.intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox