All of lore.kernel.org
 help / color / mirror / Atom feed
* [linux-rt-devel:linux-5.10.y-rt 180/279] kernel/time/hrtimer.c:2000:6: warning: no previous prototype for function 'cpu_chill'
@ 2021-02-20  8:27 kernel test robot
  2021-02-20  9:30 ` Philip Li
  0 siblings, 1 reply; 2+ messages in thread
From: kernel test robot @ 2021-02-20  8:27 UTC (permalink / raw)
  To: kbuild-all

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

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/rt/linux-rt-devel.git linux-5.10.y-rt
head:   822f2ce254f44f8e65ccb2cdbd7cf4cf2208a3ff
commit: d8f742a94a343c1e55119961ef3bc669d3aa3c00 [180/279] ARM: Allow to enable RT
config: arm-randconfig-r031-20210220 (attached as .config)
compiler: clang version 12.0.0 (https://github.com/llvm/llvm-project c9439ca36342fb6013187d0a69aef92736951476)
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 arm cross compiling tool for clang build
        # apt-get install binutils-arm-linux-gnueabi
        # https://git.kernel.org/pub/scm/linux/kernel/git/rt/linux-rt-devel.git/commit/?id=d8f742a94a343c1e55119961ef3bc669d3aa3c00
        git remote add linux-rt-devel https://git.kernel.org/pub/scm/linux/kernel/git/rt/linux-rt-devel.git
        git fetch --no-tags linux-rt-devel linux-5.10.y-rt
        git checkout d8f742a94a343c1e55119961ef3bc669d3aa3c00
        # save the attached .config to linux build tree
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=arm 

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

All warnings (new ones prefixed by >>):

   kernel/time/hrtimer.c:120:21: warning: initializer overrides prior initialization of this subobject [-Winitializer-overrides]
           [CLOCK_REALTIME]        = HRTIMER_BASE_REALTIME,
                                     ^~~~~~~~~~~~~~~~~~~~~
   kernel/time/hrtimer.c:118:27: note: previous initialization is here
           [0 ... MAX_CLOCKS - 1]  = HRTIMER_MAX_CLOCK_BASES,
                                     ^~~~~~~~~~~~~~~~~~~~~~~
   kernel/time/hrtimer.c:121:22: warning: initializer overrides prior initialization of this subobject [-Winitializer-overrides]
           [CLOCK_MONOTONIC]       = HRTIMER_BASE_MONOTONIC,
                                     ^~~~~~~~~~~~~~~~~~~~~~
   kernel/time/hrtimer.c:118:27: note: previous initialization is here
           [0 ... MAX_CLOCKS - 1]  = HRTIMER_MAX_CLOCK_BASES,
                                     ^~~~~~~~~~~~~~~~~~~~~~~
   kernel/time/hrtimer.c:122:21: warning: initializer overrides prior initialization of this subobject [-Winitializer-overrides]
           [CLOCK_BOOTTIME]        = HRTIMER_BASE_BOOTTIME,
                                     ^~~~~~~~~~~~~~~~~~~~~
   kernel/time/hrtimer.c:118:27: note: previous initialization is here
           [0 ... MAX_CLOCKS - 1]  = HRTIMER_MAX_CLOCK_BASES,
                                     ^~~~~~~~~~~~~~~~~~~~~~~
   kernel/time/hrtimer.c:123:17: warning: initializer overrides prior initialization of this subobject [-Winitializer-overrides]
           [CLOCK_TAI]             = HRTIMER_BASE_TAI,
                                     ^~~~~~~~~~~~~~~~
   kernel/time/hrtimer.c:118:27: note: previous initialization is here
           [0 ... MAX_CLOCKS - 1]  = HRTIMER_MAX_CLOCK_BASES,
                                     ^~~~~~~~~~~~~~~~~~~~~~~
>> kernel/time/hrtimer.c:2000:6: warning: no previous prototype for function 'cpu_chill' [-Wmissing-prototypes]
   void cpu_chill(void)
        ^
   kernel/time/hrtimer.c:2000:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
   void cpu_chill(void)
   ^
   static 
   kernel/time/hrtimer.c:428:20: warning: unused function 'debug_hrtimer_free' [-Wunused-function]
   static inline void debug_hrtimer_free(struct hrtimer *timer)
                      ^
   kernel/time/hrtimer.c:621:19: warning: unused function 'hrtimer_hres_active' [-Wunused-function]
   static inline int hrtimer_hres_active(void)
                     ^
   kernel/time/hrtimer.c:1726:20: warning: unused function '__hrtimer_peek_ahead_timers' [-Wunused-function]
   static inline void __hrtimer_peek_ahead_timers(void) { }
                      ^
   8 warnings generated.


vim +/cpu_chill +2000 kernel/time/hrtimer.c

edbeda46322fbc Al Viro         2017-06-07  1995  
ca2a97742c3689 Thomas Gleixner 2012-03-07  1996  #ifdef CONFIG_PREEMPT_RT
ca2a97742c3689 Thomas Gleixner 2012-03-07  1997  /*
ca2a97742c3689 Thomas Gleixner 2012-03-07  1998   * Sleep for 1 ms in hope whoever holds what we want will let it go.
ca2a97742c3689 Thomas Gleixner 2012-03-07  1999   */
ca2a97742c3689 Thomas Gleixner 2012-03-07 @2000  void cpu_chill(void)
ca2a97742c3689 Thomas Gleixner 2012-03-07  2001  {
ca2a97742c3689 Thomas Gleixner 2012-03-07  2002  	unsigned int freeze_flag = current->flags & PF_NOFREEZE;
ca2a97742c3689 Thomas Gleixner 2012-03-07  2003  	struct task_struct *self = current;
ca2a97742c3689 Thomas Gleixner 2012-03-07  2004  	ktime_t chill_time;
ca2a97742c3689 Thomas Gleixner 2012-03-07  2005  
ca2a97742c3689 Thomas Gleixner 2012-03-07  2006  	raw_spin_lock_irq(&self->pi_lock);
ca2a97742c3689 Thomas Gleixner 2012-03-07  2007  	self->saved_state = self->state;
ca2a97742c3689 Thomas Gleixner 2012-03-07  2008  	__set_current_state_no_track(TASK_UNINTERRUPTIBLE);
ca2a97742c3689 Thomas Gleixner 2012-03-07  2009  	raw_spin_unlock_irq(&self->pi_lock);
ca2a97742c3689 Thomas Gleixner 2012-03-07  2010  
ca2a97742c3689 Thomas Gleixner 2012-03-07  2011  	chill_time = ktime_set(0, NSEC_PER_MSEC);
ca2a97742c3689 Thomas Gleixner 2012-03-07  2012  
ca2a97742c3689 Thomas Gleixner 2012-03-07  2013  	current->flags |= PF_NOFREEZE;
ca2a97742c3689 Thomas Gleixner 2012-03-07  2014  	schedule_hrtimeout(&chill_time, HRTIMER_MODE_REL_HARD);
ca2a97742c3689 Thomas Gleixner 2012-03-07  2015  	if (!freeze_flag)
ca2a97742c3689 Thomas Gleixner 2012-03-07  2016  		current->flags &= ~PF_NOFREEZE;
ca2a97742c3689 Thomas Gleixner 2012-03-07  2017  
ca2a97742c3689 Thomas Gleixner 2012-03-07  2018  	raw_spin_lock_irq(&self->pi_lock);
ca2a97742c3689 Thomas Gleixner 2012-03-07  2019  	__set_current_state_no_track(self->saved_state);
ca2a97742c3689 Thomas Gleixner 2012-03-07  2020  	self->saved_state = TASK_RUNNING;
ca2a97742c3689 Thomas Gleixner 2012-03-07  2021  	raw_spin_unlock_irq(&self->pi_lock);
ca2a97742c3689 Thomas Gleixner 2012-03-07  2022  }
ca2a97742c3689 Thomas Gleixner 2012-03-07  2023  EXPORT_SYMBOL(cpu_chill);
ca2a97742c3689 Thomas Gleixner 2012-03-07  2024  #endif
ca2a97742c3689 Thomas Gleixner 2012-03-07  2025  

:::::: The code at line 2000 was first introduced by commit
:::::: ca2a97742c368981d127fcaf7699756da6233d97 rt: Introduce cpu_chill()

:::::: TO: Thomas Gleixner <tglx@linutronix.de>
:::::: CC: Sebastian Andrzej Siewior <bigeasy@linutronix.de>

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

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

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [linux-rt-devel:linux-5.10.y-rt 180/279] kernel/time/hrtimer.c:2000:6: warning: no previous prototype for function 'cpu_chill'
  2021-02-20  8:27 [linux-rt-devel:linux-5.10.y-rt 180/279] kernel/time/hrtimer.c:2000:6: warning: no previous prototype for function 'cpu_chill' kernel test robot
@ 2021-02-20  9:30 ` Philip Li
  0 siblings, 0 replies; 2+ messages in thread
From: Philip Li @ 2021-02-20  9:30 UTC (permalink / raw)
  To: kbuild-all

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

On Sat, Feb 20, 2021 at 04:27:35PM +0800, kernel test robot wrote:
> tree:   https://git.kernel.org/pub/scm/linux/kernel/git/rt/linux-rt-devel.git linux-5.10.y-rt
> head:   822f2ce254f44f8e65ccb2cdbd7cf4cf2208a3ff
> commit: d8f742a94a343c1e55119961ef3bc669d3aa3c00 [180/279] ARM: Allow to enable RT
> config: arm-randconfig-r031-20210220 (attached as .config)
> compiler: clang version 12.0.0 (https://github.com/llvm/llvm-project c9439ca36342fb6013187d0a69aef92736951476)
> 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 arm cross compiling tool for clang build
>         # apt-get install binutils-arm-linux-gnueabi
>         # https://git.kernel.org/pub/scm/linux/kernel/git/rt/linux-rt-devel.git/commit/?id=d8f742a94a343c1e55119961ef3bc669d3aa3c00
>         git remote add linux-rt-devel https://git.kernel.org/pub/scm/linux/kernel/git/rt/linux-rt-devel.git
>         git fetch --no-tags linux-rt-devel linux-5.10.y-rt
>         git checkout d8f742a94a343c1e55119961ef3bc669d3aa3c00
>         # save the attached .config to linux build tree
>         COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=arm 
> 
> If you fix the issue, kindly add following tag as appropriate
> Reported-by: kernel test robot <lkp@intel.com>
kindly ignore this report, we will remove the testing of 5.x.y-rt per
early feedback.

Thanks

> 
> All warnings (new ones prefixed by >>):
> 
>    kernel/time/hrtimer.c:120:21: warning: initializer overrides prior initialization of this subobject [-Winitializer-overrides]
>            [CLOCK_REALTIME]        = HRTIMER_BASE_REALTIME,
>                                      ^~~~~~~~~~~~~~~~~~~~~
>    kernel/time/hrtimer.c:118:27: note: previous initialization is here
>            [0 ... MAX_CLOCKS - 1]  = HRTIMER_MAX_CLOCK_BASES,
>                                      ^~~~~~~~~~~~~~~~~~~~~~~
>    kernel/time/hrtimer.c:121:22: warning: initializer overrides prior initialization of this subobject [-Winitializer-overrides]
>            [CLOCK_MONOTONIC]       = HRTIMER_BASE_MONOTONIC,
>                                      ^~~~~~~~~~~~~~~~~~~~~~
>    kernel/time/hrtimer.c:118:27: note: previous initialization is here
>            [0 ... MAX_CLOCKS - 1]  = HRTIMER_MAX_CLOCK_BASES,
>                                      ^~~~~~~~~~~~~~~~~~~~~~~
>    kernel/time/hrtimer.c:122:21: warning: initializer overrides prior initialization of this subobject [-Winitializer-overrides]
>            [CLOCK_BOOTTIME]        = HRTIMER_BASE_BOOTTIME,
>                                      ^~~~~~~~~~~~~~~~~~~~~
>    kernel/time/hrtimer.c:118:27: note: previous initialization is here
>            [0 ... MAX_CLOCKS - 1]  = HRTIMER_MAX_CLOCK_BASES,
>                                      ^~~~~~~~~~~~~~~~~~~~~~~
>    kernel/time/hrtimer.c:123:17: warning: initializer overrides prior initialization of this subobject [-Winitializer-overrides]
>            [CLOCK_TAI]             = HRTIMER_BASE_TAI,
>                                      ^~~~~~~~~~~~~~~~
>    kernel/time/hrtimer.c:118:27: note: previous initialization is here
>            [0 ... MAX_CLOCKS - 1]  = HRTIMER_MAX_CLOCK_BASES,
>                                      ^~~~~~~~~~~~~~~~~~~~~~~
> >> kernel/time/hrtimer.c:2000:6: warning: no previous prototype for function 'cpu_chill' [-Wmissing-prototypes]
>    void cpu_chill(void)
>         ^
>    kernel/time/hrtimer.c:2000:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
>    void cpu_chill(void)
>    ^
>    static 
>    kernel/time/hrtimer.c:428:20: warning: unused function 'debug_hrtimer_free' [-Wunused-function]
>    static inline void debug_hrtimer_free(struct hrtimer *timer)
>                       ^
>    kernel/time/hrtimer.c:621:19: warning: unused function 'hrtimer_hres_active' [-Wunused-function]
>    static inline int hrtimer_hres_active(void)
>                      ^
>    kernel/time/hrtimer.c:1726:20: warning: unused function '__hrtimer_peek_ahead_timers' [-Wunused-function]
>    static inline void __hrtimer_peek_ahead_timers(void) { }
>                       ^
>    8 warnings generated.
> 
> 
> vim +/cpu_chill +2000 kernel/time/hrtimer.c
> 
> edbeda46322fbc Al Viro         2017-06-07  1995  
> ca2a97742c3689 Thomas Gleixner 2012-03-07  1996  #ifdef CONFIG_PREEMPT_RT
> ca2a97742c3689 Thomas Gleixner 2012-03-07  1997  /*
> ca2a97742c3689 Thomas Gleixner 2012-03-07  1998   * Sleep for 1 ms in hope whoever holds what we want will let it go.
> ca2a97742c3689 Thomas Gleixner 2012-03-07  1999   */
> ca2a97742c3689 Thomas Gleixner 2012-03-07 @2000  void cpu_chill(void)
> ca2a97742c3689 Thomas Gleixner 2012-03-07  2001  {
> ca2a97742c3689 Thomas Gleixner 2012-03-07  2002  	unsigned int freeze_flag = current->flags & PF_NOFREEZE;
> ca2a97742c3689 Thomas Gleixner 2012-03-07  2003  	struct task_struct *self = current;
> ca2a97742c3689 Thomas Gleixner 2012-03-07  2004  	ktime_t chill_time;
> ca2a97742c3689 Thomas Gleixner 2012-03-07  2005  
> ca2a97742c3689 Thomas Gleixner 2012-03-07  2006  	raw_spin_lock_irq(&self->pi_lock);
> ca2a97742c3689 Thomas Gleixner 2012-03-07  2007  	self->saved_state = self->state;
> ca2a97742c3689 Thomas Gleixner 2012-03-07  2008  	__set_current_state_no_track(TASK_UNINTERRUPTIBLE);
> ca2a97742c3689 Thomas Gleixner 2012-03-07  2009  	raw_spin_unlock_irq(&self->pi_lock);
> ca2a97742c3689 Thomas Gleixner 2012-03-07  2010  
> ca2a97742c3689 Thomas Gleixner 2012-03-07  2011  	chill_time = ktime_set(0, NSEC_PER_MSEC);
> ca2a97742c3689 Thomas Gleixner 2012-03-07  2012  
> ca2a97742c3689 Thomas Gleixner 2012-03-07  2013  	current->flags |= PF_NOFREEZE;
> ca2a97742c3689 Thomas Gleixner 2012-03-07  2014  	schedule_hrtimeout(&chill_time, HRTIMER_MODE_REL_HARD);
> ca2a97742c3689 Thomas Gleixner 2012-03-07  2015  	if (!freeze_flag)
> ca2a97742c3689 Thomas Gleixner 2012-03-07  2016  		current->flags &= ~PF_NOFREEZE;
> ca2a97742c3689 Thomas Gleixner 2012-03-07  2017  
> ca2a97742c3689 Thomas Gleixner 2012-03-07  2018  	raw_spin_lock_irq(&self->pi_lock);
> ca2a97742c3689 Thomas Gleixner 2012-03-07  2019  	__set_current_state_no_track(self->saved_state);
> ca2a97742c3689 Thomas Gleixner 2012-03-07  2020  	self->saved_state = TASK_RUNNING;
> ca2a97742c3689 Thomas Gleixner 2012-03-07  2021  	raw_spin_unlock_irq(&self->pi_lock);
> ca2a97742c3689 Thomas Gleixner 2012-03-07  2022  }
> ca2a97742c3689 Thomas Gleixner 2012-03-07  2023  EXPORT_SYMBOL(cpu_chill);
> ca2a97742c3689 Thomas Gleixner 2012-03-07  2024  #endif
> ca2a97742c3689 Thomas Gleixner 2012-03-07  2025  
> 
> :::::: The code at line 2000 was first introduced by commit
> :::::: ca2a97742c368981d127fcaf7699756da6233d97 rt: Introduce cpu_chill()
> 
> :::::: TO: Thomas Gleixner <tglx@linutronix.de>
> :::::: CC: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
> 
> ---
> 0-DAY CI Kernel Test Service, Intel Corporation
> https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org


> _______________________________________________
> kbuild-all mailing list -- kbuild-all(a)lists.01.org
> To unsubscribe send an email to kbuild-all-leave(a)lists.01.org

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2021-02-20  9:30 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2021-02-20  8:27 [linux-rt-devel:linux-5.10.y-rt 180/279] kernel/time/hrtimer.c:2000:6: warning: no previous prototype for function 'cpu_chill' kernel test robot
2021-02-20  9:30 ` Philip Li

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.