Received-SPF: pass (google.com: domain of linux-kernel+bounces-92830-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) client-ip=2604:1380:4601:e00::3;
Precedence: bulk
MIME-Version: 1.0
References: <20240305113057.56468-1-pmenzel@molgen.mpg.de> <20240305182320.GA541715@bhelgaas>
 <AM9P193MB10797B556A8C4C4B389038D5D3222@AM9P193MB1079.EURP193.PROD.OUTLOOK.COM>
In-Reply-To: <AM9P193MB10797B556A8C4C4B389038D5D3222@AM9P193MB1079.EURP193.PROD.OUTLOOK.COM>
Reply-To: bjorn@helgaas.com
From: Bjorn Helgaas <bjorn.helgaas@gmail.com>
Date: Tue, 5 Mar 2024 12:32:42 -0600
Message-ID: <CABhMZUV7cTvAta--5T7DUE8m=L=wA3jDfdxELXCUdLxiwu5N4A@mail.gmail.com>
Subject: Re: [PATCH] PCI/DPC: Quirk PIO log size for Intel Raptor Lake Root Ports
To: Niels van Aert <nvaert1986@hotmail.com>
Cc: Bjorn Helgaas <helgaas@kernel.org>, Paul Menzel <pmenzel@molgen.mpg.de>, 
	Bjorn Helgaas <bhelgaas@google.com>, Mika Westerberg <mika.westerberg@linux.intel.com>, 
	"linux-pci@vger.kernel.org" <linux-pci@vger.kernel.org>, 
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

On Tue, Mar 5, 2024 at 12:28=E2=80=AFPM Niels van Aert <nvaert1986@hotmail.=
com> wrote:
>
> Bjorn,
>
> Feel free to add my name Niels van Aert and e-mail address.

Done, thanks very much!

> ________________________________
> From: Bjorn Helgaas <helgaas@kernel.org>
> Sent: Tuesday, March 5, 2024 7:23 PM
> To: Paul Menzel <pmenzel@molgen.mpg.de>
> Cc: Bjorn Helgaas <bhelgaas@google.com>; Mika Westerberg <mika.westerberg=
@linux.intel.com>; nvaert1986@hotmail.com <nvaert1986@hotmail.com>; linux-p=
ci@vger.kernel.org <linux-pci@vger.kernel.org>; linux-kernel@vger.kernel.or=
g <linux-kernel@vger.kernel.org>
> Subject: Re: [PATCH] PCI/DPC: Quirk PIO log size for Intel Raptor Lake Ro=
ot Ports
>
> On Tue, Mar 05, 2024 at 12:30:56PM +0100, Paul Menzel wrote:
> > Commit 5459c0b70467 ("PCI/DPC: Quirk PIO log size for certain Intel Roo=
t
> > Ports") and commit 3b8803494a06 ("PCI/DPC: Quirk PIO log size for Intel=
 Ice
> > Lake Root Ports") add quirks for Ice, Tiger and Alder Lake Root Ports.
> > System firmware for Raptor Lake still has the bug, so Linux logs the
> > warning below on several Raptor Lake systems like Dell Precision 3581 w=
ith
> > Intel Raptor Lake processor (0W18NX) system firmware/BIOS version 1.10.=
1.
> >
> >     pci 0000:00:07.0: [8086:a76e] type 01 class 0x060400
> >     pci 0000:00:07.0: PME# supported from D0 D3hot D3cold
> >     pci 0000:00:07.0: PTM enabled (root), 4ns granularity
> >     pci 0000:00:07.0: DPC: RP PIO log size 0 is invalid
> >     pci 0000:00:07.1: [8086:a73f] type 01 class 0x060400
> >     pci 0000:00:07.1: PME# supported from D0 D3hot D3cold
> >     pci 0000:00:07.1: PTM enabled (root), 4ns granularity
> >     pci 0000:00:07.1: DPC: RP PIO log size 0 is invalid
> >
> > So, apply the quirk for Raptor Lake Root Ports as well.
> >
> > This also enables the DPC driver to dump the RP PIO Log registers when =
DPC
> > is triggered.
> >
> > Closes: https://bugzilla.kernel.org/show_bug.cgi?id=3D218560
> > Cc: Mika Westerberg <mika.westerberg@linux.intel.com>
> > Cc: nvaert1986@hotmail.com
> > Signed-off-by: Paul Menzel <pmenzel@molgen.mpg.de>
>
> Added stable tag and applied to pci/dpc for v6.9 with tags:
>
>   Reported-by: <nvaert1986@hotmail.com>
>   Closes: https://bugzilla.kernel.org/show_bug.cgi?id=3D218560
>   Signed-off-by: Paul Menzel <pmenzel@molgen.mpg.de>
>   Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
>   Cc: <stable@vger.kernel.org>
>   Cc: Mika Westerberg <mika.westerberg@linux.intel.com>
>   Cc: <nvaert1986@hotmail.com>
>
> nvaert1986@, thanks very much for your report.  Let me know if you
> prefer to omit your email or add your real name.  Happy to do either.
>
> > ---
> >  drivers/pci/quirks.c | 2 ++
> >  1 file changed, 2 insertions(+)
> >
> > diff --git a/drivers/pci/quirks.c b/drivers/pci/quirks.c
> > index d797df6e5f3e..663d838fa861 100644
> > --- a/drivers/pci/quirks.c
> > +++ b/drivers/pci/quirks.c
> > @@ -6225,6 +6225,8 @@ DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_INTEL, 0x9=
a2b, dpc_log_size);
> >  DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_INTEL, 0x9a2d, dpc_log_size);
> >  DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_INTEL, 0x9a2f, dpc_log_size);
> >  DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_INTEL, 0x9a31, dpc_log_size);
> > +DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_INTEL, 0xa73f, dpc_log_size);
> > +DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_INTEL, 0xa76e, dpc_log_size);
> >  #endif
> >
> >  /*
> > --
> > 2.43.0
> >