All of lore.kernel.org
 help / color / mirror / Atom feed
* [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.