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-...@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 <l...@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-...@lists.01.org

Reply via email to