Re: [RFC] amd64_edac: syndromes loading

[Doug Thompson <norsk5@yahoo.com>]

--- On Thu, 11/5/09, Borislav Petkov <borislav.petkov@amd.com> wrote:

> From: Borislav Petkov <borislav.petkov@amd.com>
> Subject: Re: [RFC] amd64_edac: syndromes loading
> To: "H. Peter Anvin" <hpa@zytor.com>
> Cc: "Pavel Machek" <pavel@ucw.cz>, "Doug Thompson" <norsk5@yahoo.com>, "Ingo Molnar" <mingo@elte.hu>, "Thomas Gleixner" <tglx@linutronix.de>, "x86" <x86@kernel.org>, "LKML" <linux-kernel@vger.kernel.org>
> Date: Thursday, November 5, 2009, 6:27 AM
> Hi,
> 
> The good news is they've come up with a modified algorithm
> which will
> require a smaller table, roughly 1/4th the size of the
> current 10K one.
> Now, on a second thought and IMHO, we should simply add
> another .c file
> instantiating those two x4 and x8 tables statically and
> linking them
> into the edac code. This way you
> 
> 1) don't have the additional complexity of adding firmware
> handling code
> and thus don't add a dependency on the firmware API
> 
> 2) don't have to actually carry two firmware images with
> the kernel
> 
> See, the natural use case for those tables are big machines
> which do not
> care about 4K memory wasted in ECC decoding tables when the
> recovery
> from the missed early warnings of a failing DIMM module is
> much more
> expensive.

very well put.  when nodes that have 64 or 128 GIGA-bytes of RAM are involved, 4k or even 32kb "loss" is in the noise.

Features vs costs is always something to weigh. In some cases loss of 4k in a 1 Gb machine might not be tolerated (though I accept it).

doug t

> 
> Hmm... ?
> 
> -- 
> Regards/Gruss,
> Boris.
> 
> Operating | Advanced Micro Devices GmbH
>   System  | Karl-Hammerschmidt-Str. 34, 85609
> Dornach b. München, Germany
>  Research | Geschäftsführer: Andrew Bowd, Thomas M.
> McCoy, Giuliano Meroni
>   Center  | Sitz: Dornach, Gemeinde Aschheim,
> Landkreis München
>   (OSRC)  | Registergericht München, HRB Nr.
> 43632
> 
>