Messages in this thread | | | Date | Mon, 17 Nov 2014 12:15:04 +0100 | From | Tomasz Pala <> | Subject | Re: [PATCH] amd64_edac: Build module on x86-32 |
| |
On Wed, Nov 05, 2014 at 15:56:03 +0100, Borislav Petkov wrote:
>> There's not much of if related (system is running 3.14.4): > > CONFIG_EDAC_DEBUG gives additional debugging output and this is without > it but it doesn't matter - I see you have a K8 box.
Oh, I've misread the 'DEBUG' part. Unfortunately I've got only K8s to test.
> Just FYI though, one serious advantage of 64-bit is that gets orders of > magnitude more testing than 32-bit so if you still are contemplating a > 64-bit switch someday, remember that fact. :)
Yes, I'm aware of this and won't put 32-bit on new pieces of hardware. Well, it has always been a pain to support Old, Running, Important Systems;)
>> - the AMD64 families of memory controllers (K8 and F10h) >> + the AMD64 families of memory controllers, everything >= K8. >> >> "everything >= K8" mislead me. > > Ok, it is supposed to say, on everything K8 and later. K8 is what you > have. What would make it more understandable?
"K8, K10 and everything >= 15h"? To emphase there is a gap? Dunno...
> So here's an updated version of your patch:
That looks fine, thank you for your assistance and patience!
regards,
> --- > From: Tomasz Pala <gotar@polanet.pl> > Subject: [PATCH] amd64_edac: Build module on x86-32 > > By popular demand, enable amd64_edac on 32-bit too. > > Boris: > - update Kconfig text. > - add a warning on load which states that 32-bit configurations are unsupported. > > Signed-off-by: Tomasz Pala <gotar@polanet.pl> > Link: http://lkml.kernel.org/r/20141102102212.GA7034@polanet.pl > Signed-off-by: Borislav Petkov <bp@suse.de> > --- > drivers/edac/Kconfig | 6 +++--- > drivers/edac/amd64_edac.c | 5 +++++ > 2 files changed, 8 insertions(+), 3 deletions(-) > > diff --git a/drivers/edac/Kconfig b/drivers/edac/Kconfig > index 7072c2892d63..4316c9e955b3 100644 > --- a/drivers/edac/Kconfig > +++ b/drivers/edac/Kconfig > @@ -105,11 +105,11 @@ config EDAC_GHES > In doubt, say 'Y'. > > config EDAC_AMD64 > - tristate "AMD64 (Opteron, Athlon64) K8, F10h" > - depends on EDAC_MM_EDAC && AMD_NB && X86_64 && EDAC_DECODE_MCE > + tristate "AMD64 (Opteron, Athlon64)" > + depends on EDAC_MM_EDAC && AMD_NB && EDAC_DECODE_MCE > help > Support for error detection and correction of DRAM ECC errors on > - the AMD64 families of memory controllers (K8 and F10h) > + the AMD64 families (>= K8) of memory controllers. > > config EDAC_AMD64_ERROR_INJECTION > bool "Sysfs HW Error injection facilities" > diff --git a/drivers/edac/amd64_edac.c b/drivers/edac/amd64_edac.c > index 1a1d7c43a20f..17638d7cf5c2 100644 > --- a/drivers/edac/amd64_edac.c > +++ b/drivers/edac/amd64_edac.c > @@ -3035,6 +3035,11 @@ static int __init amd64_edac_init(void) > goto err_no_instances; > > setup_pci_device(); > + > +#ifdef CONFIG_X86_32 > + amd64_err("%s on 32-bit is unsupported. USE AT YOUR OWN RISK!\n", EDAC_MOD_STR); > +#endif > + > return 0; > > err_no_instances:
-- Tomasz Pala <gotar@pld-linux.org>
| |