Commit 9cb8b0f2 authored by Breno Leitao's avatar Breno Leitao Committed by Tejun Heo
Browse files

workqueue: replace BUG_ON with panic in panic_on_wq_watchdog 



Replace BUG_ON() with panic() in panic_on_wq_watchdog(). This is not
a bug condition but a deliberate forced panic requested by the user
via module parameters to crash the system for debugging purposes.

Using panic() instead of BUG_ON() makes this intent clearer and provides
more informative output about which threshold was exceeded and the actual
values, making it easier to diagnose the stall condition from crash dumps.

Signed-off-by: default avatarBreno Leitao <leitao@debian.org>
Signed-off-by: default avatarTejun Heo <tj@kernel.org>
parent f84c9dd3

kernel/workqueue.c

+6 −2
Original line number Diff line number Diff line
@@ -7637,10 +7637,14 @@ static void panic_on_wq_watchdog(unsigned int stall_time_sec) 


	if (wq_panic_on_stall) {

		wq_stall++;

		BUG_ON(wq_stall >= wq_panic_on_stall);

		if (wq_stall >= wq_panic_on_stall)

			panic("workqueue: %u stall(s) exceeded threshold %u\n",

			      wq_stall, wq_panic_on_stall);

	}



	BUG_ON(wq_panic_on_stall_time && stall_time_sec >= wq_panic_on_stall_time);

	if (wq_panic_on_stall_time && stall_time_sec >= wq_panic_on_stall_time)

		panic("workqueue: stall lasted %us, exceeding threshold %us\n",

		      stall_time_sec, wq_panic_on_stall_time);

}



static void wq_watchdog_reset_touched(void)