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

[Karolina Stolarek <karolina.stolarek@xxxxxxxxxx>]

Hi Jon,

Sorry for catching this one so late:

On 14/02/2025 03:35, Jon Pan-Doh wrote:
> +
> +static struct attribute *aer_attrs[] __ro_after_init = {
					^^^^^^^^^^^^^^^
This is a copy-paste error. These attributes are in the read-only region 
and can't be written to, so please remove it in the next version.

Also, what value do we have to write to turn off ratelimiting 
completely? Can we handle that as a special case?

All the best,
Karolina

> +	&dev_attr_ratelimit_cor_log.attr,
> +	&dev_attr_ratelimit_uncor_log.attr,
> +	NULL
> +};