Re: [PATCH v5 5/8] PCI/AER: Rename struct aer_stats to aer_report

[Bjorn Helgaas <helgaas@kernel.org>]

On Fri, Mar 21, 2025 at 03:16:48PM -0700, Paul E. McKenney wrote:
> On Fri, Mar 21, 2025 at 05:01:15PM -0500, Bjorn Helgaas wrote:
> > On Thu, Mar 20, 2025 at 06:58:03PM -0700, Jon Pan-Doh wrote:
> > > Update name to reflect the broader definition of structs/variables that
> > > are stored (e.g. ratelimits). This is a preparatory patch for adding rate
> > > limit support.
> > > 
> > > Signed-off-by: Karolina Stolarek <karolina.stolarek@oracle.com>
> > > Signed-off-by: Jon Pan-Doh <pandoh@google.com>
> > > Reported-by: Sargun Dhillon <sargun@meta.com>
> > 
> > What did Sargun report?  Is there a bug fix in here?  Can we include a
> > URL to whatever Sargun reported?
> 
> He reported RCU CPU stall warnings and CSD-lock warnings internally
> within Meta, so sorry, no useful URL.

Oh, I see now how this happened via your ack email, Paul.  So I think
it would make sense for Jon to add Sargun's reported-by to "[PATCH v5
6/8] PCI/AER: Introduce ratelimit for error logs", along with a line
in the commit log connecting Sargun with the RCU CPU stall warnings,
etc., because that's the patch that actually addresses those warnings.

I wouldn't add it to the other patches because it's just confusing.

Bjorn