All of lore.kernel.org
 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: 200+ 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 11:59 ` Christian Zigotzky
2021-04-30 15:26 ` Deucher, Alexander
2021-04-30 15:26   ` Deucher, Alexander
2021-04-30 15:26   ` Deucher, Alexander
2021-04-30 15:56   ` Gustavo A. R. Silva
2021-04-30 15:56     ` Gustavo A. R. Silva
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-06 23:55   ` Gustavo A. R. Silva
2021-05-07  6:43   ` Christian Zigotzky
2021-05-07  6:43     ` Christian Zigotzky
2021-05-08 11:33     ` Christian Zigotzky
2021-05-08 11:33       ` Christian Zigotzky
2021-05-09 23:04       ` Gustavo A. R. Silva
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 12:48               ` Geert Uytterhoeven
2023-11-13 13:12               ` Christian Zigotzky
2023-11-13 13:12                 ` Christian Zigotzky
2023-11-14  9:45               ` Christian Zigotzky
2023-11-14  9:45                 ` Christian Zigotzky
2023-11-15  8:33                 ` Geert Uytterhoeven
2023-11-15  8:33                   ` Geert Uytterhoeven
2023-11-15  8:33                   ` Geert Uytterhoeven
2023-11-15  9:22                   ` Gerd Hoffmann
2023-11-15  9:22                     ` Gerd Hoffmann
2023-11-15  9:22                     ` Gerd Hoffmann
2023-11-16 13:13                     ` Geert Uytterhoeven
2023-11-16 13:13                       ` Geert Uytterhoeven
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
2021-11-04 21:42   ` [PATCH] drm/virtio: Fix NULL dereference error in virtio_gpu_poll Vivek Kasireddy
2021-11-05 18:05     ` Christian Zigotzky
2021-11-05 18:05       ` Christian Zigotzky
2021-11-08 12:56     ` Gerd Hoffmann
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 15:10         ` Christian Zigotzky
2021-11-09 16:58         ` Bjorn Helgaas
2021-11-09 16:58           ` Bjorn Helgaas
2021-11-09 22:40           ` Krzysztof Wilczyński
2021-11-09 22:40             ` Krzysztof Wilczyński
2021-11-09 23:05             ` Arnd Bergmann
2021-11-09 23:05               ` Arnd Bergmann
2021-11-09 23:18               ` Krzysztof Wilczyński
2021-11-09 23:18                 ` Krzysztof Wilczyński
2021-11-10 16:42                 ` Robert Święcki
2021-11-10 16:42                   ` Robert Święcki
2021-11-10  3:52             ` Damien Le Moal
2021-11-10  3:52               ` Damien Le Moal
2021-11-10 18:07       ` Christian Zigotzky
2021-11-10 18:07         ` Christian Zigotzky
2021-11-10 18:41         ` Bjorn Helgaas
2021-11-10 18:41           ` Bjorn Helgaas
2021-11-10 19:09           ` Marc Zyngier
2021-11-10 19:09             ` Marc Zyngier
2021-11-11  5:24             ` Christian Zigotzky
2021-11-11  5:24               ` Christian Zigotzky
2021-11-11  7:13               ` Marc Zyngier
2021-11-11  7:13                 ` Marc Zyngier
2021-11-11  7:47                 ` Christian Zigotzky
2021-11-11  7:47                   ` Christian Zigotzky
2021-11-11 10:20                   ` Marc Zyngier
2021-11-11 10:20                     ` Marc Zyngier
2021-11-11 10:44                     ` Christian Zigotzky
2021-11-11 10:44                       ` Christian Zigotzky
2021-11-11 11:24                       ` Marc Zyngier
2021-11-11 11:24                         ` Marc Zyngier
2021-11-11 11:54                         ` Christian Zigotzky
2021-11-11 11:54                           ` Christian Zigotzky
2021-11-11 22:21                     ` Olof Johansson
2021-11-11 22:21                       ` Olof Johansson
2021-11-12 11:49                       ` Segher Boessenkool
2021-11-12 11:49                         ` Segher Boessenkool
2021-11-11 17:39         ` Marc Zyngier
2021-11-11 17:39           ` Marc Zyngier
2021-11-12  9:40           ` Christian Zigotzky
2021-11-12  9:40             ` Christian Zigotzky
2021-11-12 10:11             ` Christian Zigotzky
2021-11-12 10:11               ` Christian Zigotzky
2021-11-12 11:00               ` Christian Zigotzky
2021-11-12 11:00                 ` Christian Zigotzky
2021-11-12 13:41             ` Marc Zyngier
2021-11-12 13:41               ` Marc Zyngier
2021-11-12 14:15               ` Christian Zigotzky
2021-11-12 14:15                 ` Christian Zigotzky
2021-11-12 14:46                 ` Marc Zyngier
2021-11-12 14:46                   ` Marc Zyngier
2021-11-12 15:01                   ` Christian Zigotzky
2021-11-12 15:01                     ` Christian Zigotzky
2021-11-12 15:05                     ` 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-12 16:14         ` [irqchip: irq/irqchip-fixes] of/irq: Don't ignore interrupt-controller when interrupt-map failed irqchip-bot for Marc Zyngier
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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.