[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20250918230629.GA1928363@bhelgaas>
Date: Thu, 18 Sep 2025 18:06:29 -0500
From: Bjorn Helgaas <helgaas@...nel.org>
To: Shuai Xue <xueshuai@...ux.alibaba.com>
Cc: rostedt@...dmis.org, lukas@...ner.de, linux-pci@...r.kernel.org,
linux-kernel@...r.kernel.org, linux-edac@...r.kernel.org,
linux-trace-kernel@...r.kernel.org, ilpo.jarvinen@...ux.intel.com,
mattc@...estorage.com, Jonathan.Cameron@...wei.com,
bhelgaas@...gle.com, tony.luck@...el.com, bp@...en8.de,
mhiramat@...nel.org, mathieu.desnoyers@...icios.com,
oleg@...hat.com, naveen@...nel.org, davem@...emloft.net,
anil.s.keshavamurthy@...el.com, mark.rutland@....com,
peterz@...radead.org, tianruidong@...ux.alibaba.com
Subject: Re: [PATCH v9 0/2] add PCI hotplug and PCIe link tracepoint
On Wed, Jul 23, 2025 at 11:31:06AM +0800, Shuai Xue wrote:
> changes since v8:
> - rewrite commit log from Bjorn
> - move pci_hp_event to a common place (include/trace/events/pci.h) per Ilpo
> - rename hotplug event strings per Bjorn and Lukas
> - add PCIe link tracepoint per Bjorn, Lukas, and Ilpo
>
> Hotplug events are critical indicators for analyzing hardware health, and
> surprise link downs can significantly impact system performance and reliability.
> In addition, PCIe link speed degradation directly impacts system performance and
> often indicates hardware issues such as faulty devices, physical layer problems,
> or configuration errors.
>
> This patch set add PCI hotplug and PCIe link tracepoint to help analyze PCI
> hotplug events and PCIe link speed degradation.
>
> Shuai Xue (2):
> PCI: trace: Add a generic RAS tracepoint for hotplug event
> PCI: trace: Add a RAS tracepoint to monitor link speed changes
>
> drivers/pci/hotplug/pciehp_ctrl.c | 33 +++++++--
> drivers/pci/hotplug/pciehp_hpc.c | 5 +-
> drivers/pci/pci.c | 2 +-
> drivers/pci/pci.h | 12 ++-
> drivers/pci/pcie/bwctrl.c | 4 +-
> drivers/pci/probe.c | 10 ++-
> include/linux/pci.h | 1 +
> include/trace/events/pci.h | 119 ++++++++++++++++++++++++++++++
> include/uapi/linux/pci.h | 7 ++
> 9 files changed, 177 insertions(+), 16 deletions(-)
> create mode 100644 include/trace/events/pci.h
Hi Shuai,
I lost track of what happened with this series. It seemed like we
were pretty close but there were some trivial issues, and I don't
think we ever merged it. Did I miss something, or are you waiting on
me?
Bjorn
Powered by blists - more mailing lists