* [PATCH] sched/core: add rcu_read_lock/unlock() protection
@ 2021-01-26 8:46 qiang.zhang
2021-01-26 9:20 ` Peter Zijlstra
2021-01-26 9:23 ` 回复: " Zhang, Qiang
0 siblings, 2 replies; 3+ messages in thread
From: qiang.zhang @ 2021-01-26 8:46 UTC (permalink / raw)
To: valentin.schneider; +Cc: peterz, linux-kernel
From: Zqiang <qiang.zhang@windriver.com>
Due to for_each_process_thread belongs to RCU read operation,
need to add rcu_read_lock/unlock() protection.
Signed-off-by: Zqiang <qiang.zhang@windriver.com>
---
kernel/sched/core.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/kernel/sched/core.c b/kernel/sched/core.c
index 8c5481077c9c..c3f0103fdf53 100644
--- a/kernel/sched/core.c
+++ b/kernel/sched/core.c
@@ -7738,6 +7738,7 @@ static void dump_rq_tasks(struct rq *rq, const char *loglvl)
lockdep_assert_held(&rq->lock);
printk("%sCPU%d enqueued tasks (%u total):\n", loglvl, cpu, rq->nr_running);
+ rcu_read_lock();
for_each_process_thread(g, p) {
if (task_cpu(p) != cpu)
continue;
@@ -7747,6 +7748,7 @@ static void dump_rq_tasks(struct rq *rq, const char *loglvl)
printk("%s\tpid: %d, name: %s\n", loglvl, p->pid, p->comm);
}
+ rcu_read_unlock();
}
int sched_cpu_dying(unsigned int cpu)
--
2.17.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] sched/core: add rcu_read_lock/unlock() protection
2021-01-26 8:46 [PATCH] sched/core: add rcu_read_lock/unlock() protection qiang.zhang
@ 2021-01-26 9:20 ` Peter Zijlstra
2021-01-26 9:23 ` 回复: " Zhang, Qiang
1 sibling, 0 replies; 3+ messages in thread
From: Peter Zijlstra @ 2021-01-26 9:20 UTC (permalink / raw)
To: qiang.zhang; +Cc: valentin.schneider, linux-kernel
On Tue, Jan 26, 2021 at 04:46:51PM +0800, qiang.zhang@windriver.com wrote:
> From: Zqiang <qiang.zhang@windriver.com>
>
> Due to for_each_process_thread belongs to RCU read operation,
> need to add rcu_read_lock/unlock() protection.
>
> Signed-off-by: Zqiang <qiang.zhang@windriver.com>
> ---
> kernel/sched/core.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/kernel/sched/core.c b/kernel/sched/core.c
> index 8c5481077c9c..c3f0103fdf53 100644
> --- a/kernel/sched/core.c
> +++ b/kernel/sched/core.c
> @@ -7738,6 +7738,7 @@ static void dump_rq_tasks(struct rq *rq, const char *loglvl)
> lockdep_assert_held(&rq->lock);
>
> printk("%sCPU%d enqueued tasks (%u total):\n", loglvl, cpu, rq->nr_running);
> + rcu_read_lock();
> for_each_process_thread(g, p) {
> if (task_cpu(p) != cpu)
> continue;
> @@ -7747,6 +7748,7 @@ static void dump_rq_tasks(struct rq *rq, const char *loglvl)
>
> printk("%s\tpid: %d, name: %s\n", loglvl, p->pid, p->comm);
> }
> + rcu_read_unlock();
We're in stop machine, with IRQs disabled, please explain how this can
make any difference?
^ permalink raw reply [flat|nested] 3+ messages in thread
* 回复: [PATCH] sched/core: add rcu_read_lock/unlock() protection
2021-01-26 8:46 [PATCH] sched/core: add rcu_read_lock/unlock() protection qiang.zhang
2021-01-26 9:20 ` Peter Zijlstra
@ 2021-01-26 9:23 ` Zhang, Qiang
1 sibling, 0 replies; 3+ messages in thread
From: Zhang, Qiang @ 2021-01-26 9:23 UTC (permalink / raw)
To: valentin.schneider; +Cc: peterz, linux-kernel
________________________________________
发件人: Zhang, Qiang <qiang.zhang@windriver.com>
发送时间: 2021年1月26日 16:29
收件人: valentin.schneider@arm.com
抄送: peterz@infradead.org; linux-kernel@vger.kernel.org
主题: [PATCH] sched/core: add rcu_read_lock/unlock() protection
>From: Zqiang <qiang.zhang@windriver.com>
>Due to for_each_process_thread belongs to RCU read operation,
>need to add rcu_read_lock/unlock() protection.
Sorry to disturb you
I find it's already in the RCU critical zone
Please ignore this change
>Signed-off-by: Zqiang <qiang.zhang@windriver.com>
>---
> kernel/sched/core.c | 2 ++
> 1 file changed, 2 insertions(+)
>
>diff --git a/kernel/sched/core.c b/kernel/sched/core.c
>index 8c5481077c9c..c3f0103fdf53 100644
>--- a/kernel/sched/core.c
>+++ b/kernel/sched/core.c
>@@ -7738,6 +7738,7 @@ static void dump_rq_tasks(struct rq *rq, const char *loglvl)
> lockdep_assert_held(&rq->lock);
>
> printk("%sCPU%d enqueued tasks (%u total):\n", loglvl, cpu, rq->nr_running);
>+ rcu_read_lock();
> for_each_process_thread(g, p) {
> if (task_cpu(p) != cpu)
> continue;
>@@ -7747,6 +7748,7 @@ static void dump_rq_tasks(struct rq *rq, const char *loglvl)
>
> printk("%s\tpid: %d, name: %s\n", loglvl, p->pid, p->comm);
> }
>+ rcu_read_unlock();
> }
>
> int sched_cpu_dying(unsigned int cpu)
--
2.17.1
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2021-01-26 18:06 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-01-26 8:46 [PATCH] sched/core: add rcu_read_lock/unlock() protection qiang.zhang
2021-01-26 9:20 ` Peter Zijlstra
2021-01-26 9:23 ` 回复: " Zhang, Qiang
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).