Re: [PATCH v3 2/2] EDAC/amd64: Set zn_regs_v2 for all Family 1Ah models

From: Borislav Petkov

Date: Sun Oct 04 2026 - 13:53:55 EST


On Sun, Oct 04, 2026 at 01:05:25PM -0400, Yazen Ghannam wrote:
> I'd rather we load unconditionally for all models with the same base
> behavior using sane defaults. Then we can have model-specific patches
> for variations, if needed.

I'd rather not because I keep getting all those: amd64_edac doesn't load on my
machine reports. Well, after a while it turns out that it should not load
there in the first place.

And then there's the managerial checkbox patch which needs to add support for
their new model just because... does it even make sense to add support? Oh, we
didn't even think of that but it says "Unsupported" so we thought we should
"fix" the error message...

So I don't want to have that unnecessary waste of everything. And if a f/m/s
would keep my sanity, then I'm perfectly fine with it.

> Essentially, we could avoid a whole class of patches for derivative
> (client, embedded, etc.) products. The module would load with the sane
> defaults. If the test folks find an issue, then we can have a
> model-specific patch.

Only on well-tested and supported configurations. Everything else doesn't
work.

--
Regards/Gruss,
Boris.

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