Re: [PATCH v15 11/15] EDAC: Add memory repair control feature

From: Borislav Petkov
Date: Tue Nov 19 2024 - 07:38:03 EST


On Fri, Nov 15, 2024 at 12:21:16PM +0000, Shiju Jose wrote:
> >Ok, and how is the user supposed to know what those mean?
>
> Print in string format?, may be as 'persist'/'volatile'?

That sounds like an abuse of sysfs (Greg?) to me and even if it were possible,
you need explanation what those strings mean.

> I am fine with adding the support for expose the ranges of these,
> but makes more sense to do it when a driver surfaces that can do it.

So how do you envision to do it otherwise? The user is supposed to guess the
ranges?

That's not a good UI IMO.

--
Regards/Gruss,
Boris.

https://people.kernel.org/tglx/notes-about-netiquette