Re: [PATCH v8] PCI: hotplug: Add a generic RAS tracepoint for hotplug event

[Lukas Wunner <lukas@wunner.de>]

On Tue, May 20, 2025 at 03:52:56PM +0300, Ilpo Järvinen wrote:
> On Tue, 20 May 2025, Lukas Wunner wrote:
> > A link speed event could contain a "reason" field
> > which indicates why the link speed changed,
> > e.g. "hotplug", "autonomous", "thermal", "retrain", etc.
> > 
> > In other words, instead of mixing the infomation for hotplug
> > and link speed events together in one event, a separate link
> > speed event could point to hotplug as one possible reason for
> > the new speed.
> 
> It will be somewhat challenging to link LBMS into what caused it, 
> especially in cases where there is more than one LBMS following a single 
> Link Retraining.
> 
> Do you have opinion on should the event be only recorded from LBMS/LABS 
> if the speed changed from the previous value? The speed should probably 
> also be reported also for the first time (initial enumeration, hotplugging 
> a new board).

One idea would be to amend struct pcie_bwctrl_data with an
enum member describing the reason.

pcie_bwnotif_irq() uses that reason when reporting the speed change
in a trace event.

After an Endpoint has been removed, the Downstream Port or Root Port
above resets the reason to "hotplug", so that the next link event
is assigned that reason.

Similarly pcie_set_target_speed() could be amended with an enum argument
for the reason and it would set that in struct pcie_bwctrl_data before
calling pcie_bwctrl_change_speed().

Thanks,

Lukas