Re: [PATCH v3 tip/core/rcu 1/9] rcu: Add call_rcu_tasks()

From: Oleg Nesterov
Date: Sat Aug 02 2014 - 10:59:12 EST


On 07/31, Paul E. McKenney wrote:
>
> + rcu_read_lock();
> + for_each_process_thread(g, t) {
> + if (t != current && ACCESS_ONCE(t->on_rq) &&
> + !is_idle_task(t)) {

I didn't notice this check before, but it is not needed. for_each_process()
can see the idle threads, there are not on process/thread lists.

But this doesn't really matter, the main problem is that I still think that
for_each_process_thread() can't really work anyway.

Oleg.

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/