Hi Oliver, I love your patch! Yet something to improve: [auto build test ERROR on powerpc/next] [also build test ERROR on v5.0-rc4 next-20190207] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci/linux/commits/Oliver-O-Halloran/powerpc-eeh-Use-debugfs_create_u32-for-eeh_max_freezes/20190208-145918 base: https://git.kernel.org/pub/scm/linux/kernel/git/powerpc/linux.git next config: powerpc-allyesconfig (attached as .config) compiler: powerpc64-linux-gnu-gcc (Debian 8.2.0-11) 8.2.0 reproduce: wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross chmod +x ~/bin/make.cross # save the attached .config to linux build tree GCC_VERSION=8.2.0 make.cross ARCH=powerpc Note: the linux-review/Oliver-O-Halloran/powerpc-eeh-Use-debugfs_create_u32-for-eeh_max_freezes/20190208-145918 HEAD a8dcd44575537e3e67a44fe3139b273a64c0f620 builds fine. It only hurts bisectibility. All errors (new ones prefixed by >>): >> arch/powerpc/kernel/eeh.c:1840: error: unterminated #ifdef #ifdef CONFIG_DEBUG_FS vim +1840 arch/powerpc/kernel/eeh.c 7f52a526f arch/powerpc/kernel/eeh.c Gavin Shan 2014-04-24 1835 ^1da177e4 arch/ppc64/kernel/eeh.c Linus Torvalds 2005-04-16 1836 static int __init eeh_init_proc(void) ^1da177e4 arch/ppc64/kernel/eeh.c Linus Torvalds 2005-04-16 1837 { 7f52a526f arch/powerpc/kernel/eeh.c Gavin Shan 2014-04-24 1838 if (machine_is(pseries) || machine_is(powernv)) { 3f3942aca arch/powerpc/kernel/eeh.c Christoph Hellwig 2018-05-15 1839 proc_create_single("powerpc/eeh", 0, NULL, proc_eeh_show); 7f52a526f arch/powerpc/kernel/eeh.c Gavin Shan 2014-04-24 @1840 #ifdef CONFIG_DEBUG_FS :::::: The code at line 1840 was first introduced by commit :::::: 7f52a526f64c69c913f0027fbf43821ff0b3a7d7 powerpc/eeh: Allow to disable EEH :::::: TO: Gavin Shan :::::: CC: Benjamin Herrenschmidt --- 0-DAY kernel test infrastructure Open Source Technology Center https://lists.01.org/pipermail/kbuild-all Intel Corporation