Re: [PATCH v4 2/2] powerpc/mce: Remove per cpu variables from MCE handlers

[kernel test robot <lkp@intel.com>]

[-- Attachment #1: Type: text/plain, Size: 5011 bytes --]

Hi Ganesh,

Thank you for the patch! Yet something to improve:

[auto build test ERROR on powerpc/next]
[also build test ERROR on v5.11-rc4 next-20210122]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]

url:    https://github.com/0day-ci/linux/commits/Ganesh-Goudar/powerpc-mce-Reduce-the-size-of-event-arrays/20210124-191230
base:   https://git.kernel.org/pub/scm/linux/kernel/git/powerpc/linux.git next
config: powerpc-randconfig-r005-20210124 (attached as .config)
compiler: clang version 12.0.0 (https://github.com/llvm/llvm-project bd3a387ee76f58caa0d7901f3f84e9bb3d006f27)
reproduce (this is a W=1 build):
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # install powerpc cross compiling tool for clang build
        # apt-get install binutils-powerpc-linux-gnu
        # https://github.com/0day-ci/linux/commit/fab6401db419da33d1757ebf519f030ab758ae7a
        git remote add linux-review https://github.com/0day-ci/linux
        git fetch --no-tags linux-review Ganesh-Goudar/powerpc-mce-Reduce-the-size-of-event-arrays/20210124-191230
        git checkout fab6401db419da33d1757ebf519f030ab758ae7a
        # save the attached .config to linux build tree
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=powerpc 

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>

All errors (new ones prefixed by >>):

>> arch/powerpc/kernel/setup-common.c:940:2: error: implicit declaration of function 'mce_init' [-Werror,-Wimplicit-function-declaration]
           mce_init();
           ^
   1 error generated.


vim +/mce_init +940 arch/powerpc/kernel/setup-common.c

   847	
   848	/*
   849	 * Called into from start_kernel this initializes memblock, which is used
   850	 * to manage page allocation until mem_init is called.
   851	 */
   852	void __init setup_arch(char **cmdline_p)
   853	{
   854		kasan_init();
   855	
   856		*cmdline_p = boot_command_line;
   857	
   858		/* Set a half-reasonable default so udelay does something sensible */
   859		loops_per_jiffy = 500000000 / HZ;
   860	
   861		/* Unflatten the device-tree passed by prom_init or kexec */
   862		unflatten_device_tree();
   863	
   864		/*
   865		 * Initialize cache line/block info from device-tree (on ppc64) or
   866		 * just cputable (on ppc32).
   867		 */
   868		initialize_cache_info();
   869	
   870		/* Initialize RTAS if available. */
   871		rtas_initialize();
   872	
   873		/* Check if we have an initrd provided via the device-tree. */
   874		check_for_initrd();
   875	
   876		/* Probe the machine type, establish ppc_md. */
   877		probe_machine();
   878	
   879		/* Setup panic notifier if requested by the platform. */
   880		setup_panic();
   881	
   882		/*
   883		 * Configure ppc_md.power_save (ppc32 only, 64-bit machines do
   884		 * it from their respective probe() function.
   885		 */
   886		setup_power_save();
   887	
   888		/* Discover standard serial ports. */
   889		find_legacy_serial_ports();
   890	
   891		/* Register early console with the printk subsystem. */
   892		register_early_udbg_console();
   893	
   894		/* Setup the various CPU maps based on the device-tree. */
   895		smp_setup_cpu_maps();
   896	
   897		/* Initialize xmon. */
   898		xmon_setup();
   899	
   900		/* Check the SMT related command line arguments (ppc64). */
   901		check_smt_enabled();
   902	
   903		/* Parse memory topology */
   904		mem_topology_setup();
   905	
   906		/*
   907		 * Release secondary cpus out of their spinloops at 0x60 now that
   908		 * we can map physical -> logical CPU ids.
   909		 *
   910		 * Freescale Book3e parts spin in a loop provided by firmware,
   911		 * so smp_release_cpus() does nothing for them.
   912		 */
   913	#ifdef CONFIG_SMP
   914		smp_setup_pacas();
   915	
   916		/* On BookE, setup per-core TLB data structures. */
   917		setup_tlb_core_data();
   918	#endif
   919		/* Print various info about the machine that has been gathered so far. */
   920		print_system_info();
   921	
   922		/* Reserve large chunks of memory for use by CMA for KVM. */
   923		kvm_cma_reserve();
   924	
   925		/*  Reserve large chunks of memory for us by CMA for hugetlb */
   926		gigantic_hugetlb_cma_reserve();
   927	
   928		klp_init_thread_info(&init_task);
   929	
   930		init_mm.start_code = (unsigned long)_stext;
   931		init_mm.end_code = (unsigned long) _etext;
   932		init_mm.end_data = (unsigned long) _edata;
   933		init_mm.brk = klimit;
   934	
   935		mm_iommu_init(&init_mm);
   936		irqstack_early_init();
   937		exc_lvl_early_init();
   938		emergency_stack_init();
   939	
 > 940		mce_init();

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org

[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 25713 bytes --]