Commit 451637e4 authored by Yaowei Bai's avatar Yaowei Bai Committed by Linus Torvalds

kernel/watchdog.c: is_hardlockup can be boolean

Make is_hardlockup return bool to improve readability due to this
particular function only using either one or zero as its return value.

No functional change.
Signed-off-by: default avatarYaowei Bai <bywxiaobai@163.com>
Reviewed-by: default avatarAaron Tomlin <atomlin@redhat.com>
Acked-by: default avatarDon Zickus <dzickus@redhat.com>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
parent b6478740
...@@ -263,15 +263,15 @@ void touch_softlockup_watchdog_sync(void) ...@@ -263,15 +263,15 @@ void touch_softlockup_watchdog_sync(void)
#ifdef CONFIG_HARDLOCKUP_DETECTOR #ifdef CONFIG_HARDLOCKUP_DETECTOR
/* watchdog detector functions */ /* watchdog detector functions */
static int is_hardlockup(void) static bool is_hardlockup(void)
{ {
unsigned long hrint = __this_cpu_read(hrtimer_interrupts); unsigned long hrint = __this_cpu_read(hrtimer_interrupts);
if (__this_cpu_read(hrtimer_interrupts_saved) == hrint) if (__this_cpu_read(hrtimer_interrupts_saved) == hrint)
return 1; return true;
__this_cpu_write(hrtimer_interrupts_saved, hrint); __this_cpu_write(hrtimer_interrupts_saved, hrint);
return 0; return false;
} }
#endif #endif
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment