mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Cole Leavitt <cole@unwrap.rs>
To: Mika Westerberg <mika.westerberg@linux.intel.com>
Cc: "Bjorn Helgaas" <bhelgaas@google.com>,
	"Mika Westerberg" <westeri@kernel.org>,
	"Lukas Wunner" <lukas@wunner.de>,
	"Andreas Noever" <andreas.noever@gmail.com>,
	"Yehezkel Bernat" <YehezkelShB@gmail.com>,
	"Ilpo Järvinen" <ilpo.jarvinen@linux.intel.com>,
	linux-pci@vger.kernel.org, linux-usb@vger.kernel.org,
	linux-kernel@vger.kernel.org
Subject: Re: [PATCH 2/2] thunderbolt: Don't resume the software CM if the host controller is gone
Date: Sun,  8 Nov 2026 00:18:29 -0500	[thread overview]
Message-ID: <20261108051829.152803-1-cole@unwrap.rs> (raw)
In-Reply-To: <20261019042920.GX176164@black.igk.intel.com>

Hi Mika,

On Mon, Oct 19, 2026 at 06:29:20AM +0200, Mika Westerberg wrote:
> This is not supposed to happen with SW CM systems. I've seen a couple of
> times Lenovo BIOS has the PCIe root port hotplug enabled and that's against
> what the Barlow Ridge BIOS configuration checklist says. Please check if
> there is newer BIOS (Lenovo has been fixing these) and of not report it to
> Lenovo.

Thanks, and sorry for the slow reply. You were right. Every oops I
captured was on BIOS N4FET30W (1.11), where pciehp was bound to the
Barlow Ridge root port (0000:80:1b.4, "pciehp: Slot(25)"). The machine
now runs N4FET52W (1.33), and that root port no longer gets the hotplug
service: only PME, AER and bandwidth control are bound to it. So the
slow-link "replacement" that started all of this cannot happen there
anymore.

Please drop this patch.

For 1/2: the trigger on this machine is gone with the new BIOS, but the
ordering problem is in pciehp itself. A device replacement detected in
pciehp_resume_noirq() still starts the removal from the IRQ thread in
the middle of the noirq phase, unserialized against ->resume_noirq() of
the drivers below the port (on today's mainline, fa4ad798f6f6,
pciehp_core.c:310 calls pciehp_request(), and pciehp_ctrl.c:145 wakes
the IRQ thread directly unless pciehp_poll_mode is set). Bjorn, Lukas:
if that is not worth changing without a reproducer on correctly
configured hardware, I will drop 1/2 as well.

Thanks,
Cole

      reply	other threads:[~2026-11-08  5:18 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-10-18 16:37 [PATCH 0/2] PCI: pciehp / thunderbolt: Fix resume with a slow Thunderbolt host link Cole Leavitt
2026-10-18 16:39 ` [PATCH 1/2] PCI: pciehp: Defer device replacement handling until ->resume() Cole Leavitt
2026-10-18 16:39   ` [PATCH 2/2] thunderbolt: Don't resume the software CM if the host controller is gone Cole Leavitt
2026-10-19  4:29     ` Mika Westerberg
2026-11-08  5:18       ` Cole Leavitt [this message]

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=20261108051829.152803-1-cole@unwrap.rs \
    --to=cole@unwrap.rs \
    --cc=YehezkelShB@gmail.com \
    --cc=andreas.noever@gmail.com \
    --cc=bhelgaas@google.com \
    --cc=ilpo.jarvinen@linux.intel.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-pci@vger.kernel.org \
    --cc=linux-usb@vger.kernel.org \
    --cc=lukas@wunner.de \
    --cc=mika.westerberg@linux.intel.com \
    --cc=westeri@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

all inboxes | Powered by JetHome®