Re: [PATCH v2 1/2] powerpc/mce: Avoid using irq_work_queue() in realmode

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

Hi Ganesh,

Thank you for the patch! Yet something to improve:

[auto build test ERROR on powerpc/next]
[also build test ERROR on v5.16-rc2 next-20211124]
[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-Avoid-using-irq_work_queue-in-realmode/20211124-130459
base:   https://git.kernel.org/pub/scm/linux/kernel/git/powerpc/linux.git next
config: powerpc-allnoconfig (https://download.01.org/0day-ci/archive/20211124/202111241736.ZgcO0Sk3-lkp@intel.com/config)
compiler: powerpc-linux-gcc (GCC) 11.2.0
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
        # https://github.com/0day-ci/linux/commit/bac24ec52edd7013115ad594974f64a30565266d
        git remote add linux-review https://github.com/0day-ci/linux
        git fetch --no-tags linux-review Ganesh-Goudar/powerpc-mce-Avoid-using-irq_work_queue-in-realmode/20211124-130459
        git checkout bac24ec52edd7013115ad594974f64a30565266d
        # save the config file to linux build tree
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.2.0 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/time.c: In function '____timer_interrupt':
>> arch/powerpc/kernel/time.c:598:25: error: implicit declaration of function 'mce_run_late_handlers' [-Werror=implicit-function-declaration]
     598 |                         mce_run_late_handlers();
         |                         ^~~~~~~~~~~~~~~~~~~~~
   cc1: all warnings being treated as errors


vim +/mce_run_late_handlers +598 arch/powerpc/kernel/time.c

   590	
   591		old_regs = set_irq_regs(regs);
   592	
   593		trace_timer_interrupt_entry(regs);
   594	
   595		if (test_irq_work_pending()) {
   596			clear_irq_work_pending();
   597			if (IS_ENABLED(CONFIG_PPC_BOOK3S_64))
 > 598				mce_run_late_handlers();
   599			irq_work_run();
   600		}
   601	
   602		now = get_tb();
   603		if (now >= *next_tb) {
   604			*next_tb = ~(u64)0;
   605			if (evt->event_handler)
   606				evt->event_handler(evt);
   607			__this_cpu_inc(irq_stat.timer_irqs_event);
   608		} else {
   609			now = *next_tb - now;
   610			if (now <= decrementer_max)
   611				set_dec(now);
   612			/* We may have raced with new irq work */
   613			if (test_irq_work_pending())
   614				set_dec(1);
   615			__this_cpu_inc(irq_stat.timer_irqs_others);
   616		}
   617	
   618		trace_timer_interrupt_exit(regs);
   619	
   620		set_irq_regs(old_regs);
   621	}
   622	EXPORT_SYMBOL(timer_interrupt);
   623	

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