diff options
| author | Nicholas Piggin <npiggin@gmail.com> | 2024-06-25 14:42:44 +0300 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2024-09-12 12:10:27 +0300 |
| commit | 0b93c4f4caa8f9ac39aeee3d00380fc76462e31a (patch) | |
| tree | fde58d65d874bd37ba54482dc15b8b77489111ed | |
| parent | 9178eb8ebcd887ab75e54ac40d538e54bb9c7788 (diff) | |
| download | linux-0b93c4f4caa8f9ac39aeee3d00380fc76462e31a.tar.xz | |
workqueue: wq_watchdog_touch is always called with valid CPU
[ Upstream commit 18e24deb1cc92f2068ce7434a94233741fbd7771 ]
Warn in the case it is called with cpu == -1. This does not appear
to happen anywhere.
Signed-off-by: Nicholas Piggin <npiggin@gmail.com>
Reviewed-by: Paul E. McKenney <paulmck@kernel.org>
Signed-off-by: Tejun Heo <tj@kernel.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>
| -rw-r--r-- | kernel/workqueue.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/kernel/workqueue.c b/kernel/workqueue.c index f3b6ac232e21..4da8a5e702f8 100644 --- a/kernel/workqueue.c +++ b/kernel/workqueue.c @@ -5893,6 +5893,8 @@ notrace void wq_watchdog_touch(int cpu) { if (cpu >= 0) per_cpu(wq_watchdog_touched_cpu, cpu) = jiffies; + else + WARN_ONCE(1, "%s should be called with valid CPU", __func__); wq_watchdog_touched = jiffies; } |