linuxppc-dev.lists.ozlabs.org archive mirror
 help / color / mirror / Atom feed
From: Bjorn Helgaas <helgaas@kernel.org>
To: Christian Zigotzky <chzigotzky@xenosoft.de>
Cc: Manivannan Sadhasivam <mani@kernel.org>,
	Bjorn Helgaas <bhelgaas@google.com>,
	linux-pci@vger.kernel.org, mad skateman <madskateman@gmail.com>,
	"R.T.Dickinson" <rtd2@xtra.co.nz>,
	Christian Zigotzky <info@xenosoft.de>,
	linuxppc-dev <linuxppc-dev@lists.ozlabs.org>,
	hypexed@yahoo.com.au, Darren Stevens <darren@stevens-zone.net>,
	debian-powerpc@lists.debian.org,
	John Paul Adrian Glaubitz <glaubitz@physik.fu-berlin.de>,
	Lukas Wunner <lukas@wunner.de>,
	regressions@lists.linux.dev,
	luigi burdo <intermediadc@hotmail.com>, Al <al@datazap.net>
Subject: Re: [PPC] Boot problems after the pci-v6.18-changes
Date: Wed, 12 Nov 2025 18:44:48 -0600	[thread overview]
Message-ID: <20251113004448.GA2251273@bhelgaas> (raw)
In-Reply-To: <f4331d1c-8695-19c3-608b-210b3242aaf0@xenosoft.de>

On Wed, Nov 12, 2025 at 04:40:18AM +0100, Christian Zigotzky wrote:
> On 11/11/2025 01:20 PM, Bjorn Helgaas wrote:
> > On Tue, Nov 11, 2025 at 06:15:20AM +0100, Christian Zigotzky wrote:
> >> On 11/07/2025 06:06 AM, Christian Zigotzky wrote:
> >>> On 11/05/2025 11:09 PM, Bjorn Helgaas wrote:
> >>>>> I tested your patch with the RC4 of kernel 6.18 today. Unfortunately
> >>> it
> >>>>> doesn't solve the boot issue.
> >>>>
> >>>> Thanks for testing that.  I see now why that approach doesn't work:
> >>>> quirk_disable_aspm_l0s_l1() calls pci_disable_link_state(), which
> >>>> updates the permissible ASPM link states, but pci_disable_link_state()
> >>>> only works for devices at the downstream end of a link.  It doesn't
> >>>> work at all for Root Ports, which are at the upstream end of a link.
> >>>>
> >>>> Christian, you originally reported that both X5000 and X1000 were
> >>>> broken.  I suspect X1000 may have been fixed in v6.18-rc3 by
> >>>> df5192d9bb0e ("PCI/ASPM: Enable only L0s and L1 for devicetree
> >>>> platforms"), but I would love to have confirmation of that.
> >>>
> >>> Hello Bjorn,
> >>>
> >>> I will enable CONFIG_PCIEASPM and CONFIG_PCIEASPM_DEFAULT for the RC5 of
> >>> kernel 6.18 and test it with the X1000.
> >>
> >> I tested the RC5 of kernel 6.18 with CONFIG_PCIEASPM and
> >> CONFIG_PCIEASPM_DEFAULT enabled on my X1000 today. Unfortunately the boot
> >> problems are still present.
> >
> > Thanks.  Can you post a dmesg somewhere so I can see what the relevant
> > device IDs are?  Can be with any kernel, doesn't have to be v6.18.  We
> > need the Vendor and Device IDs to add a quirk.
> 
> X1000 kernel 6.18.0-rc5 dmesg:
> https://github.com/user-attachments/files/23491291/dmesg_x1000.txt

Thanks!  This shows all the Root Ports are [1959:a002]:

  pci 0000:00:10.0: [1959:a002] type 01 class 0x060400 PCIe Root Port
  pci 0000:00:10.0: PCI bridge to [bus 01]
  pci 0000:01:00.0: [1002:6898] type 00 class 0x030000 PCIe Legacy Endpoint

I'm confused because Hypexed's 6.18.0-a7-dmesg.log from
https://github.com/chzigotzky/kernels/issues/17#issuecomment-3400419966
shows the same Root Ports, and apparently it booted fine even though
we enabled *everything*: 

  pci 0000:00:10.0: [1959:a002] type 01 class 0x060400 PCIe Root Port
  pci 0000:00:10.0: PCI bridge to [bus 01]
  pci 0000:01:00.0: [1002:6610] type 00 class 0x030000 PCIe Legacy Endpoint
  pci 0000:01:00.0: ASPM: DT platform, enabling L0s-up L0s-dw L1 ASPM-L1.1 ASPM-L1.2 PCI-PM-L1.1 PCI-PM-L1.2

It's *possible* that ASPM doesn't work on your endpoint (AMD Cypress
XT [Radeon HD 5870]) but does work on Hypexed's endpoint (AMD Oland XT
[Radeon HD 8670 / R5 340X OEM / R7 250/350/350X OEM]), but that seems
unlikely to me.

Maybe we just give up on ASPM on the [1959:a002] device.  A web search
doesn't show many users of it, so I don't know how many people would
care.

Bjorn


  reply	other threads:[~2025-11-13  0:44 UTC|newest]

Thread overview: 150+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-04-30 11:59 Radeon NI: GIT kernel with the nislands_smc commit doesn't boot on a Freescale P5040 board and P.A.Semi Nemo board Christian Zigotzky
2021-04-30 15:26 ` Deucher, Alexander
2021-04-30 15:56   ` Gustavo A. R. Silva
2021-05-03 22:25 ` [FSL P50x0] Xorg always restarts again and again after the the PowerPC updates 5.13-1 Christian Zigotzky
2021-05-04  4:56   ` Christophe Leroy
2021-05-04  7:21     ` Christian Zigotzky
2021-05-04  7:47       ` Christophe Leroy
2021-05-04  8:29         ` Christian Zigotzky
2021-05-04  8:58           ` Christophe Leroy
2021-05-04  9:09             ` Christian Zigotzky
2021-05-04  9:11               ` Christophe Leroy
2021-05-04  9:46                 ` Christian Zigotzky
2021-05-04  9:49                   ` Christophe Leroy
2021-05-04 10:07                     ` Christian Zigotzky
2021-05-04 11:02                       ` Christian Zigotzky
2021-05-04 13:48                         ` Christian Zigotzky
2021-05-04 14:48                           ` Christophe Leroy
2021-05-04 14:52                             ` Christian Zigotzky
2021-05-04 14:41                         ` Christophe Leroy
2021-05-04 14:59                           ` Christian Zigotzky
2021-05-04 15:17                             ` Christophe Leroy
2021-05-05 12:43                               ` Christian Zigotzky
2021-05-06  6:12                                 ` Christophe Leroy
2021-05-06  7:56                                   ` Christian Zigotzky
2021-05-06  8:09                                     ` Christophe Leroy
2021-05-06 13:58                                       ` Christian Zigotzky
2021-05-08 16:39                                         ` Christian Zigotzky
2021-05-09 12:27                                           ` Christian Zigotzky
2021-05-09 17:43                                             ` Christophe Leroy
2021-05-09 21:46                                               ` Christian Zigotzky
2021-05-09 17:36                                           ` Christophe Leroy
2021-05-13  6:47                                             ` Christian Zigotzky
2021-05-13 10:01                                               ` Christophe Leroy
2021-05-13 15:19                                                 ` Christian Zigotzky
2021-05-13 15:51                                                   ` Christophe Leroy
2021-05-13 16:35                                                     ` Christian Zigotzky
2021-05-13 17:00                                                       ` Christophe Leroy
2021-05-13 20:20                                                         ` Christian Zigotzky
2021-05-13 22:58                                                           ` Nicholas Piggin
2021-05-14  0:19                                                             ` Christian Zigotzky
2021-05-04 14:44                   ` Christophe Leroy
2021-05-06 23:55 ` Radeon NI: GIT kernel with the nislands_smc commit doesn't boot on a Freescale P5040 board and P.A.Semi Nemo board Gustavo A. R. Silva
2021-05-07  6:43   ` Christian Zigotzky
2021-05-08 11:33     ` Christian Zigotzky
2021-05-09 23:04       ` Gustavo A. R. Silva
2023-11-02 14:45       ` Fbdev issue after the drm updates 'drm-next-2023-10-31-1' Christian Zigotzky
2023-11-07  8:36         ` Christian Zigotzky
2023-11-12 14:23           ` Christian Zigotzky
2023-11-13 12:48             ` Geert Uytterhoeven
2023-11-13 13:12               ` Christian Zigotzky
2023-11-14  9:45               ` Christian Zigotzky
2023-11-15  8:33                 ` Geert Uytterhoeven
2023-11-15  9:22                   ` Gerd Hoffmann
2023-11-16 13:13                     ` Geert Uytterhoeven
2021-11-04 10:45 ` [VirtIO GPU] Xorg doesn't start with the DRM updates 'drm-next-2021-11-03' in a virtual e5500 QEMU KVM-HV machine on a Freescale P5040 board Christian Zigotzky
     [not found]   ` <20211104214249.1802789-1-vivek.kasireddy@intel.com>
2021-11-05 18:05     ` [PATCH] drm/virtio: Fix NULL dereference error in virtio_gpu_poll Christian Zigotzky
2021-11-08 14:48   ` [FSL P50x0] Cyrus+ board doesn't boot with the PowerPC updates 5.16-1 Christian Zigotzky
2021-11-09 14:45     ` [PASEMI] Nemo board doesn't recognize any ATA disks with the pci-v5.16 updates Christian Zigotzky
2021-11-09 15:10       ` Christian Zigotzky
2021-11-09 16:58         ` Bjorn Helgaas
2021-11-09 22:40           ` Krzysztof Wilczyński
2021-11-09 23:05             ` Arnd Bergmann
2021-11-09 23:18               ` Krzysztof Wilczyński
2021-11-10 16:42                 ` Robert Święcki
2021-11-10  3:52             ` Damien Le Moal
2021-11-10 18:07       ` Christian Zigotzky
2021-11-10 18:41         ` Bjorn Helgaas
2021-11-10 19:09           ` Marc Zyngier
2021-11-11  5:24             ` Christian Zigotzky
2021-11-11  7:13               ` Marc Zyngier
2021-11-11  7:47                 ` Christian Zigotzky
2021-11-11 10:20                   ` Marc Zyngier
2021-11-11 10:44                     ` Christian Zigotzky
2021-11-11 11:24                       ` Marc Zyngier
2021-11-11 11:54                         ` Christian Zigotzky
2021-11-11 22:21                     ` Olof Johansson
2021-11-12 11:49                       ` Segher Boessenkool
2021-11-11 17:39         ` Marc Zyngier
2021-11-12  9:40           ` Christian Zigotzky
2021-11-12 10:11             ` Christian Zigotzky
2021-11-12 11:00               ` Christian Zigotzky
2021-11-12 13:41             ` Marc Zyngier
2021-11-12 14:15               ` Christian Zigotzky
2021-11-12 14:46                 ` Marc Zyngier
2021-11-12 15:01                   ` Christian Zigotzky
2021-11-12 15:05                     ` Christian Zigotzky
2025-10-08 16:35                       ` [PPC] Boot problems after the pci-v6.18-changes Christian Zigotzky
2025-10-08 16:40                         ` John Paul Adrian Glaubitz
2025-10-08 16:47                           ` Christian Zigotzky
2025-10-08 19:51                         ` Bjorn Helgaas
2025-10-09  4:54                           ` Christian Zigotzky
2025-10-09  5:37                             ` Lukas Wunner
2025-10-11  5:12                               ` Christian Zigotzky
2025-10-11  5:25                                 ` Lukas Wunner
2025-10-11 15:11                                   ` Manivannan Sadhasivam
2025-10-11 19:34                                     ` Christian Zigotzky
2025-10-12  7:47                                       ` Christian Zigotzky
2025-10-12  8:26                                         ` Christian Zigotzky
2025-10-13  4:46                                     ` Christian Zigotzky
2025-10-13  5:02                                       ` Christian Zigotzky
2025-10-13  5:23                                         ` Christian Zigotzky
2025-10-13 14:50                                           ` Christian Zigotzky
2025-10-13 15:02                                             ` Christian Zigotzky
2025-10-13 15:38                                               ` Christian Zigotzky
2025-10-13 15:58                                             ` Manivannan Sadhasivam
2025-10-14  4:48                                               ` Christian Zigotzky
2025-10-14  4:55                                                 ` Christian Zigotzky
2025-10-15  6:41                                                   ` Manivannan Sadhasivam
2025-10-13 16:01                                         ` Manivannan Sadhasivam
2025-10-15  8:13                                     ` Herve Codina
2025-10-15 11:30                                       ` Christian Zigotzky
2025-10-15 11:58                                         ` Herve Codina
2025-10-15 12:27                                           ` Christian Zigotzky
2025-10-15 12:59                                             ` Herve Codina
2025-10-15 13:14                                               ` Christian Zigotzky
2025-10-15 13:17                                                 ` Christian Zigotzky
2025-10-15 13:34                                                 ` Herve Codina
2025-10-15 16:54                                                   ` Christian Zigotzky
2025-10-15 17:15                                                     ` Christian Zigotzky
2025-10-16  3:28                                                   ` Christian Zigotzky
2025-10-16  7:36                                                     ` Christian Zigotzky
2025-10-16  7:53                                                       ` Manivannan Sadhasivam
2025-10-16 10:44                                                         ` Christian Zigotzky
2025-10-17  5:35                                                           ` Christian Zigotzky
2025-10-15 16:23                                               ` Manivannan Sadhasivam
2025-10-15 13:07                                             ` Christian Zigotzky
2025-10-15 12:50                                           ` Manivannan Sadhasivam
2025-10-23  7:38                                             ` Herve Codina
2025-10-23  8:49                                               ` Manivannan Sadhasivam
2025-10-23  9:19                                                 ` Herve Codina
2025-10-23 16:59                                                   ` Bjorn Helgaas
2025-10-15 23:40                                       ` Bjorn Helgaas
2025-10-13 21:06                         ` Bjorn Helgaas
2025-10-14 18:51                           ` Bjorn Helgaas
2025-10-30 22:11                           ` Bjorn Helgaas
2025-11-01  5:39                             ` Christian Zigotzky
2025-11-01  7:50                               ` Christian Zigotzky
2025-11-01  7:59                               ` Christian Zigotzky
2025-11-01 17:06                                 ` Manivannan Sadhasivam
2025-11-03 18:28                                   ` Christian Zigotzky
2025-11-05 22:09                                     ` Bjorn Helgaas
2025-11-06  8:48                                       ` Hongxing Zhu
2025-11-06 18:25                                         ` Bjorn Helgaas
2025-11-07  2:26                                           ` Hongxing Zhu
2025-11-07  5:06                                       ` Christian Zigotzky
2025-11-11  5:15                                         ` Christian Zigotzky
2025-11-11 12:20                                           ` Bjorn Helgaas
2025-11-12  3:40                                             ` Christian Zigotzky
2025-11-13  0:44                                               ` Bjorn Helgaas [this message]
2021-11-09 16:13     ` [FSL P50x0] Cyrus+ board doesn't boot with the PowerPC updates 5.16-1 Christophe Leroy

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=20251113004448.GA2251273@bhelgaas \
    --to=helgaas@kernel.org \
    --cc=al@datazap.net \
    --cc=bhelgaas@google.com \
    --cc=chzigotzky@xenosoft.de \
    --cc=darren@stevens-zone.net \
    --cc=debian-powerpc@lists.debian.org \
    --cc=glaubitz@physik.fu-berlin.de \
    --cc=hypexed@yahoo.com.au \
    --cc=info@xenosoft.de \
    --cc=intermediadc@hotmail.com \
    --cc=linux-pci@vger.kernel.org \
    --cc=linuxppc-dev@lists.ozlabs.org \
    --cc=lukas@wunner.de \
    --cc=madskateman@gmail.com \
    --cc=mani@kernel.org \
    --cc=regressions@lists.linux.dev \
    --cc=rtd2@xtra.co.nz \
    /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;
as well as URLs for NNTP newsgroup(s).