From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 3166BCCF9EA for ; Tue, 28 Oct 2025 23:33:45 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:Message-ID: Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:References: List-Owner; bh=tEpYgo6pIFjXvjA6f3uSjln+/ppfxHTqF9GiI/gkd/4=; b=UfpMkWH7NfLcrG dJniw01y98eA41W17UrnkUmIpqgzEWMnjGOr2yO7GDcRPMYlxabYp2wz3xO/04mjvhyOJpDZRaLL+ lic6LdMNgzU918PVhFPVl61Q4e6ylugjvx+SFjpb5t0gPP75Pgn9r8Hclk2LGqyXY9GPoUcDhpjEE irc7S/RHImDC6bJZ+n1fajdq5IurioxxXFR59ajCcCkob9WmemDFAfreQaP2100IkIoVf7S8H6SHt QTbzMSJdfTv3apvSqvUIbGC7YSON8byaIQ3M+tJyXyJ6SpKgmXIqvB05A6VfQoDRZuTJlpHfhp96N X64cXO3KDHdtABX/ojkw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98.2 #2 (Red Hat Linux)) id 1vDtC5-0000000GnqU-24Zo; Tue, 28 Oct 2025 23:33:37 +0000 Received: from tor.source.kernel.org ([2600:3c04:e001:324:0:1991:8:25]) by bombadil.infradead.org with esmtps (Exim 4.98.2 #2 (Red Hat Linux)) id 1vDtC4-0000000GnqO-1hS8 for linux-rockchip@lists.infradead.org; Tue, 28 Oct 2025 23:33:36 +0000 Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by tor.source.kernel.org (Postfix) with ESMTP id 7B4A060222; Tue, 28 Oct 2025 23:33:35 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id F27E0C4CEE7; Tue, 28 Oct 2025 23:33:34 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1761694415; bh=HUOsnm5Nk+6xTsUUMwpytKpcd3je4nJFnLJK5pa2yWw=; h=Date:From:To:Cc:Subject:In-Reply-To:From; b=QZBDyun8yiZBdSUzQHF1+2JJRcD4znNaEFpCr/zDqada5768JIuN44wzkJC+6msSh q7w55UUYDhZ+XA3yCU3qfZ1uIjNtiyug2HCW6jX9hoNH7V3G3bY98nAuA2++vlN9K8 wAlXc+k7Zl//pAJrqV/wInOao3V7lmsfOubcVuRSZ1ulRRN2bzla0nbDigJMqMkwg4 lYp7mBo4dWuUMn4i9PLfa/KQYLpRadb64qLud7B1zWmJxbjHfkam1+Obj5JpVMPSSy E48LLXKHAIdrUksbZjOSUqTnDLUuB9CB5ZJYZIIyRly1x7k8JGZimV7QnAl2+LyQ1z D5dy17RW2HUsA== Date: Tue, 28 Oct 2025 18:33:33 -0500 From: Bjorn Helgaas To: Christian Zigotzky Cc: Johan Hovold , linux-pci@vger.kernel.org, Manivannan Sadhasivam , FUKAUMI Naoki , Herve Codina , Diederik de Haas , Dragan Simic , linuxppc-dev@lists.ozlabs.org, linux-rockchip@lists.infradead.org, linux-kernel@vger.kernel.org, Bjorn Helgaas , Shawn Lin , Frank Li , "R.T.Dickinson" , mad skateman , hypexed@yahoo.com.au Subject: Re: [PATCH] PCI/ASPM: Enable only L0s and L1 for devicetree platforms Message-ID: <20251028233333.GA1467459@bhelgaas> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <3aa95b26-7801-476e-840f-5976b0ee11c1@xenosoft.de> X-BeenThere: linux-rockchip@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Upstream kernel work for Rockchip platforms List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "Linux-rockchip" Errors-To: linux-rockchip-bounces+linux-rockchip=archiver.kernel.org@lists.infradead.org On Mon, Oct 27, 2025 at 06:12:24PM +0100, Christian Zigotzky wrote: > Hi All, > > I activated CONFIG_PCIEASPM and CONFIG_PCIEASPM_DEFAULT again for the RC3 of > kernel 6.18. Unfortunately my AMD Radeon HD6870 doesn't work with the latest > patches. > > But that doesn't matter because we disable the above kernel options by > default. We don't need power management for PCI Express because of boot > issues and performance issues. It matters to me! The kernel should work correctly with or without CONFIG_PCIEASPM and any of the CONFIG_PCIEASPM_* settings. We can't expect users to know a magic combination of config settings to make things work. I assume AMD Radeon HD6870 is used on a variety of platforms, and I don't see other reports of ASPM problems with it, so I suspect the problem is something else. - v6.17 CONFIG_PCIEASPM=y, works OK - v6.18-rc3 CONFIG_PCIEASPM unset, works OK, as expected since we don't do anything with ASPM - v6.18-rc3 CONFIG_PCIEASPM=y, boot fails (this report, no logs) I looked at Hypexed's logs from https://github.com/chzigotzky/kernels/issues/17#issuecomment-3400419966, all of which worked fine: - 6.18.0-a8-dmesg.log, looks like CONFIG_PCIEASPM unset, so we would expect this to be fine. - 6.18.0-a7-dmesg.log, CONFIG_PCIEASPM=y, ASPM fully enabled, reported to work fine. Hardware name: pasemi,nemo PA6T 0x900102 A-EON Amigaone X1000 Found PA-PXP PCI host bridge. pci 0000:00:10.0: [1959:a002] type 01 class 0x060400 PCIe Root Port All the Root Ports are [1959:a002], and ASPM for 01:00.0 and 05:12.0 (apparently the only endpoints that advertise ASPM) seemed to work fine. - 6.18.0-a7-2-dmesg.log, looks like CONFIG_PCIEASPM unset, so we would expect this to be fine. So the only data point I see is that [1959:a002] seems to work. Christian, can you collect the output of "sudo lspci -vv" from your machine where CONFIG_PCIEASPM=y doesn't work? Doesn't matter what kernel you're running when you collect it. I assume your machine is this: Hardware name: varisys,CYRUS5040 e5500 0x80240012 CoreNet Generic Found FSL PCI host bridge at 0x0000000ffe200000. Firmware bus number: 0->1 Found FSL PCI host bridge at 0x0000000ffe201000. Firmware bus number: 0->8 fsl-pci ffe200000.pcie: PCI host bridge to bus 0000:00 pci 0000:00:00.0: [1957:0451] type 01 class 0x060400 PCIe Root Port pci 0000:01:00.0: [1002:6738] type 00 class 0x030000 PCIe Legacy Endpoint pci 0000:01:00.1: [1002:aa88] type 00 class 0x040300 PCIe Legacy Endpoint [1957:0451] Freescale (some kind of Root Port) [1002:6738] AMD Barts XT [Radeon HD 6870] [1002:aa88] AMD Barts HDMI Audio [Radeon HD 6790/6850/6870 / 7720 OEM] I don't see any real info about that Freescale Root Port. If you have a chance, could you try the patch below on top of v6.18-rc3 with CONFIG_PCIEASPM=y? diff --git a/drivers/pci/quirks.c b/drivers/pci/quirks.c index 214ed060ca1b..2b6d4e0958aa 100644 --- a/drivers/pci/quirks.c +++ b/drivers/pci/quirks.c @@ -2524,6 +2524,7 @@ static void quirk_disable_aspm_l0s_l1(struct pci_dev *dev) * disable both L0s and L1 for now to be safe. */ DECLARE_PCI_FIXUP_FINAL(PCI_VENDOR_ID_ASMEDIA, 0x1080, quirk_disable_aspm_l0s_l1); +DECLARE_PCI_FIXUP_FINAL(PCI_VENDOR_ID_FREESCALE, 0x0451, quirk_disable_aspm_l0s_l1); /* * Some Pericom PCIe-to-PCI bridges in reverse mode need the PCIe Retrain _______________________________________________ Linux-rockchip mailing list Linux-rockchip@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-rockchip