Re: [PATCH v4 7/7] PCI/AER: Add sysfs attributes for log ratelimits

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Mar 20, 2025 at 6:02 PM Sathyanarayanan Kuppuswamy
<sathyanarayanan.kuppuswamy@xxxxxxxxxxxxxxx> wrote:
>
> On 3/20/25 1:20 AM, Jon Pan-Doh wrote:
> > +What:                /sys/bus/pci/devices/<dev>/aer/ratelimit_in_5secs_cor_log
>
> Why not just use ratelimit_cor_log and ratelimit_uncor_log? Any way the
> detail about 5 second window would be available in the Documentation.

This was the original name. It seems to have gone full circle so I'm not sure
how to resolve it. Personally, I think Karolina's suggestion is a good
compromise
of detail vs. verboseness:

- Jonathan: ratelimit_in_5secs_cor_log[1]
- Karolina: ratelimit_burst_cor_log[2]

[1] https://lore.kernel.org/linux-pci/CAMC_AXW9nE-q_8qqX+1KOeYdTQVoUDovY03aPbLGZBpe9HCcWQ@xxxxxxxxxxxxxx/
[2] https://lore.kernel.org/linux-pci/d75a96f1-5162-4ec4-971b-9ebd4cfa5447@xxxxxxxxxx/

Thanks,
Jon





[Index of Archives]     [DMA Engine]     [Linux Coverity]     [Linux USB]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Greybus]

  Powered by Linux