On Mon, Nov 04, 2024 at 01:05:31PM +0000, Shiju Jose wrote:
> More detailed explanation of PPR and memory sparing and use cases was added
> in Documentation/edac/memory_repair.rst, which is part of the last common
> patch ("EDAC: Add documentation for RAS feature control") added for
> documentation of various RAS features supported in this series. Was not sure
> the file to be part of this patch or not.
If the commit message doesn't contain a justification for a patch's existence,
why do you even bother sending it?
IOW, no redirections pls - just state here what the use case is in short. You
can always go nuts into details in the docs.
> persist_mode used to readback the value of persist_mode presently set. For
> eg. 1 - soft memory sparing for a sparing instance, though the CXL memory
> device supports both soft and hard sparing, which is configurable.
> persist_mode_avail used to return the temporary and permanent repair
> capability of the device.
Wait, sysfs does a one value per file thing. What does persist_mode_avail
give?
Surely you can't dump a list of all available modes...
