public inbox for linux-usb@vger.kernel.org
 help / color / mirror / Atom feed
From: Mika Westerberg <mika.westerberg@linux.intel.com>
To: Kenneth Crudup <kenny@panix.com>
Cc: linux-usb@vger.kernel.org
Subject: Re: So, I had to revert d6d458d42e1 ("Handle DisplayPort tunnel activation asynchronously") too, to stop my resume crashes
Date: Mon, 3 Mar 2025 12:46:18 +0200	[thread overview]
Message-ID: <20250303104618.GP3713119@black.fi.intel.com> (raw)
In-Reply-To: <2b55ab09-8389-4916-8acc-abc73f234d21@panix.com>

Hi Kenneth,

Like discussed, let's deal one issue at the time.

It is really hard to debug anything if you keep changing the steps so
please let's keep these as separate issues:

1) Hang/crash during resume when dock + NVMe is disconnected before resume.
2) Monitor issue over DP tunnel.

For the first is this now solved if you revert
9d573d19547b3fae0c1d4e5fce52bdad3fda3664?

You can "isolate" this to PCIe side completely by doing the steps with the
commit but don't connect any monitors. Then you can do the steps (just
these , don't throw in any additional steps unless you think they are
needed but then mention them):

1. Boot the system up, nothing connected.
2. Connect TBT 4 dock to the host (no monitors)
3. Connect TBT 3 NVMe to the TBT 4 dock (no monitors)
4. Verify that the PCIe devices such as the NVMe are visible and working.
5. Suspend the system by closing the lid.
6. Unplug the device chain from the host.
7. Resume the system by opening the lid.

Expectation: System resumes just fine, PCIe devices are gone but system is
responsive.
Actual result: System does not resume and is not responsive.

If this gets solved by the revert then that's one issue nailed, good.

----------------------------------

For the second issue, I'm not sure I know the steps but since you mention
reverting d6d458d42e1e ("thunderbolt: Handle DisplayPort tunnel activation
asynchronously"), it should trigger pretty much any time you plug in
monitor so we can follow different and hopefully simpler steps:

1. Boot the system up, nothing connected.
2. Connect TBT 4 dock to the system.
3. Connect monitor to the TBT 4 dock.

Expectation: Monitor shows the screen properly.
Actual result: Blank screen.

For this issue please enable "thunderbolt.dyndbg=p+" in the kernel command
line so we can see in the dmesg what is going on. Once you reproduce (if
above steps then no need to mention, if different steps mention exactly the
simplest steps you do to reproduce) provide full dmesg of this run. I will
then take a look.

Thanks!

On Sun, Mar 02, 2025 at 08:30:55AM -0800, Kenneth Crudup wrote:
> 
> Forgot to add the dmesg.
> 
> -K
> 
> On 3/2/25 08:26, Kenneth Crudup wrote:
> > 
> > FWIW, seeing a Metric F'ton (13171) of these after testing a hibernate
> > cycle- I guess now that my resumes are completing these are occurring
> > now:
> > 
> > thunderbolt 0000:00:0d.3: hotplug event from non existent switch 1:d
> > (unplug: 0)
> > 
> > This is one of my onboard ports (presumably the right one I use all the
> > time at home):
> > 
> > ----
> > 0000:00:0d.3 USB controller [0c03]: Intel Corporation Alder Lake-P
> > Thunderbolt 4 NHI #1 [8086:466d] (rev 02) (prog-if 40 [USB4 Host
> > Interface])
> >          Subsystem: Dell Device [1028:0af3]
> >          Flags: bus master, fast devsel, latency 0, IRQ 16, IOMMU group 8
> >          Memory at 6040200000 (64-bit, non-prefetchable) [size=256K]
> >          Memory at 60402e3000 (64-bit, non-prefetchable) [size=4K]
> >          Capabilities: [80] Power Management version 3
> >          Capabilities: [88] MSI: Enable- Count=1/1 Maskable- 64bit+
> >          Capabilities: [a0] MSI-X: Enable+ Count=16 Masked-
> >          Kernel driver in use: thunderbolt
> > ----
> > 
> > -K
> > 
> 
> -- 
> Kenneth R. Crudup / Sr. SW Engineer, Scott County Consulting, Orange County
> CA



  reply	other threads:[~2025-03-03 10:46 UTC|newest]

Thread overview: 34+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-03-02  4:57 So, I had to revert d6d458d42e1 ("Handle DisplayPort tunnel activation asynchronously") too, to stop my resume crashes Kenneth Crudup
2025-03-02  5:36 ` Kenneth Crudup
2025-03-02 16:26   ` Kenneth Crudup
2025-03-02 16:30     ` Kenneth Crudup
2025-03-03 10:46       ` Mika Westerberg [this message]
2025-03-03 11:02         ` Kenneth Crudup
2025-03-03 11:21           ` Mika Westerberg
2025-03-03 11:38             ` Kenneth Crudup
2025-03-03 11:45               ` Kenneth Crudup
2025-03-03 11:55                 ` Mika Westerberg
2025-03-03 12:39                   ` Kenneth Crudup
2025-03-03 12:51                     ` Kenneth Crudup
2025-03-03 11:53               ` Mika Westerberg
2025-03-03 12:33                 ` Kenneth Crudup
2025-03-03 13:13                   ` Mika Westerberg
2025-03-03 13:19                     ` Kenneth Crudup
2025-03-03 13:23                       ` Mika Westerberg
2025-03-03 13:46                         ` Mika Westerberg
2025-03-03 13:53                           ` Kenneth Crudup
2025-03-03 14:01                             ` Mika Westerberg
2025-03-03 14:10                               ` Kenneth Crudup
2025-03-03 14:20                                 ` Mika Westerberg
2025-03-03 14:33                                   ` Kenneth Crudup
2025-03-03 17:58                                     ` Mika Westerberg
2025-03-03 18:20                                       ` Kenneth Crudup
2025-03-03 19:44                                         ` Kenneth Crudup
2025-03-04  8:27                                           ` Mika Westerberg
2025-03-04 12:52                                             ` Kenneth Crudup
2025-03-04 13:40                                               ` Mika Westerberg
2025-03-04 13:48                                                 ` Kenneth Crudup
2025-03-04 13:51                                                   ` Mika Westerberg
2025-03-04 17:29                                                     ` Kenneth Crudup
2025-03-05  8:31                                                       ` Mika Westerberg
2025-03-03 14:17                               ` Kenneth Crudup

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=20250303104618.GP3713119@black.fi.intel.com \
    --to=mika.westerberg@linux.intel.com \
    --cc=kenny@panix.com \
    --cc=linux-usb@vger.kernel.org \
    /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