Subject: Re: [PATCH] papr/scm: Add bad memory ranges to nvdimm bad ranges

From: Philip Li <[email-masked]>
Date: Mon, 20 Apr 2020 11:02:36 +0800

On Mon, Apr 13, 2020 at 04:50:38PM +0530, Santosh Sivaraj wrote:
> kbuild test robot <[email-masked]> writes:
> 
> > Hi Santosh,
> >
> > Thank you for the patch! Yet something to improve:
> >
> > [auto build test ERROR on powerpc/next]
> > [also build test ERROR on v5.7-rc1 next-20200412]
> > [if your patch is applied to the wrong git tree, please drop us a note to help
> > improve the system. BTW, we also suggest to use '--base' option to specify the
> > base tree in git format-patch, please see
> > https://stackoverflow.com/a/37406982]
> 
> This patch depends on "powerpc/mce: Add MCE notification chain" [1].
got it, thanks for input, though currently the bot is not able to figure
out this yet for two separated patch sets, here the --base may help.

> 
> [1]: https://lore.kernel.org/linuxppc-dev/[email-masked]/
> 
> Thanks,
> Santosh
> 
> >
> > url:    https://github.com/0day-ci/linux/commits/Santosh-Sivaraj/papr-scm-Add-bad-memory-ranges-to-nvdimm-bad-ranges/20200401-171233
> > base:   https://git.kernel.org/pub/scm/linux/kernel/git/powerpc/linux.git next
> > config: powerpc-allyesconfig (attached as .config)
> > compiler: powerpc64-linux-gcc (GCC) 9.3.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=9.3.0 make.cross ARCH=powerpc 
> >
> > If you fix the issue, kindly add following tag as appropriate
> > Reported-by: kbuild test robot <[email-masked]>
> >
> > All errors (new ones prefixed by >>):
> >
> >    arch/powerpc/platforms/pseries/papr_scm.c: In function 'papr_scm_init':
> >>> arch/powerpc/platforms/pseries/papr_scm.c:584:3: error: implicit declaration of function 'mce_register_notifier'; did you mean 'bus_register_notifier'? [-Werror=implicit-function-declaration]
> >      584 |   mce_register_notifier(&mce_ue_nb);
> >          |   ^~~~~~~~~~~~~~~~~~~~~
> >          |   bus_register_notifier
> >    arch/powerpc/platforms/pseries/papr_scm.c: In function 'papr_scm_exit':
> >>> arch/powerpc/platforms/pseries/papr_scm.c:592:2: error: implicit declaration of function 'mce_unregister_notifier'; did you mean 'bus_unregister_notifier'? [-Werror=implicit-function-declaration]
> >      592 |  mce_unregister_notifier(&mce_ue_nb);
> >          |  ^~~~~~~~~~~~~~~~~~~~~~~
> >          |  bus_unregister_notifier
> >    cc1: some warnings being treated as errors
> >
> > vim +584 arch/powerpc/platforms/pseries/papr_scm.c
> >
> >    577	
> >    578	static int __init papr_scm_init(void)
> >    579	{
> >    580		int ret;
> >    581	
> >    582		ret = platform_driver_register(&papr_scm_driver);
> >    583		if (!ret)
> >  > 584			mce_register_notifier(&mce_ue_nb);
> >    585	
> >    586		return ret;
> >    587	}
> >    588	module_init(papr_scm_init);
> >    589	
> >    590	static void __exit papr_scm_exit(void)
> >    591	{
> >  > 592		mce_unregister_notifier(&mce_ue_nb);
> >    593		platform_driver_unregister(&papr_scm_driver);
> >    594	}
> >    595	module_exit(papr_scm_exit);
> >    596	
> >
> > ---
> > 0-DAY CI Kernel Test Service, Intel Corporation
> > https://lists.01.org/hyperkitty/list/[email-masked]
>