* [rcu:dev.2020.02.29a 43/43] kernel/rcu/tasks.h:369:2: error: implicit declaration of function 'call_rcu_tasks_generic'
@ 2020-03-04 16:40 ` kbuild test robot
0 siblings, 0 replies; 8+ messages in thread
From: kbuild test robot @ 2020-03-04 16:40 UTC (permalink / raw)
To: Paul E. McKenney; +Cc: kbuild-all, linux-kernel
[-- Attachment #1: Type: text/plain, Size: 30043 bytes --]
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@intel.com>
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
[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 18633 bytes --]
^ permalink raw reply [flat|nested] 8+ messages in thread
* [rcu:dev.2020.02.29a 43/43] kernel/rcu/tasks.h:369:2: error: implicit declaration of function 'call_rcu_tasks_generic'
@ 2020-03-04 16:40 ` kbuild test robot
0 siblings, 0 replies; 8+ messages in thread
From: kbuild test robot @ 2020-03-04 16:40 UTC (permalink / raw)
To: kbuild-all
[-- Attachment #1: Type: text/plain, Size: 30648 bytes --]
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@intel.com>
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(a)lists.01.org
[-- Attachment #2: config.gz --]
[-- Type: application/gzip, Size: 18633 bytes --]
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [rcu:dev.2020.02.29a 43/43] kernel/rcu/tasks.h:369:2: error: implicit declaration of function 'call_rcu_tasks_generic'
2020-03-04 16:40 ` kbuild test robot
@ 2020-03-04 17:24 ` Paul E. McKenney
-1 siblings, 0 replies; 8+ messages in thread
From: Paul E. McKenney @ 2020-03-04 17:24 UTC (permalink / raw)
To: kbuild test robot; +Cc: kbuild-all, linux-kernel
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@intel.com>
This is for an obsolete branch. I believe that the replacement commit
8dd788e3ab55 ("rcu-tasks: Add an RCU-tasks rude variant") fixes this.
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
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [rcu:dev.2020.02.29a 43/43] kernel/rcu/tasks.h:369:2: error: implicit declaration of function 'call_rcu_tasks_generic'
@ 2020-03-04 17:24 ` Paul E. McKenney
0 siblings, 0 replies; 8+ messages in thread
From: Paul E. McKenney @ 2020-03-04 17:24 UTC (permalink / raw)
To: kbuild-all
[-- Attachment #1: Type: text/plain, Size: 32089 bytes --]
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@intel.com>
This is for an obsolete branch. I believe that the replacement commit
8dd788e3ab55 ("rcu-tasks: Add an RCU-tasks rude variant") fixes this.
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(a)lists.01.org
^ permalink raw reply [flat|nested] 8+ messages in thread
* 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'
2020-03-04 17:24 ` Paul E. McKenney
@ 2020-03-05 2:29 ` Li, Philip
-1 siblings, 0 replies; 8+ messages in thread
From: Li, Philip @ 2020-03-05 2:29 UTC (permalink / raw)
To: paulmck, lkp; +Cc: kbuild-all, linux-kernel
> 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@intel.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@lists.01.org
> To unsubscribe send an email to kbuild-all-leave@lists.01.org
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [rcu:dev.2020.02.29a 43/43] kernel/rcu/tasks.h:369:2: error: implicit declaration of function 'call_rcu_tasks_generic'
@ 2020-03-05 2:29 ` Li, Philip
0 siblings, 0 replies; 8+ messages in thread
From: Li, Philip @ 2020-03-05 2:29 UTC (permalink / raw)
To: kbuild-all
[-- Attachment #1: Type: text/plain, Size: 33979 bytes --]
> 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@intel.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(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] 8+ messages in thread
* 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'
2020-03-05 2:29 ` Li, Philip
@ 2020-03-05 3:27 ` Paul E. McKenney
-1 siblings, 0 replies; 8+ messages in thread
From: Paul E. McKenney @ 2020-03-05 3:27 UTC (permalink / raw)
To: Li, Philip; +Cc: lkp, kbuild-all, linux-kernel
On Thu, Mar 05, 2020 at 02:29:26AM +0000, Li, Philip wrote:
> > 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@intel.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.
Not a problem, just wanted you all to know. Also, just in case you
guys ever have a way of stopping work on now-obsolete branches. ;-)
Thanx, Paul
> 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@lists.01.org
> > To unsubscribe send an email to kbuild-all-leave@lists.01.org
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [rcu:dev.2020.02.29a 43/43] kernel/rcu/tasks.h:369:2: error: implicit declaration of function 'call_rcu_tasks_generic'
@ 2020-03-05 3:27 ` Paul E. McKenney
0 siblings, 0 replies; 8+ messages in thread
From: Paul E. McKenney @ 2020-03-05 3:27 UTC (permalink / raw)
To: kbuild-all
[-- Attachment #1: Type: text/plain, Size: 35597 bytes --]
On Thu, Mar 05, 2020 at 02:29:26AM +0000, Li, Philip wrote:
> > 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@intel.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.
Not a problem, just wanted you all to know. Also, just in case you
guys ever have a way of stopping work on now-obsolete branches. ;-)
Thanx, Paul
> 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(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] 8+ messages in thread
end of thread, other threads:[~2020-03-05 3:27 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-03-04 16:40 [rcu:dev.2020.02.29a 43/43] kernel/rcu/tasks.h:369:2: error: implicit declaration of function 'call_rcu_tasks_generic' kbuild test robot
2020-03-04 16:40 ` kbuild test robot
2020-03-04 17:24 ` Paul E. McKenney
2020-03-04 17:24 ` Paul E. McKenney
2020-03-05 2:29 ` [kbuild-all] " Li, Philip
2020-03-05 2:29 ` Li, Philip
2020-03-05 3:27 ` [kbuild-all] " Paul E. McKenney
2020-03-05 3:27 ` Paul E. McKenney
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.