diff options
| author | Davidlohr Bueso <dave@stgolabs.net> | 2020-09-07 13:32:06 -0700 |
|---|---|---|
| committer | Daniel Thompson <daniel.thompson@linaro.org> | 2020-09-08 14:36:46 +0100 |
| commit | ece4ceaf2eba1c0da9d6b62bc59a43be6b456548 (patch) | |
| tree | 57d08d26bb6fabef10f64c2119d4dccb5e69d25f /kernel/debug/kdb/kdb_bt.c | |
| parent | b18b099e04f450cdc77bec72acefcde7042bd1f3 (diff) | |
| download | linux-stable-ece4ceaf2eba1c0da9d6b62bc59a43be6b456548.tar.gz linux-stable-ece4ceaf2eba1c0da9d6b62bc59a43be6b456548.tar.bz2 linux-stable-ece4ceaf2eba1c0da9d6b62bc59a43be6b456548.zip | |
kdb: Use newer api for tasklist scanning
This kills using the do_each_thread/while_each_thread combo to
iterate all threads and uses for_each_process_thread() instead,
maintaining semantics. while_each_thread() is ultimately racy
and deprecated; although in this particular case there is no
concurrency so it doesn't matter. Still lets trivially get rid
of two more users.
Acked-by: Oleg Nesterov <oleg@redhat.com>
Signed-off-by: Davidlohr Bueso <dbueso@suse.de>
Link: https://lore.kernel.org/r/20200907203206.21293-1-dave@stgolabs.net
Signed-off-by: Daniel Thompson <daniel.thompson@linaro.org>
Diffstat (limited to 'kernel/debug/kdb/kdb_bt.c')
| -rw-r--r-- | kernel/debug/kdb/kdb_bt.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/kernel/debug/kdb/kdb_bt.c b/kernel/debug/kdb/kdb_bt.c index 18e03aba2cfc..1f9f0e47aeda 100644 --- a/kernel/debug/kdb/kdb_bt.c +++ b/kernel/debug/kdb/kdb_bt.c @@ -149,14 +149,14 @@ kdb_bt(int argc, const char **argv) return 0; } /* Now the inactive tasks */ - kdb_do_each_thread(g, p) { + for_each_process_thread(g, p) { if (KDB_FLAG(CMD_INTERRUPT)) return 0; if (task_curr(p)) continue; if (kdb_bt1(p, mask, btaprompt)) return 0; - } kdb_while_each_thread(g, p); + } } else if (strcmp(argv[0], "btp") == 0) { struct task_struct *p; unsigned long pid; |