On Wed, Dec 22, 2010 at 06:35:40PM -0500, Seiji Aguchi wrote: > --- a/kernel/sysctl.c > +++ b/kernel/sysctl.c > @@ -81,6 +81,9 @@ > #include <linux/nmi.h> > #endif > > +#ifdef CONFIG_X86_MCE > +#include <asm/mce.h> > +#endif Please don't put ifdefs in .c files, you do that a lot for this option. Just make it work on all platforms and then you will not need the #ifdef. thanks, greg k-h