On Fri, Mar 21, 2025 at 05:39:30PM -0500, Bjorn Helgaas wrote: > 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@xxxxxxxxxx> > > > > Signed-off-by: Jon Pan-Doh <pandoh@xxxxxxxxxx> > > > > Reported-by: Sargun Dhillon <sargun@xxxxxxxx> > > > > > > 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. Works for me! Thanx, Paul
