| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <831EE4E5E37DCC428EB295A351E662495241C16E@shsmsx102.ccr.corp.intel.com>
Date: Thu, 5 Mar 2020 02:29:26 +0000
From: "Li, Philip" <philip.li@...el.com>
To: "paulmck@...nel.org" <paulmck@...nel.org>, lkp <lkp@...el.com>
CC: "kbuild-all@...ts.01.org" <kbuild-all@...ts.01.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: RE: [kbuild-all] Re: [rcu:dev.2020.02.29a 43/43]
kernel/rcu/tasks.h:369:2: error: implicit declaration of function
'call_rcu_tasks_generic'
> Subject: [kbuild-all] Re: [rcu:dev.2020.02.29a 43/43] kernel/rcu/tasks.h:369:2:
> error: implicit declaration of function 'call_rcu_tasks_generic'
>
> On Thu, Mar 05, 2020 at 12:40:11AM +0800, kbuild test robot wrote:
> > tree: https://git.kernel.org/pub/scm/linux/kernel/git/paulmck/linux-rcu.git
> dev.2020.02.29a
> > head: 61f7110d6b78f4c84ea5d5480185740840889af7
> > commit: 61f7110d6b78f4c84ea5d5480185740840889af7 [43/43] rcu-tasks: Add
> an RCU-tasks rude variant
> > config: mips-rm200_defconfig (attached as .config)
> > compiler: mipsel-linux-gcc (GCC) 5.5.0
> > reproduce:
> > wget https://raw.githubusercontent.com/intel/lkp-
> tests/master/sbin/make.cross -O ~/bin/make.cross
> > chmod +x ~/bin/make.cross
> > git checkout 61f7110d6b78f4c84ea5d5480185740840889af7
> > # save the attached .config to linux build tree
> > GCC_VERSION=5.5.0 make.cross ARCH=mips
> >
> > If you fix the issue, kindly add following tag
> > Reported-by: kbuild test robot <lkp@...el.com>
>
> This is for an obsolete branch. I believe that the replacement commit
> 8dd788e3ab55 ("rcu-tasks: Add an RCU-tasks rude variant") fixes this.
got it, Paul.
Kindly ignore later reports against this branch, that some bisections are still in progress,
sorry for the noise.
Thanks
>
> Thanx, Paul
>
> > All error/warnings (new ones prefixed by >>):
> >
> > union { typeof(x) __val; char __c[1]; } __u; \
> > ^
> > kernel/rcu/tasks.h:212:7: note: in expansion of macro 'READ_ONCE'
> > if (!READ_ONCE(t->rcu_tasks_holdout) ||
> > ^
> > kernel/rcu/tasks.h:212:18: error: 'struct task_struct' has no member named
> 'rcu_tasks_holdout'
> > if (!READ_ONCE(t->rcu_tasks_holdout) ||
> > ^
> > include/linux/compiler.h:287:22: note: in definition of macro '__READ_ONCE'
> > __read_once_size(&(x), __u.__c, sizeof(x)); \
> > ^
> > kernel/rcu/tasks.h:212:7: note: in expansion of macro 'READ_ONCE'
> > if (!READ_ONCE(t->rcu_tasks_holdout) ||
> > ^
> > kernel/rcu/tasks.h:212:18: error: 'struct task_struct' has no member named
> 'rcu_tasks_holdout'
> > if (!READ_ONCE(t->rcu_tasks_holdout) ||
> > ^
> > include/linux/compiler.h:287:42: note: in definition of macro '__READ_ONCE'
> > __read_once_size(&(x), __u.__c, sizeof(x)); \
> > ^
> > kernel/rcu/tasks.h:212:7: note: in expansion of macro 'READ_ONCE'
> > if (!READ_ONCE(t->rcu_tasks_holdout) ||
> > ^
> > kernel/rcu/tasks.h:212:18: error: 'struct task_struct' has no member named
> 'rcu_tasks_holdout'
> > if (!READ_ONCE(t->rcu_tasks_holdout) ||
> > ^
> > include/linux/compiler.h:289:30: note: in definition of macro '__READ_ONCE'
> > __read_once_size_nocheck(&(x), __u.__c, sizeof(x)); \
> > ^
> > kernel/rcu/tasks.h:212:7: note: in expansion of macro 'READ_ONCE'
> > if (!READ_ONCE(t->rcu_tasks_holdout) ||
> > ^
> > kernel/rcu/tasks.h:212:18: error: 'struct task_struct' has no member named
> 'rcu_tasks_holdout'
> > if (!READ_ONCE(t->rcu_tasks_holdout) ||
> > ^
> > include/linux/compiler.h:289:50: note: in definition of macro '__READ_ONCE'
> > __read_once_size_nocheck(&(x), __u.__c, sizeof(x)); \
> > ^
> > kernel/rcu/tasks.h:212:7: note: in expansion of macro 'READ_ONCE'
> > if (!READ_ONCE(t->rcu_tasks_holdout) ||
> > ^
> > In file included from kernel/rcu/update.c:562:0:
> > kernel/rcu/tasks.h:213:7: error: 'struct task_struct' has no member named
> 'rcu_tasks_nvcsw'
> > t->rcu_tasks_nvcsw != READ_ONCE(t->nvcsw) ||
> > ^
> > kernel/rcu/tasks.h:216:28: error: 'struct task_struct' has no member named
> 'rcu_tasks_idle_cpu'
> > !is_idle_task(t) && t->rcu_tasks_idle_cpu >= 0)) {
> > ^
> > In file included from include/linux/kernel.h:11:0,
> > from kernel/rcu/update.c:21:
> > kernel/rcu/tasks.h:217:15: error: 'struct task_struct' has no member named
> 'rcu_tasks_holdout'
> > WRITE_ONCE(t->rcu_tasks_holdout, false);
> > ^
> > include/linux/compiler.h:310:17: note: in definition of macro 'WRITE_ONCE'
> > union { typeof(x) __val; char __c[1]; } __u = \
> > ^
> > kernel/rcu/tasks.h:217:15: error: 'struct task_struct' has no member named
> 'rcu_tasks_holdout'
> > WRITE_ONCE(t->rcu_tasks_holdout, false);
> > ^
> > include/linux/compiler.h:311:30: note: in definition of macro 'WRITE_ONCE'
> > { .__val = (__force typeof(x)) (val) }; \
> > ^
> > kernel/rcu/tasks.h:217:15: error: 'struct task_struct' has no member named
> 'rcu_tasks_holdout'
> > WRITE_ONCE(t->rcu_tasks_holdout, false);
> > ^
> > include/linux/compiler.h:312:22: note: in definition of macro 'WRITE_ONCE'
> > __write_once_size(&(x), __u.__c, sizeof(x)); \
> > ^
> > kernel/rcu/tasks.h:217:15: error: 'struct task_struct' has no member named
> 'rcu_tasks_holdout'
> > WRITE_ONCE(t->rcu_tasks_holdout, false);
> > ^
> > include/linux/compiler.h:312:42: note: in definition of macro 'WRITE_ONCE'
> > __write_once_size(&(x), __u.__c, sizeof(x)); \
> > ^
> > In file included from kernel/rcu/update.c:562:0:
> > kernel/rcu/tasks.h:218:19: error: 'struct task_struct' has no member named
> 'rcu_tasks_holdout_list'
> > list_del_init(&t->rcu_tasks_holdout_list);
> > ^
> > In file included from include/linux/kernel.h:15:0,
> > from kernel/rcu/update.c:21:
> > kernel/rcu/tasks.h:233:5: error: 'struct task_struct' has no member named
> 'rcu_tasks_nvcsw'
> > t->rcu_tasks_nvcsw, t->nvcsw, t->rcu_tasks_holdout,
> > ^
> > include/linux/printk.h:300:35: note: in definition of macro 'pr_alert'
> > printk(KERN_ALERT pr_fmt(fmt), ##__VA_ARGS__)
> > ^
> > kernel/rcu/tasks.h:233:35: error: 'struct task_struct' has no member named
> 'rcu_tasks_holdout'
> > t->rcu_tasks_nvcsw, t->nvcsw, t->rcu_tasks_holdout,
> > ^
> > include/linux/printk.h:300:35: note: in definition of macro 'pr_alert'
> > printk(KERN_ALERT pr_fmt(fmt), ##__VA_ARGS__)
> > ^
> > kernel/rcu/tasks.h:234:5: error: 'struct task_struct' has no member named
> 'rcu_tasks_idle_cpu'
> > t->rcu_tasks_idle_cpu, cpu);
> > ^
> > include/linux/printk.h:300:35: note: in definition of macro 'pr_alert'
> > printk(KERN_ALERT pr_fmt(fmt), ##__VA_ARGS__)
> > ^
> > In file included from kernel/rcu/update.c:562:0:
> > kernel/rcu/tasks.h: At top level:
> > >> kernel/rcu/tasks.h:239:38: warning: 'struct rcu_tasks' declared inside
> parameter list
> > static void rcu_tasks_wait_gp(struct rcu_tasks *rtp)
> > ^
> > >> kernel/rcu/tasks.h:239:38: warning: its scope is only this definition or
> declaration, which is probably not what you want
> > kernel/rcu/tasks.h: In function 'rcu_tasks_wait_gp':
> > kernel/rcu/tasks.h:271:5: error: 'struct task_struct' has no member named
> 'rcu_tasks_nvcsw'
> > t->rcu_tasks_nvcsw = READ_ONCE(t->nvcsw);
> > ^
> > In file included from include/linux/kernel.h:11:0,
> > from kernel/rcu/update.c:21:
> > kernel/rcu/tasks.h:272:16: error: 'struct task_struct' has no member named
> 'rcu_tasks_holdout'
> > WRITE_ONCE(t->rcu_tasks_holdout, true);
> > ^
> > include/linux/compiler.h:310:17: note: in definition of macro 'WRITE_ONCE'
> > union { typeof(x) __val; char __c[1]; } __u = \
> > ^
> > kernel/rcu/tasks.h:272:16: error: 'struct task_struct' has no member named
> 'rcu_tasks_holdout'
> > WRITE_ONCE(t->rcu_tasks_holdout, true);
> > ^
> > include/linux/compiler.h:311:30: note: in definition of macro 'WRITE_ONCE'
> > { .__val = (__force typeof(x)) (val) }; \
> > ^
> > kernel/rcu/tasks.h:272:16: error: 'struct task_struct' has no member named
> 'rcu_tasks_holdout'
> > WRITE_ONCE(t->rcu_tasks_holdout, true);
> > ^
> > include/linux/compiler.h:312:22: note: in definition of macro 'WRITE_ONCE'
> > __write_once_size(&(x), __u.__c, sizeof(x)); \
> > ^
> > kernel/rcu/tasks.h:272:16: error: 'struct task_struct' has no member named
> 'rcu_tasks_holdout'
> > WRITE_ONCE(t->rcu_tasks_holdout, true);
> > ^
> > include/linux/compiler.h:312:42: note: in definition of macro 'WRITE_ONCE'
> > __write_once_size(&(x), __u.__c, sizeof(x)); \
> > ^
> > In file included from kernel/rcu/update.c:562:0:
> > kernel/rcu/tasks.h:273:15: error: 'struct task_struct' has no member named
> 'rcu_tasks_holdout_list'
> > list_add(&t->rcu_tasks_holdout_list,
> > ^
> > kernel/rcu/tasks.h:286:20: error: 'tasks_rcu_exit_srcu' undeclared (first use in
> this function)
> > synchronize_srcu(&tasks_rcu_exit_srcu);
> > ^
> > kernel/rcu/tasks.h:286:20: note: each undeclared identifier is reported only
> once for each function it appears in
> > In file included from include/linux/kernel.h:11:0,
> > from kernel/rcu/update.c:21:
> > kernel/rcu/tasks.h:313:20: error: 'rcu_task_stall_timeout' undeclared (first use
> in this function)
> > rtst = READ_ONCE(rcu_task_stall_timeout);
> > ^
> > include/linux/compiler.h:285:17: note: in definition of macro '__READ_ONCE'
> > union { typeof(x) __val; char __c[1]; } __u; \
> > ^
> > kernel/rcu/tasks.h:313:10: note: in expansion of macro 'READ_ONCE'
> > rtst = READ_ONCE(rcu_task_stall_timeout);
> > ^
> > include/linux/kernel.h:987:51: error: 'struct task_struct' has no member named
> 'rcu_tasks_holdout_list'
> > BUILD_BUG_ON_MSG(!__same_type(*(ptr), ((type *)0)->member) && \
> > ^
> > include/linux/compiler.h:374:9: note: in definition of macro
> '__compiletime_assert'
> > if (!(condition)) \
> > ^
> > include/linux/compiler.h:394:2: note: in expansion of macro
> '_compiletime_assert'
> > _compiletime_assert(condition, msg, __compiletime_assert_, __LINE__)
> > ^
> > include/linux/build_bug.h:39:37: note: in expansion of macro
> 'compiletime_assert'
> > #define BUILD_BUG_ON_MSG(cond, msg) compiletime_assert(!(cond), msg)
> > ^
> > include/linux/kernel.h:987:2: note: in expansion of macro
> 'BUILD_BUG_ON_MSG'
> > BUILD_BUG_ON_MSG(!__same_type(*(ptr), ((type *)0)->member) && \
> > ^
> > include/linux/kernel.h:987:20: note: in expansion of macro '__same_type'
> > BUILD_BUG_ON_MSG(!__same_type(*(ptr), ((type *)0)->member) && \
> > ^
> > include/linux/list.h:493:2: note: in expansion of macro 'container_of'
> > container_of(ptr, type, member)
> > ^
> > include/linux/list.h:504:2: note: in expansion of macro 'list_entry'
> > list_entry((ptr)->next, type, member)
> > ^
> > include/linux/list.h:688:13: note: in expansion of macro 'list_first_entry'
> > for (pos = list_first_entry(head, typeof(*pos), member), \
> > ^
> > kernel/rcu/tasks.h:319:3: note: in expansion of macro
> 'list_for_each_entry_safe'
> > list_for_each_entry_safe(t, t1, &rcu_tasks_holdouts,
> > ^
> > In file included from <command-line>:0:0:
> > include/linux/compiler_types.h:129:35: error: 'struct task_struct' has no
> member named 'rcu_tasks_holdout_list'
> > #define __compiler_offsetof(a, b) __builtin_offsetof(a, b)
> > ^
> > include/linux/stddef.h:17:32: note: in expansion of macro '__compiler_offsetof'
> > #define offsetof(TYPE, MEMBER) __compiler_offsetof(TYPE, MEMBER)
> > ^
> > include/linux/kernel.h:990:21: note: in expansion of macro 'offsetof'
> > ((type *)(__mptr - offsetof(type, member))); })
> > ^
> > include/linux/list.h:493:2: note: in expansion of macro 'container_of'
> > container_of(ptr, type, member)
> > ^
> > include/linux/list.h:504:2: note: in expansion of macro 'list_entry'
> > list_entry((ptr)->next, type, member)
> > ^
> > include/linux/list.h:688:13: note: in expansion of macro 'list_first_entry'
> > for (pos = list_first_entry(head, typeof(*pos), member), \
> > ^
> > kernel/rcu/tasks.h:319:3: note: in expansion of macro
> 'list_for_each_entry_safe'
> > list_for_each_entry_safe(t, t1, &rcu_tasks_holdouts,
> > --
> > ^
> > include/linux/list.h:537:2: note: in expansion of macro 'list_entry'
> > list_entry((pos)->member.next, typeof(*(pos)), member)
> > ^
> > include/linux/list.h:691:20: note: in expansion of macro 'list_next_entry'
> > pos = n, n = list_next_entry(n, member))
> > ^
> > kernel/rcu/tasks.h:319:3: note: in expansion of macro
> 'list_for_each_entry_safe'
> > list_for_each_entry_safe(t, t1, &rcu_tasks_holdouts,
> > ^
> > include/linux/kernel.h:987:51: error: 'struct task_struct' has no member named
> 'rcu_tasks_holdout_list'
> > BUILD_BUG_ON_MSG(!__same_type(*(ptr), ((type *)0)->member) && \
> > ^
> > include/linux/compiler.h:374:9: note: in definition of macro
> '__compiletime_assert'
> > if (!(condition)) \
> > ^
> > include/linux/compiler.h:394:2: note: in expansion of macro
> '_compiletime_assert'
> > _compiletime_assert(condition, msg, __compiletime_assert_, __LINE__)
> > ^
> > include/linux/build_bug.h:39:37: note: in expansion of macro
> 'compiletime_assert'
> > #define BUILD_BUG_ON_MSG(cond, msg) compiletime_assert(!(cond), msg)
> > ^
> > include/linux/kernel.h:987:2: note: in expansion of macro
> 'BUILD_BUG_ON_MSG'
> > BUILD_BUG_ON_MSG(!__same_type(*(ptr), ((type *)0)->member) && \
> > ^
> > include/linux/kernel.h:987:20: note: in expansion of macro '__same_type'
> > BUILD_BUG_ON_MSG(!__same_type(*(ptr), ((type *)0)->member) && \
> > ^
> > include/linux/list.h:493:2: note: in expansion of macro 'container_of'
> > container_of(ptr, type, member)
> > ^
> > include/linux/list.h:537:2: note: in expansion of macro 'list_entry'
> > list_entry((pos)->member.next, typeof(*(pos)), member)
> > ^
> > include/linux/list.h:691:20: note: in expansion of macro 'list_next_entry'
> > pos = n, n = list_next_entry(n, member))
> > ^
> > kernel/rcu/tasks.h:319:3: note: in expansion of macro
> 'list_for_each_entry_safe'
> > list_for_each_entry_safe(t, t1, &rcu_tasks_holdouts,
> > ^
> > include/linux/list.h:537:18: error: 'struct task_struct' has no member named
> 'rcu_tasks_holdout_list'
> > list_entry((pos)->member.next, typeof(*(pos)), member)
> > ^
> > include/linux/compiler.h:374:9: note: in definition of macro
> '__compiletime_assert'
> > if (!(condition)) \
> > ^
> > include/linux/compiler.h:394:2: note: in expansion of macro
> '_compiletime_assert'
> > _compiletime_assert(condition, msg, __compiletime_assert_, __LINE__)
> > ^
> > include/linux/build_bug.h:39:37: note: in expansion of macro
> 'compiletime_assert'
> > #define BUILD_BUG_ON_MSG(cond, msg) compiletime_assert(!(cond), msg)
> > ^
> > include/linux/kernel.h:987:2: note: in expansion of macro
> 'BUILD_BUG_ON_MSG'
> > BUILD_BUG_ON_MSG(!__same_type(*(ptr), ((type *)0)->member) && \
> > ^
> > include/linux/kernel.h:988:6: note: in expansion of macro '__same_type'
> > !__same_type(*(ptr), void), \
> > ^
> > include/linux/list.h:493:2: note: in expansion of macro 'container_of'
> > container_of(ptr, type, member)
> > ^
> > include/linux/list.h:537:2: note: in expansion of macro 'list_entry'
> > list_entry((pos)->member.next, typeof(*(pos)), member)
> > ^
> > include/linux/list.h:691:20: note: in expansion of macro 'list_next_entry'
> > pos = n, n = list_next_entry(n, member))
> > ^
> > kernel/rcu/tasks.h:319:3: note: in expansion of macro
> 'list_for_each_entry_safe'
> > list_for_each_entry_safe(t, t1, &rcu_tasks_holdouts,
> > ^
> > In file included from <command-line>:0:0:
> > include/linux/compiler_types.h:129:35: error: 'struct task_struct' has no
> member named 'rcu_tasks_holdout_list'
> > #define __compiler_offsetof(a, b) __builtin_offsetof(a, b)
> > ^
> > include/linux/stddef.h:17:32: note: in expansion of macro '__compiler_offsetof'
> > #define offsetof(TYPE, MEMBER) __compiler_offsetof(TYPE, MEMBER)
> > ^
> > include/linux/kernel.h:990:21: note: in expansion of macro 'offsetof'
> > ((type *)(__mptr - offsetof(type, member))); })
> > ^
> > include/linux/list.h:493:2: note: in expansion of macro 'container_of'
> > container_of(ptr, type, member)
> > ^
> > include/linux/list.h:537:2: note: in expansion of macro 'list_entry'
> > list_entry((pos)->member.next, typeof(*(pos)), member)
> > ^
> > include/linux/list.h:691:20: note: in expansion of macro 'list_next_entry'
> > pos = n, n = list_next_entry(n, member))
> > ^
> > kernel/rcu/tasks.h:319:3: note: in expansion of macro
> 'list_for_each_entry_safe'
> > list_for_each_entry_safe(t, t1, &rcu_tasks_holdouts,
> > ^
> > In file included from kernel/rcu/update.c:562:0:
> > kernel/rcu/tasks.h: At top level:
> > kernel/rcu/tasks.h:347:1: warning: data definition has no type or storage class
> > DEFINE_RCU_TASKS(rcu_tasks, rcu_tasks_wait_gp, call_rcu_tasks);
> > ^
> > kernel/rcu/tasks.h:347:1: error: type defaults to 'int' in declaration of
> 'DEFINE_RCU_TASKS' [-Werror=implicit-int]
> > kernel/rcu/tasks.h:347:1: warning: parameter names (without types) in
> function declaration
> > kernel/rcu/tasks.h: In function 'call_rcu':
> > >> kernel/rcu/tasks.h:369:2: error: implicit declaration of function
> 'call_rcu_tasks_generic' [-Werror=implicit-function-declaration]
> > call_rcu_tasks_generic(rhp, func, &rcu_tasks);
> > ^
> > >> kernel/rcu/tasks.h:369:37: error: 'rcu_tasks' undeclared (first use in this
> function)
> > call_rcu_tasks_generic(rhp, func, &rcu_tasks);
> > ^
> > kernel/rcu/tasks.h: In function 'synchronize_rcu':
> > >> kernel/rcu/tasks.h:393:2: error: implicit declaration of function
> 'synchronize_rcu_tasks_generic' [-Werror=implicit-function-declaration]
> > synchronize_rcu_tasks_generic(&rcu_tasks);
> > ^
> > kernel/rcu/tasks.h:393:33: error: 'rcu_tasks' undeclared (first use in this
> function)
> > synchronize_rcu_tasks_generic(&rcu_tasks);
> > ^
> > kernel/rcu/tasks.h: In function 'rcu_spawn_tasks_kthread':
> > >> kernel/rcu/tasks.h:412:2: error: implicit declaration of function
> 'rcu_spawn_tasks_kthread_generic' [-Werror=implicit-function-declaration]
> > rcu_spawn_tasks_kthread_generic(&rcu_tasks);
> > ^
> > kernel/rcu/tasks.h:412:35: error: 'rcu_tasks' undeclared (first use in this
> function)
> > rcu_spawn_tasks_kthread_generic(&rcu_tasks);
> > ^
> > kernel/rcu/tasks.h: At top level:
> > kernel/rcu/tasks.h:432:43: warning: 'struct rcu_tasks' declared inside
> parameter list
> > static void rcu_tasks_rude_wait_gp(struct rcu_tasks *rtp)
> > ^
> > kernel/rcu/tasks.h:439:1: warning: data definition has no type or storage class
> > DEFINE_RCU_TASKS(rcu_tasks_rude, rcu_tasks_rude_wait_gp,
> call_rcu_tasks_rude);
> > ^
> > kernel/rcu/tasks.h:439:1: error: type defaults to 'int' in declaration of
> 'DEFINE_RCU_TASKS' [-Werror=implicit-int]
> > kernel/rcu/tasks.h:439:1: warning: parameter names (without types) in
> function declaration
> > kernel/rcu/tasks.h: In function 'call_rcu_tasks_rude':
> > >> kernel/rcu/tasks.h:461:37: error: 'rcu_tasks_rude' undeclared (first use in this
> function)
> > call_rcu_tasks_generic(rhp, func, &rcu_tasks_rude);
> > ^
> > kernel/rcu/tasks.h: In function 'synchronize_rcu_tasks_rude':
> > kernel/rcu/tasks.h:485:33: error: 'rcu_tasks_rude' undeclared (first use in this
> function)
> > synchronize_rcu_tasks_generic(&rcu_tasks_rude);
> > ^
> > kernel/rcu/tasks.h: In function 'rcu_spawn_tasks_rude_kthread':
> > kernel/rcu/tasks.h:504:35: error: 'rcu_tasks_rude' undeclared (first use in this
> function)
> > rcu_spawn_tasks_kthread_generic(&rcu_tasks_rude);
> > ^
> > kernel/rcu/update.c: At top level:
> > kernel/rcu/tasks.h:239:13: warning: 'rcu_tasks_wait_gp' defined but not used
> [-Wunused-function]
> > static void rcu_tasks_wait_gp(struct rcu_tasks *rtp)
> > ^
> > cc1: some warnings being treated as errors
> >
> > vim +/call_rcu_tasks_generic +369 kernel/rcu/tasks.h
> >
> > 237
> > 238 /* Wait for one RCU-tasks grace period. */
> > > 239 static void rcu_tasks_wait_gp(struct rcu_tasks *rtp)
> > 240 {
> > 241 struct task_struct *g, *t;
> > 242 unsigned long lastreport;
> > 243 LIST_HEAD(rcu_tasks_holdouts);
> > 244 int fract;
> > 245
> > 246 /*
> > 247 * Wait for all pre-existing t->on_rq and t->nvcsw transitions
> > 248 * to complete. Invoking synchronize_rcu() suffices because all
> > 249 * these transitions occur with interrupts disabled. Without this
> > 250 * synchronize_rcu(), a read-side critical section that started
> > 251 * before the grace period might be incorrectly seen as having
> > 252 * started after the grace period.
> > 253 *
> > 254 * This synchronize_rcu() also dispenses with the need for a
> > 255 * memory barrier on the first store to t->rcu_tasks_holdout,
> > 256 * as it forces the store to happen after the beginning of the
> > 257 * grace period.
> > 258 */
> > 259 synchronize_rcu();
> > 260
> > 261 /*
> > 262 * There were callbacks, so we need to wait for an RCU-tasks
> > 263 * grace period. Start off by scanning the task list for tasks
> > 264 * that are not already voluntarily blocked. Mark these tasks
> > 265 * and make a list of them in rcu_tasks_holdouts.
> > 266 */
> > 267 rcu_read_lock();
> > 268 for_each_process_thread(g, t) {
> > 269 if (t != current && READ_ONCE(t->on_rq)
> && !is_idle_task(t)) {
> > 270 get_task_struct(t);
> > 271 t->rcu_tasks_nvcsw = READ_ONCE(t->nvcsw);
> > 272 WRITE_ONCE(t->rcu_tasks_holdout, true);
> > 273 list_add(&t->rcu_tasks_holdout_list,
> > 274 &rcu_tasks_holdouts);
> > 275 }
> > 276 }
> > 277 rcu_read_unlock();
> > 278
> > 279 /*
> > 280 * Wait for tasks that are in the process of exiting. This
> > 281 * does only part of the job, ensuring that all tasks that were
> > 282 * previously exiting reach the point where they have disabled
> > 283 * preemption, allowing the later synchronize_rcu() to finish
> > 284 * the job.
> > 285 */
> > 286 synchronize_srcu(&tasks_rcu_exit_srcu);
> > 287
> > 288 /*
> > 289 * Each pass through the following loop scans the list of holdout
> > 290 * tasks, removing any that are no longer holdouts. When the list
> > 291 * is empty, we are done.
> > 292 */
> > 293 lastreport = jiffies;
> > 294
> > 295 /* Start off with HZ/10 wait and slowly back off to 1 HZ wait. */
> > 296 fract = 10;
> > 297
> > 298 for (;;) {
> > 299 bool firstreport;
> > 300 bool needreport;
> > 301 int rtst;
> > 302 struct task_struct *t1;
> > 303
> > 304 if (list_empty(&rcu_tasks_holdouts))
> > 305 break;
> > 306
> > 307 /* Slowly back off waiting for holdouts */
> > 308 schedule_timeout_interruptible(HZ/fract);
> > 309
> > 310 if (fract > 1)
> > 311 fract--;
> > 312
> > 313 rtst = READ_ONCE(rcu_task_stall_timeout);
> > 314 needreport = rtst > 0 && time_after(jiffies, lastreport +
> rtst);
> > 315 if (needreport)
> > 316 lastreport = jiffies;
> > 317 firstreport = true;
> > 318 WARN_ON(signal_pending(current));
> > 319 list_for_each_entry_safe(t, t1, &rcu_tasks_holdouts,
> > 320 rcu_tasks_holdout_list) {
> > 321 check_holdout_task(t, needreport, &firstreport);
> > 322 cond_resched();
> > 323 }
> > 324 }
> > 325
> > 326 /*
> > 327 * Because ->on_rq and ->nvcsw are not guaranteed to have a full
> > 328 * memory barriers prior to them in the schedule() path, memory
> > 329 * reordering on other CPUs could cause their RCU-tasks read-
> side
> > 330 * critical sections to extend past the end of the grace period.
> > 331 * However, because these ->nvcsw updates are carried out with
> > 332 * interrupts disabled, we can use synchronize_rcu() to force the
> > 333 * needed ordering on all such CPUs.
> > 334 *
> > 335 * This synchronize_rcu() also confines all ->rcu_tasks_holdout
> > 336 * accesses to be within the grace period, avoiding the need for
> > 337 * memory barriers for ->rcu_tasks_holdout accesses.
> > 338 *
> > 339 * In addition, this synchronize_rcu() waits for exiting tasks
> > 340 * to complete their final preempt_disable() region of execution,
> > 341 * cleaning up after the synchronize_srcu() above.
> > 342 */
> > 343 synchronize_rcu();
> > 344 }
> > 345
> > 346 void call_rcu_tasks(struct rcu_head *rhp, rcu_callback_t func);
> > 347 DEFINE_RCU_TASKS(rcu_tasks, rcu_tasks_wait_gp, call_rcu_tasks);
> > 348
> > 349 /**
> > 350 * call_rcu_tasks() - Queue an RCU for invocation task-based grace
> period
> > 351 * @rhp: structure to be used for queueing the RCU updates.
> > 352 * @func: actual callback function to be invoked after the grace period
> > 353 *
> > 354 * The callback function will be invoked some time after a full grace
> > 355 * period elapses, in other words after all currently executing RCU
> > 356 * read-side critical sections have completed. call_rcu_tasks() assumes
> > 357 * that the read-side critical sections end at a voluntary context
> > 358 * switch (not a preemption!), cond_resched_rcu_qs(), entry into idle,
> > 359 * or transition to usermode execution. As such, there are no read-side
> > 360 * primitives analogous to rcu_read_lock() and rcu_read_unlock() because
> > 361 * this primitive is intended to determine that all tasks have passed
> > 362 * through a safe state, not so much for data-strcuture synchronization.
> > 363 *
> > 364 * See the description of call_rcu() for more detailed information on
> > 365 * memory ordering guarantees.
> > 366 */
> > 367 void call_rcu_tasks(struct rcu_head *rhp, rcu_callback_t func)
> > 368 {
> > > 369 call_rcu_tasks_generic(rhp, func, &rcu_tasks);
> > 370 }
> > 371 EXPORT_SYMBOL_GPL(call_rcu_tasks);
> > 372
> > 373 /**
> > 374 * synchronize_rcu_tasks - wait until an rcu-tasks grace period has
> elapsed.
> > 375 *
> > 376 * Control will return to the caller some time after a full rcu-tasks
> > 377 * grace period has elapsed, in other words after all currently
> > 378 * executing rcu-tasks read-side critical sections have elapsed. These
> > 379 * read-side critical sections are delimited by calls to schedule(),
> > 380 * cond_resched_tasks_rcu_qs(), idle execution, userspace execution,
> calls
> > 381 * to synchronize_rcu_tasks(), and (in theory, anyway) cond_resched().
> > 382 *
> > 383 * This is a very specialized primitive, intended only for a few uses in
> > 384 * tracing and other situations requiring manipulation of function
> > 385 * preambles and profiling hooks. The synchronize_rcu_tasks() function
> > 386 * is not (yet) intended for heavy use from multiple CPUs.
> > 387 *
> > 388 * See the description of synchronize_rcu() for more detailed information
> > 389 * on memory ordering guarantees.
> > 390 */
> > 391 void synchronize_rcu_tasks(void)
> > 392 {
> > > 393 synchronize_rcu_tasks_generic(&rcu_tasks);
> > 394 }
> > 395 EXPORT_SYMBOL_GPL(synchronize_rcu_tasks);
> > 396
> > 397 /**
> > 398 * rcu_barrier_tasks - Wait for in-flight call_rcu_tasks() callbacks.
> > 399 *
> > 400 * Although the current implementation is guaranteed to wait, it is not
> > 401 * obligated to, for example, if there are no pending callbacks.
> > 402 */
> > 403 void rcu_barrier_tasks(void)
> > 404 {
> > 405 /* There is only one callback queue, so this is easy. ;-) */
> > 406 synchronize_rcu_tasks();
> > 407 }
> > 408 EXPORT_SYMBOL_GPL(rcu_barrier_tasks);
> > 409
> > 410 static int __init rcu_spawn_tasks_kthread(void)
> > 411 {
> > > 412 rcu_spawn_tasks_kthread_generic(&rcu_tasks);
> > 413 return 0;
> > 414 }
> > 415 core_initcall(rcu_spawn_tasks_kthread);
> > 416
> > 417 ////////////////////////////////////////////////////////////////////////
> > 418 //
> > 419 // "Rude" variant of Tasks RCU, inspired by Steve Rostedt's trick of
> > 420 // passing an empty function to schedule_on_each_cpu(). This approach
> > 421 // provides an asynchronous call_rcu_rude() API and batching of
> concurrent
> > 422 // calls to the synchronous synchronize_rcu_rude() API. This sends IPIs
> > 423 // far and wide and induces otherwise unnecessary context switches on all
> > 424 // online CPUs, whether online or not.
> > 425
> > 426 // Empty function to allow workqueues to force a context switch.
> > 427 static void rcu_tasks_be_rude(struct work_struct *work)
> > 428 {
> > 429 }
> > 430
> > 431 // Wait for one rude RCU-tasks grace period.
> > 432 static void rcu_tasks_rude_wait_gp(struct rcu_tasks *rtp)
> > 433 {
> > 434 schedule_on_each_cpu(rcu_tasks_be_rude);
> > 435 }
> > 436 EXPORT_SYMBOL_GPL(rcu_tasks_rude_wait_gp);
> > 437
> > 438 void call_rcu_tasks_rude(struct rcu_head *rhp, rcu_callback_t func);
> > 439 DEFINE_RCU_TASKS(rcu_tasks_rude, rcu_tasks_rude_wait_gp,
> call_rcu_tasks_rude);
> > 440
> > 441 /**
> > 442 * call_rcu_tasks_rude() - Queue a callback rude task-based grace period
> > 443 * @rhp: structure to be used for queueing the RCU updates.
> > 444 * @func: actual callback function to be invoked after the grace period
> > 445 *
> > 446 * The callback function will be invoked some time after a full grace
> > 447 * period elapses, in other words after all currently executing RCU
> > 448 * read-side critical sections have completed. call_rcu_tasks_rude()
> > 449 * assumes that the read-side critical sections end at context switch,
> > 450 * cond_resched_rcu_qs(), or transition to usermode execution. As such,
> > 451 * there are no read-side primitives analogous to rcu_read_lock() and
> > 452 * rcu_read_unlock() because this primitive is intended to determine
> > 453 * that all tasks have passed through a safe state, not so much for
> > 454 * data-strcuture synchronization.
> > 455 *
> > 456 * See the description of call_rcu() for more detailed information on
> > 457 * memory ordering guarantees.
> > 458 */
> > 459 void call_rcu_tasks_rude(struct rcu_head *rhp, rcu_callback_t func)
> > 460 {
> > > 461 call_rcu_tasks_generic(rhp, func, &rcu_tasks_rude);
> > 462 }
> > 463 EXPORT_SYMBOL_GPL(call_rcu_tasks_rude);
> > 464
> >
> > ---
> > 0-DAY CI Kernel Test Service, Intel Corporation
> > https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
>
> _______________________________________________
> kbuild-all mailing list -- kbuild-all@...ts.01.org
> To unsubscribe send an email to kbuild-all-leave@...ts.01.org
Powered by blists - more mailing lists