Messages in this thread | | | Date | Wed, 20 May 2009 09:51:56 -0700 | From | Randy Dunlap <> | Subject | Re: linux-next: Tree for May 20 (amd64_edac) |
| |
Borislav Petkov wrote: > Hi, > > On Wed, May 20, 2009 at 09:16:09AM -0700, Randy Dunlap wrote: >> Stephen Rothwell wrote: >>> Hi all, >>> >>> Changes since 20090519: >>> >>> Undropped tree: >>> edac-amd >>> >>> The edac-amd tree lost its build failure. >> >> When K8_NB is not enabled: >> >> drivers/built-in.o: In function `amd64_edac_init': >> amd64_edac.c:(.init.text+0x254f6): undefined reference to `cache_k8_northbridges' >> amd64_edac.c:(.init.text+0x25c38): undefined reference to `num_k8_northbridges' > > ah, well, one dependency goes another one comes in :(. Thanks, will fix... > >> Error with EDAC_AMD64 =y or =m. >> Error on both i386 and on x86_64. >> >> >> Comments on Kconfig for EDAC_AMD64: >> >> config EDAC_AMD64 >> tristate "AMD64 (Opteron, Athlon64) K8, F10h, F11h" >> depends on EDAC_MM_EDAC && X86 && PCI >> default m >> help >> Support for error detection and correction on the AMD 64 >> Families of Memory Controllers (K8, F10h and F11h) >> >> >> Does this actually apply to X86_32 or should the depends be on >> X86_64 instead of X86? > > and since K8_NB depends on X86_64, this answers that question :).
Yes.
>> The default y on EDAC_MM_EDAC and default m on EDAC_AMD64 should >> be removed. > > The reasoning behind those comes from Ingo: people who enable EDAC > should get the modules automatically on instead of going through the > different entries in the subtree. Which sounds kind of reasonable, no? > > See > > http://marc.info/?l=linux-kernel&m=124173363813847 > > for details.
Distros don't use defaults or defconfig anyway, and having them enabled bloats kernels. I'm sure that Linus has also expressed opinions about enabling options in this way, but I can't find it just now.
-- ~Randy LPC 2009, Sept. 23-25, Portland, Oregon http://linuxplumbersconf.org/2009/
| |