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 -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@xxxxxxxxxx For more info on Linux MM, see: http://www.linux-mm.org/ . Fight unfair telecom policy in Canada: sign http://dissolvethecrtc.ca/ Don't email: <a href=mailto:"dont@xxxxxxxxx";> email@xxxxxxxxx </a>