From: Philip Li <philip.li@intel.com>
To: kbuild-all@lists.01.org
Subject: 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'
Date: Sat, 20 Feb 2021 17:30:29 +0800 [thread overview]
Message-ID: <20210220093029.GA69268@pl-dbox> (raw)
In-Reply-To: <202102201625.yt6cYNku-lkp@intel.com>
[-- 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
prev parent reply other threads:[~2021-02-20 9:30 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
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 message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20210220093029.GA69268@pl-dbox \
--to=philip.li@intel.com \
--cc=kbuild-all@lists.01.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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.