diff options
| author | Linus Torvalds <[email protected]> | 2008-03-24 11:07:15 -0700 |
|---|---|---|
| committer | Linus Torvalds <[email protected]> | 2008-03-24 11:07:15 -0700 |
| commit | 92896bd9fd75b1c993b92874d339a8088bb75560 (patch) | |
| tree | 2bd8bc5a8c8ee10a4b1fd62804281d2b486067f0 | |
| parent | d2532dd20a126020de407c1c2476a75b53fce7ac (diff) | |
Don't 'printk()' while holding xtime lock for writing
The printk() can deadlock because it can wake up klogd(), and
task enqueueing will try to read the time in order to set a hrtimer.
Reported-by: Marcin Slusarz <[email protected]>
Debugged-by: Peter Zijlstra <[email protected]>
Cc: Ingo Molnar <[email protected]>
Cc: Thomas Gleixner <[email protected]>
Signed-off-by: Linus Torvalds <[email protected]>
| -rw-r--r-- | kernel/time/timekeeping.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/kernel/time/timekeeping.c b/kernel/time/timekeeping.c index 671af612b768..a3fa587c350c 100644 --- a/kernel/time/timekeeping.c +++ b/kernel/time/timekeeping.c @@ -191,8 +191,12 @@ static void change_clocksource(void) tick_clock_notify(); + /* + * We're holding xtime lock and waking up klogd would deadlock + * us on enqueue. So no printing! printk(KERN_INFO "Time: %s clocksource has been installed.\n", clock->name); + */ } #else static inline void change_clocksource(void) { } |