Re: [PATCH v8 1/7] PCI: Don't expose pcie_read_tlp_log() outside of PCI subsystem
From: Yazen Ghannam <yazen.ghannam@amd.com>
Date: 2025-01-08 16:05:41
Also in:
linux-pci, lkml
From: Yazen Ghannam <yazen.ghannam@amd.com>
Date: 2025-01-08 16:05:41
Also in:
linux-pci, lkml
On Wed, Dec 18, 2024 at 04:37:41PM +0200, Ilpo Järvinen wrote:
pcie_read_tlp_log() was exposed by the commit 0a5a46a6a61b ("PCI/AER:
Generalize TLP Header Log reading") but this is now considered a
mistake. No drivers outside of PCI subsystem should build their own
diagnostic logging but should rely on PCI core doing it for them.
There's currently one driver (ixgbe) doing it independently which was
the initial reason why the export was added but it was decided by the
PCI maintainer that it's something that should be eliminated.
Remove the unwanted EXPORT of pcie_read_tlp_log() and remove it from
include/linux/aer.h.
Link: https://lore.kernel.org/all/20240322193011.GA701027@bhelgaas/ (local)
Signed-off-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
Reviewed-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>Reviewed-by: Yazen Ghannam <yazen.ghannam@amd.com> Thanks, Yazen