49 lines
2.0 KiB
Diff
49 lines
2.0 KiB
Diff
From 8d1a7c12a52fa392744d952a28ff00f63fbc9acf Mon Sep 17 00:00:00 2001
|
|
Message-Id: <8d1a7c12a52fa392744d952a28ff00f63fbc9acf.1601675152.git.zanussi@kernel.org>
|
|
In-Reply-To: <5b5a156f9808b1acf1205606e03da117214549ea.1601675151.git.zanussi@kernel.org>
|
|
References: <5b5a156f9808b1acf1205606e03da117214549ea.1601675151.git.zanussi@kernel.org>
|
|
From: Steven Rostedt <rostedt@goodmis.org>
|
|
Date: Mon, 18 Mar 2013 15:12:49 -0400
|
|
Subject: [PATCH 113/333] sched/workqueue: Only wake up idle workers if not
|
|
blocked on sleeping spin lock
|
|
Origin: https://www.kernel.org/pub/linux/kernel/projects/rt/4.19/older/patches-4.19.148-rt64.tar.xz
|
|
|
|
In -rt, most spin_locks() turn into mutexes. One of these spin_lock
|
|
conversions is performed on the workqueue gcwq->lock. When the idle
|
|
worker is worken, the first thing it will do is grab that same lock and
|
|
it too will block, possibly jumping into the same code, but because
|
|
nr_running would already be decremented it prevents an infinite loop.
|
|
|
|
But this is still a waste of CPU cycles, and it doesn't follow the method
|
|
of mainline, as new workers should only be woken when a worker thread is
|
|
truly going to sleep, and not just blocked on a spin_lock().
|
|
|
|
Check the saved_state too before waking up new workers.
|
|
|
|
|
|
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
|
|
Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
|
|
---
|
|
kernel/sched/core.c | 4 +++-
|
|
1 file changed, 3 insertions(+), 1 deletion(-)
|
|
|
|
diff --git a/kernel/sched/core.c b/kernel/sched/core.c
|
|
index 2aa1d2a9ee5e..be048e7066d1 100644
|
|
--- a/kernel/sched/core.c
|
|
+++ b/kernel/sched/core.c
|
|
@@ -3541,8 +3541,10 @@ static void __sched notrace __schedule(bool preempt)
|
|
* If a worker went to sleep, notify and ask workqueue
|
|
* whether it wants to wake up a task to maintain
|
|
* concurrency.
|
|
+ * Only call wake up if prev isn't blocked on a sleeping
|
|
+ * spin lock.
|
|
*/
|
|
- if (prev->flags & PF_WQ_WORKER) {
|
|
+ if (prev->flags & PF_WQ_WORKER && !prev->saved_state) {
|
|
struct task_struct *to_wakeup;
|
|
|
|
to_wakeup = wq_worker_sleeping(prev);
|
|
--
|
|
2.17.1
|
|
|