diff options
| author | Johannes Weiner <hannes@saeurebad.de> | 2008-06-27 15:07:21 +0200 |
|---|---|---|
| committer | Ingo Molnar <mingo@elte.hu> | 2008-06-30 15:03:08 +0200 |
| commit | dd7a1e5615b1719c0fdffee1ea5a7820ac8141a6 (patch) | |
| tree | eecc48d905d53ee4e8b71cb2b88c0651c76922a4 /kernel | |
| parent | 8d5be7f4e8515af461cbc8f07687ccc81507d508 (diff) | |
| download | op-kernel-dev-dd7a1e5615b1719c0fdffee1ea5a7820ac8141a6.zip op-kernel-dev-dd7a1e5615b1719c0fdffee1ea5a7820ac8141a6.tar.gz | |
softlockup: fix watchdog task wakeup frequency
Updating the timestamp more often is pointless as we print the warnings
only if we exceed the threshold. And the check for hung tasks relies on
the last timestamp, so it will keep working correctly, too.
Signed-off-by: Johannes Weiner <hannes@saeurebad.de>
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Diffstat (limited to 'kernel')
| -rw-r--r-- | kernel/softlockup.c | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/kernel/softlockup.c b/kernel/softlockup.c index 97977ec..d53ab70 100644 --- a/kernel/softlockup.c +++ b/kernel/softlockup.c @@ -130,8 +130,11 @@ void softlockup_tick(void) now = get_timestamp(this_cpu); - /* Wake up the high-prio watchdog task every second: */ - if (now > (touch_timestamp + 1)) + /* + * Wake up the high-prio watchdog task twice per + * threshold timespan. + */ + if (now > touch_timestamp + softlockup_thresh/2) wake_up_process(per_cpu(watchdog_task, this_cpu)); /* Warn about unreasonable delays: */ |
