Re: [PATCH] RAS/CEC: Move non-debug attributes out of debugfs

From: Dave Hansen
Date: Thu Mar 23 2023 - 18:01:34 EST


On 3/23/23 14:52, Meyer, Kyle wrote:
>> Also, why *should* these be part of the stable sysfs ABI? What app is
>> using them? Why does it need them?
> We have system scripts that adjust decay_interval and action_threshold.
> They can't access those attributes when lockdown is enabled. If there is a
> more appropriate place for those attributes, please let me know.

Thanks for the info. That helps a bit. But, I'd also appreciate if you
could expand on this a little more. What "system scripts" are these?
Who is using them? What are they trying to accomplish?

We can try to find the best home for these attributes with that info in
hand, if it's not sysfs.

>> Why these two and only these two? What's left in debugfs?
> The other attributes (pfn and array) are used to test CEC. They are only
> created when RAS_CEC_DEBUG is enabled.

Oh, that's good info too. Can you please include that in some form in
your new changelog?