Re: [PATCH 0/4] PCI: Consolidate TLP Log reading and printing

From: "Ilpo Järvinen" <ilpo.jarvinen@linux.intel.com>
To: linux-pci@vger.kernel.org,  "Wang,
	Qingshun" <qingshun.wang@linux.intel.com>
Cc: Bjorn Helgaas <bhelgaas@google.com>,
	 Jesse Brandeburg <jesse.brandeburg@intel.com>,
	 intel-wired-lan@lists.osuosl.org,
	Tony Nguyen <anthony.l.nguyen@intel.com>,
	 Ard Biesheuvel <ardb@kernel.org>, Borislav Petkov <bp@alien8.de>,
	 "David S. Miller" <davem@davemloft.net>,
	 Eric Dumazet <edumazet@google.com>,
	Jakub Kicinski <kuba@kernel.org>,
	 linux-edac@vger.kernel.org, linux-efi@vger.kernel.org,
	 LKML <linux-kernel@vger.kernel.org>,
	linuxppc-dev@lists.ozlabs.org,
	 Mahesh J Salgaonkar <mahesh@linux.ibm.com>,
	 Netdev <netdev@vger.kernel.org>,
	Oliver O'Halloran <oohall@gmail.com>,
	 Paolo Abeni <pabeni@redhat.com>, Tony Luck <tony.luck@intel.com>
Subject: Re: [PATCH 0/4] PCI: Consolidate TLP Log reading and printing
Date: Wed, 7 Feb 2024 13:50:37 +0200 (EET)	[thread overview]
Message-ID: <d1168e4a-c766-b3c2-bb74-c8dbae984cff@linux.intel.com> (raw)
In-Reply-To: <20240206135717.8565-1-ilpo.jarvinen@linux.intel.com>

[-- Attachment #1: Type: text/plain, Size: 1545 bytes --]

Adding Cc Quigshun which I ended up forgotting despite thinking it at one 
point.

-- 
 i.

On Tue, 6 Feb 2024, Ilpo Järvinen wrote:

> This series consolidates AER & DPC TLP Log handling code. Helpers are
> added for reading and printing the TLP Log and the format is made to
> include E-E Prefixes in both cases (previously only one DPC RP PIO
> displayed the E-E Prefixes).
> 
> I'd appreciate if people familiar with ixgbe could check the error
> handling conversion within the driver is correct.
> 
> Ilpo Järvinen (4):
>   PCI/AER: Cleanup register variable
>   PCI: Generalize TLP Header Log reading
>   PCI: Add TLP Prefix reading into pcie_read_tlp_log()
>   PCI: Create helper to print TLP Header and Prefix Log
> 
>  drivers/firmware/efi/cper.c                   |  4 +-
>  drivers/net/ethernet/intel/ixgbe/ixgbe_main.c | 39 +++------
>  drivers/pci/ats.c                             |  2 +-
>  drivers/pci/pci.c                             | 79 +++++++++++++++++++
>  drivers/pci/pci.h                             |  2 +-
>  drivers/pci/pcie/aer.c                        | 28 ++-----
>  drivers/pci/pcie/dpc.c                        | 31 ++++----
>  drivers/pci/probe.c                           | 14 ++--
>  include/linux/aer.h                           | 16 ++--
>  include/linux/pci.h                           |  2 +-
>  include/ras/ras_event.h                       | 10 +--
>  include/uapi/linux/pci_regs.h                 |  2 +
>  12 files changed, 145 insertions(+), 84 deletions(-)
> 
> 