From: Ville Syrjälä <[email protected]> This was just extra debug to hunt down a regression, which got fixed.
This reverts commit d77cc11a9ab0558923d304683ad2307ec0cd5681. Signed-off-by: Ville Syrjälä <[email protected]> --- kernel/freezer.c | 12 ++---------- 1 file changed, 2 insertions(+), 10 deletions(-) diff --git a/kernel/freezer.c b/kernel/freezer.c index e211184c26c7..4fad0e6fca64 100644 --- a/kernel/freezer.c +++ b/kernel/freezer.c @@ -125,16 +125,8 @@ static int __set_task_frozen(struct task_struct *p, void *arg) /* * It's dangerous to freeze with locks held; there be dragons there. */ - if (!(state & __TASK_FREEZABLE_UNSAFE)) { - static bool warned = false; - - if (!warned && debug_locks && p->lockdep_depth) { - debug_show_held_locks(p); - WARN(1, "%s/%d holding locks while freezing\n", - p->comm, task_pid_nr(p)); - warned = true; - } - } + if (!(state & __TASK_FREEZABLE_UNSAFE)) + WARN_ON_ONCE(debug_locks && p->lockdep_depth); #endif WRITE_ONCE(p->__state, TASK_FROZEN); -- 2.41.0
