public inbox for llvm@lists.linux.dev
 help / color / mirror / Atom feed
From: kernel test robot <lkp@intel.com>
To: Aaron Tomlin <atomlin@redhat.com>
Cc: llvm@lists.linux.dev, kbuild-all@lists.01.org
Subject: Re: [RFC PATCH v2] tick/sched: Ensure quiet_vmstat() is called when the idle tick was stopped too
Date: Sat, 23 Apr 2022 00:54:17 +0800	[thread overview]
Message-ID: <202204230047.FesC3TWd-lkp@intel.com> (raw)
In-Reply-To: <20220422112053.3695526-1-atomlin@redhat.com>

Hi Aaron,

[FYI, it's a private test report for your RFC patch.]
[auto build test ERROR on v5.18-rc3]
[also build test ERROR on next-20220422]
[cannot apply to tip/timers/nohz]
[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/intel-lab-lkp/linux/commits/Aaron-Tomlin/tick-sched-Ensure-quiet_vmstat-is-called-when-the-idle-tick-was-stopped-too/20220422-192320
base:    b2d229d4ddb17db541098b83524d901257e93845
config: x86_64-randconfig-a016 (https://download.01.org/0day-ci/archive/20220423/202204230047.FesC3TWd-lkp@intel.com/config)
compiler: clang version 15.0.0 (https://github.com/llvm/llvm-project 5bd87350a5ae429baf8f373cb226a57b62f87280)
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/intel-lab-lkp/linux/commit/0be2b64445273bddc478567c0846c1046f28995a
        git remote add linux-review https://github.com/intel-lab-lkp/linux
        git fetch --no-tags linux-review Aaron-Tomlin/tick-sched-Ensure-quiet_vmstat-is-called-when-the-idle-tick-was-stopped-too/20220422-192320
        git checkout 0be2b64445273bddc478567c0846c1046f28995a
        # save the config file
        mkdir build_dir && cp config build_dir/.config
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=x86_64 SHELL=/bin/bash drivers/media/i2c/ kernel/

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 >>):

   In file included from kernel/cpu.c:13:
   In file included from include/linux/sched/isolation.h:6:
>> include/linux/tick.h:313:22: error: use of undeclared identifier 'tick_cpu_sched'
                   ts = this_cpu_ptr(&tick_cpu_sched);
                                      ^
>> include/linux/tick.h:313:22: error: use of undeclared identifier 'tick_cpu_sched'
>> include/linux/tick.h:313:22: error: use of undeclared identifier 'tick_cpu_sched'
>> include/linux/tick.h:313:6: error: assigning to 'struct tick_sched *' from incompatible type 'void'
                   ts = this_cpu_ptr(&tick_cpu_sched);
                      ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>> include/linux/tick.h:315:9: error: incomplete definition of type 'struct tick_sched'
                   if (ts->tick_stopped)
                       ~~^
   include/linux/tick.h:310:9: note: forward declaration of 'struct tick_sched'
           struct tick_sched *ts;
                  ^
>> include/linux/tick.h:316:4: error: call to undeclared function 'quiet_vmstat'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]
                           quiet_vmstat();
                           ^
   In file included from kernel/cpu.c:18:
   In file included from include/linux/oom.h:11:
   In file included from include/linux/mm.h:1719:
>> include/linux/vmstat.h:299:6: error: conflicting types for 'quiet_vmstat'
   void quiet_vmstat(void);
        ^
   include/linux/tick.h:316:4: note: previous implicit declaration is here
                           quiet_vmstat();
                           ^
   7 errors generated.
--
   In file included from kernel/softirq.c:27:
>> include/linux/tick.h:313:22: error: use of undeclared identifier 'tick_cpu_sched'
                   ts = this_cpu_ptr(&tick_cpu_sched);
                                      ^
>> include/linux/tick.h:313:22: error: use of undeclared identifier 'tick_cpu_sched'
>> include/linux/tick.h:313:22: error: use of undeclared identifier 'tick_cpu_sched'
>> include/linux/tick.h:313:6: error: assigning to 'struct tick_sched *' from incompatible type 'void'
                   ts = this_cpu_ptr(&tick_cpu_sched);
                      ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>> include/linux/tick.h:315:9: error: incomplete definition of type 'struct tick_sched'
                   if (ts->tick_stopped)
                       ~~^
   include/linux/tick.h:310:9: note: forward declaration of 'struct tick_sched'
           struct tick_sched *ts;
                  ^
   5 errors generated.
--
   In file included from kernel/irq_work.c:17:
>> include/linux/tick.h:313:22: error: use of undeclared identifier 'tick_cpu_sched'
                   ts = this_cpu_ptr(&tick_cpu_sched);
                                      ^
>> include/linux/tick.h:313:22: error: use of undeclared identifier 'tick_cpu_sched'
>> include/linux/tick.h:313:22: error: use of undeclared identifier 'tick_cpu_sched'
>> include/linux/tick.h:313:6: error: assigning to 'struct tick_sched *' from incompatible type 'void'
                   ts = this_cpu_ptr(&tick_cpu_sched);
                      ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>> include/linux/tick.h:315:9: error: incomplete definition of type 'struct tick_sched'
                   if (ts->tick_stopped)
                       ~~^
   include/linux/tick.h:310:9: note: forward declaration of 'struct tick_sched'
           struct tick_sched *ts;
                  ^
>> include/linux/tick.h:316:4: error: call to undeclared function 'quiet_vmstat'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]
                           quiet_vmstat();
                           ^
   6 errors generated.
--
   In file included from kernel/sched/core.c:30:
   In file included from include/linux/sched/isolation.h:6:
>> include/linux/tick.h:313:22: error: use of undeclared identifier 'tick_cpu_sched'
                   ts = this_cpu_ptr(&tick_cpu_sched);
                                      ^
>> include/linux/tick.h:313:22: error: use of undeclared identifier 'tick_cpu_sched'
>> include/linux/tick.h:313:22: error: use of undeclared identifier 'tick_cpu_sched'
>> include/linux/tick.h:313:6: error: assigning to 'struct tick_sched *' from incompatible type 'void'
                   ts = this_cpu_ptr(&tick_cpu_sched);
                      ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>> include/linux/tick.h:315:9: error: incomplete definition of type 'struct tick_sched'
                   if (ts->tick_stopped)
                       ~~^
   include/linux/tick.h:310:9: note: forward declaration of 'struct tick_sched'
           struct tick_sched *ts;
                  ^
   kernel/sched/core.c:3518:6: warning: no previous prototype for function 'sched_set_stop_task' [-Wmissing-prototypes]
   void sched_set_stop_task(int cpu, struct task_struct *stop)
        ^
   kernel/sched/core.c:3518:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
   void sched_set_stop_task(int cpu, struct task_struct *stop)
   ^
   static 
   kernel/sched/core.c:5235:20: warning: no previous prototype for function 'task_sched_runtime' [-Wmissing-prototypes]
   unsigned long long task_sched_runtime(struct task_struct *p)
                      ^
   kernel/sched/core.c:5235:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
   unsigned long long task_sched_runtime(struct task_struct *p)
   ^
   static 
   kernel/sched/core.c:8952:6: warning: no previous prototype for function 'idle_task_exit' [-Wmissing-prototypes]
   void idle_task_exit(void)
        ^
   kernel/sched/core.c:8952:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
   void idle_task_exit(void)
   ^
   static 
   kernel/sched/core.c:9187:5: warning: no previous prototype for function 'sched_cpu_activate' [-Wmissing-prototypes]
   int sched_cpu_activate(unsigned int cpu)
       ^
   kernel/sched/core.c:9187:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
   int sched_cpu_activate(unsigned int cpu)
   ^
   static 
   kernel/sched/core.c:9232:5: warning: no previous prototype for function 'sched_cpu_deactivate' [-Wmissing-prototypes]
   int sched_cpu_deactivate(unsigned int cpu)
       ^
   kernel/sched/core.c:9232:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
   int sched_cpu_deactivate(unsigned int cpu)
   ^
   static 
   kernel/sched/core.c:9307:5: warning: no previous prototype for function 'sched_cpu_starting' [-Wmissing-prototypes]
   int sched_cpu_starting(unsigned int cpu)
       ^
   kernel/sched/core.c:9307:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
   int sched_cpu_starting(unsigned int cpu)
   ^
   static 
   kernel/sched/core.c:9328:5: warning: no previous prototype for function 'sched_cpu_wait_empty' [-Wmissing-prototypes]
   int sched_cpu_wait_empty(unsigned int cpu)
       ^
   kernel/sched/core.c:9328:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
   int sched_cpu_wait_empty(unsigned int cpu)
   ^
   static 
   kernel/sched/core.c:9370:5: warning: no previous prototype for function 'sched_cpu_dying' [-Wmissing-prototypes]
   int sched_cpu_dying(unsigned int cpu)
       ^
   kernel/sched/core.c:9370:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
   int sched_cpu_dying(unsigned int cpu)
   ^
   static 
   kernel/sched/core.c:9393:13: warning: no previous prototype for function 'sched_init_smp' [-Wmissing-prototypes]
   void __init sched_init_smp(void)
               ^
   kernel/sched/core.c:9393:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
   void __init sched_init_smp(void)
   ^
   static 
   kernel/sched/core.c:9454:13: warning: no previous prototype for function 'sched_init' [-Wmissing-prototypes]
   void __init sched_init(void)
               ^
   kernel/sched/core.c:9454:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
   void __init sched_init(void)
   ^
   static 
   10 warnings and 5 errors generated.
--
   In file included from kernel/sched/fair.c:38:
   In file included from include/linux/sched/isolation.h:6:
>> include/linux/tick.h:313:22: error: use of undeclared identifier 'tick_cpu_sched'
                   ts = this_cpu_ptr(&tick_cpu_sched);
                                      ^
>> include/linux/tick.h:313:22: error: use of undeclared identifier 'tick_cpu_sched'
>> include/linux/tick.h:313:22: error: use of undeclared identifier 'tick_cpu_sched'
>> include/linux/tick.h:313:6: error: assigning to 'struct tick_sched *' from incompatible type 'void'
                   ts = this_cpu_ptr(&tick_cpu_sched);
                      ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>> include/linux/tick.h:315:9: error: incomplete definition of type 'struct tick_sched'
                   if (ts->tick_stopped)
                       ~~^
   include/linux/tick.h:310:9: note: forward declaration of 'struct tick_sched'
           struct tick_sched *ts;
                  ^
   kernel/sched/fair.c:11137:6: warning: no previous prototype for function 'task_vruntime_update' [-Wmissing-prototypes]
   void task_vruntime_update(struct rq *rq, struct task_struct *p, bool in_fi)
        ^
   kernel/sched/fair.c:11137:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
   void task_vruntime_update(struct rq *rq, struct task_struct *p, bool in_fi)
   ^
   static 
   1 warning and 5 errors generated.
--
   In file included from kernel/sched/build_policy.c:33:
   In file included from kernel/sched/sched.h:62:
>> include/linux/tick.h:313:22: error: use of undeclared identifier 'tick_cpu_sched'
                   ts = this_cpu_ptr(&tick_cpu_sched);
                                      ^
>> include/linux/tick.h:313:22: error: use of undeclared identifier 'tick_cpu_sched'
>> include/linux/tick.h:313:22: error: use of undeclared identifier 'tick_cpu_sched'
>> include/linux/tick.h:313:6: error: assigning to 'struct tick_sched *' from incompatible type 'void'
                   ts = this_cpu_ptr(&tick_cpu_sched);
                      ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>> include/linux/tick.h:315:9: error: incomplete definition of type 'struct tick_sched'
                   if (ts->tick_stopped)
                       ~~^
   include/linux/tick.h:310:9: note: forward declaration of 'struct tick_sched'
           struct tick_sched *ts;
                  ^
   In file included from kernel/sched/build_policy.c:43:
   kernel/sched/rt.c:666:6: warning: no previous prototype for function 'sched_rt_bandwidth_account' [-Wmissing-prototypes]
   bool sched_rt_bandwidth_account(struct rt_rq *rt_rq)
        ^
   kernel/sched/rt.c:666:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
   bool sched_rt_bandwidth_account(struct rt_rq *rt_rq)
   ^
   static 
   1 warning and 5 errors generated.
--
   In file included from kernel/irq/manage.c:21:
   In file included from include/linux/sched/isolation.h:6:
>> include/linux/tick.h:313:22: error: use of undeclared identifier 'tick_cpu_sched'
                   ts = this_cpu_ptr(&tick_cpu_sched);
                                      ^
>> include/linux/tick.h:313:22: error: use of undeclared identifier 'tick_cpu_sched'
>> include/linux/tick.h:313:22: error: use of undeclared identifier 'tick_cpu_sched'
>> include/linux/tick.h:313:6: error: assigning to 'struct tick_sched *' from incompatible type 'void'
                   ts = this_cpu_ptr(&tick_cpu_sched);
                      ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>> include/linux/tick.h:315:9: error: incomplete definition of type 'struct tick_sched'
                   if (ts->tick_stopped)
                       ~~^
   include/linux/tick.h:310:9: note: forward declaration of 'struct tick_sched'
           struct tick_sched *ts;
                  ^
>> include/linux/tick.h:316:4: error: call to undeclared function 'quiet_vmstat'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]
                           quiet_vmstat();
                           ^
   In file included from kernel/irq/manage.c:25:
   In file included from kernel/irq/internals.h:486:
   In file included from include/linux/debugfs.h:15:
   In file included from include/linux/fs.h:33:
   In file included from include/linux/percpu-rwsem.h:7:
   In file included from include/linux/rcuwait.h:6:
   In file included from include/linux/sched/signal.h:6:
   include/linux/signal.h:97:11: warning: array index 3 is past the end of the array (which contains 1 element) [-Warray-bounds]
                   return (set->sig[3] | set->sig[2] |
                           ^        ~
   arch/x86/include/asm/signal.h:24:2: note: array 'sig' declared here
           unsigned long sig[_NSIG_WORDS];
           ^
   In file included from kernel/irq/manage.c:25:
   In file included from kernel/irq/internals.h:486:
   In file included from include/linux/debugfs.h:15:
   In file included from include/linux/fs.h:33:
   In file included from include/linux/percpu-rwsem.h:7:
   In file included from include/linux/rcuwait.h:6:
   In file included from include/linux/sched/signal.h:6:
   include/linux/signal.h:97:25: warning: array index 2 is past the end of the array (which contains 1 element) [-Warray-bounds]
                   return (set->sig[3] | set->sig[2] |
                                         ^        ~
   arch/x86/include/asm/signal.h:24:2: note: array 'sig' declared here
           unsigned long sig[_NSIG_WORDS];
           ^
   In file included from kernel/irq/manage.c:25:
   In file included from kernel/irq/internals.h:486:
   In file included from include/linux/debugfs.h:15:
   In file included from include/linux/fs.h:33:
   In file included from include/linux/percpu-rwsem.h:7:
   In file included from include/linux/rcuwait.h:6:
   In file included from include/linux/sched/signal.h:6:
   include/linux/signal.h:98:4: warning: array index 1 is past the end of the array (which contains 1 element) [-Warray-bounds]
                           set->sig[1] | set->sig[0]) == 0;
                           ^        ~
   arch/x86/include/asm/signal.h:24:2: note: array 'sig' declared here
           unsigned long sig[_NSIG_WORDS];
           ^
   In file included from kernel/irq/manage.c:25:
   In file included from kernel/irq/internals.h:486:
   In file included from include/linux/debugfs.h:15:
   In file included from include/linux/fs.h:33:
   In file included from include/linux/percpu-rwsem.h:7:
   In file included from include/linux/rcuwait.h:6:
   In file included from include/linux/sched/signal.h:6:
   include/linux/signal.h:100:11: warning: array index 1 is past the end of the array (which contains 1 element) [-Warray-bounds]
                   return (set->sig[1] | set->sig[0]) == 0;
                           ^        ~
   arch/x86/include/asm/signal.h:24:2: note: array 'sig' declared here
           unsigned long sig[_NSIG_WORDS];
           ^
   In file included from kernel/irq/manage.c:25:
   In file included from kernel/irq/internals.h:486:
   In file included from include/linux/debugfs.h:15:
   In file included from include/linux/fs.h:33:
   In file included from include/linux/percpu-rwsem.h:7:
   In file included from include/linux/rcuwait.h:6:
   In file included from include/linux/sched/signal.h:6:
   include/linux/signal.h:113:11: warning: array index 3 is past the end of the array (which contains 1 element) [-Warray-bounds]
                   return  (set1->sig[3] == set2->sig[3]) &&
                            ^         ~
   arch/x86/include/asm/signal.h:24:2: note: array 'sig' declared here
           unsigned long sig[_NSIG_WORDS];
           ^
   In file included from kernel/irq/manage.c:25:
   In file included from kernel/irq/internals.h:486:
   In file included from include/linux/debugfs.h:15:
   In file included from include/linux/fs.h:33:
   In file included from include/linux/percpu-rwsem.h:7:
   In file included from include/linux/rcuwait.h:6:
   In file included from include/linux/sched/signal.h:6:
   include/linux/signal.h:113:27: warning: array index 3 is past the end of the array (which contains 1 element) [-Warray-bounds]
                   return  (set1->sig[3] == set2->sig[3]) &&
                                            ^         ~
   arch/x86/include/asm/signal.h:24:2: note: array 'sig' declared here
           unsigned long sig[_NSIG_WORDS];
           ^
   In file included from kernel/irq/manage.c:25:
   In file included from kernel/irq/internals.h:486:
   In file included from include/linux/debugfs.h:15:
   In file included from include/linux/fs.h:33:
   In file included from include/linux/percpu-rwsem.h:7:
   In file included from include/linux/rcuwait.h:6:
   In file included from include/linux/sched/signal.h:6:
   include/linux/signal.h:114:5: warning: array index 2 is past the end of the array (which contains 1 element) [-Warray-bounds]
                           (set1->sig[2] == set2->sig[2]) &&
                            ^         ~
   arch/x86/include/asm/signal.h:24:2: note: array 'sig' declared here
           unsigned long sig[_NSIG_WORDS];
           ^
   In file included from kernel/irq/manage.c:25:
   In file included from kernel/irq/internals.h:486:
   In file included from include/linux/debugfs.h:15:
   In file included from include/linux/fs.h:33:
   In file included from include/linux/percpu-rwsem.h:7:
   In file included from include/linux/rcuwait.h:6:
   In file included from include/linux/sched/signal.h:6:
..


vim +/tick_cpu_sched +313 include/linux/tick.h

   307	
   308	static inline void tick_nohz_user_enter_prepare(void)
   309	{
   310		struct tick_sched *ts;
   311	
   312		if (tick_nohz_full_cpu(smp_processor_id())) {
 > 313			ts = this_cpu_ptr(&tick_cpu_sched);
   314	
 > 315			if (ts->tick_stopped)
 > 316				quiet_vmstat();
   317			rcu_nocb_flush_deferred_wakeup();
   318		}
   319	}
   320	

-- 
0-DAY CI Kernel Test Service
https://01.org/lkp

           reply	other threads:[~2022-04-22 16:54 UTC|newest]

Thread overview: expand[flat|nested]  mbox.gz  Atom feed
 [parent not found: <20220422112053.3695526-1-atomlin@redhat.com>]

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=202204230047.FesC3TWd-lkp@intel.com \
    --to=lkp@intel.com \
    --cc=atomlin@redhat.com \
    --cc=kbuild-all@lists.01.org \
    --cc=llvm@lists.linux.dev \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox