Commit a3b13c23 authored by Ingo Molnar's avatar Ingo Molnar Committed by Linus Torvalds

softlockup: use cpu_clock() instead of sched_clock()

sched_clock() is not a reliable time-source, use cpu_clock() instead.
Signed-off-by: default avatarIngo Molnar <mingo@elte.hu>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
parent 3806204c
...@@ -40,14 +40,16 @@ static struct notifier_block panic_block = { ...@@ -40,14 +40,16 @@ static struct notifier_block panic_block = {
* resolution, and we don't need to waste time with a big divide when * resolution, and we don't need to waste time with a big divide when
* 2^30ns == 1.074s. * 2^30ns == 1.074s.
*/ */
static unsigned long get_timestamp(void) static unsigned long get_timestamp(int this_cpu)
{ {
return sched_clock() >> 30; /* 2^30 ~= 10^9 */ return cpu_clock(this_cpu) >> 30; /* 2^30 ~= 10^9 */
} }
void touch_softlockup_watchdog(void) void touch_softlockup_watchdog(void)
{ {
__raw_get_cpu_var(touch_timestamp) = get_timestamp(); int this_cpu = raw_smp_processor_id();
__raw_get_cpu_var(touch_timestamp) = get_timestamp(this_cpu);
} }
EXPORT_SYMBOL(touch_softlockup_watchdog); EXPORT_SYMBOL(touch_softlockup_watchdog);
...@@ -91,7 +93,7 @@ void softlockup_tick(void) ...@@ -91,7 +93,7 @@ void softlockup_tick(void)
return; return;
} }
now = get_timestamp(); now = get_timestamp(this_cpu);
/* Wake up the high-prio watchdog task every second: */ /* Wake up the high-prio watchdog task every second: */
if (now > (touch_timestamp + 1)) if (now > (touch_timestamp + 1))
......
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