diff options
| author | Thomas Gleixner <tglx@linutronix.de> | 2007-09-16 09:36:43 -0400 |
|---|---|---|
| committer | Thomas Gleixner <tglx@chaos.(none)> | 2007-09-16 09:36:43 -0400 |
| commit | 3be9095063885d482b87d3875ea7f28e635882d0 (patch) | |
| tree | c29f9a90a244032515c4f830e1d73d4dd2a3f1ac | |
| parent | d0174640eedc1cd756754f03afe2dbb3d56de74e (diff) | |
timekeeping: access rtc outside of xtime lock
Lockdep complains about the access of rtc in timekeeping_suspend
inside the interrupt disabled region of the write locked xtime lock.
Move the access outside.
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Cc: John Stultz <johnstul@us.ibm.com>
| -rw-r--r-- | kernel/time/timekeeping.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/kernel/time/timekeeping.c b/kernel/time/timekeeping.c index acc417b5a9b7..f682091fa890 100644 --- a/kernel/time/timekeeping.c +++ b/kernel/time/timekeeping.c | |||
| @@ -325,9 +325,10 @@ static int timekeeping_suspend(struct sys_device *dev, pm_message_t state) | |||
| 325 | { | 325 | { |
| 326 | unsigned long flags; | 326 | unsigned long flags; |
| 327 | 327 | ||
| 328 | timekeeping_suspend_time = read_persistent_clock(); | ||
| 329 | |||
| 328 | write_seqlock_irqsave(&xtime_lock, flags); | 330 | write_seqlock_irqsave(&xtime_lock, flags); |
| 329 | timekeeping_suspended = 1; | 331 | timekeeping_suspended = 1; |
| 330 | timekeeping_suspend_time = read_persistent_clock(); | ||
| 331 | write_sequnlock_irqrestore(&xtime_lock, flags); | 332 | write_sequnlock_irqrestore(&xtime_lock, flags); |
| 332 | 333 | ||
| 333 | clockevents_notify(CLOCK_EVT_NOTIFY_SUSPEND, NULL); | 334 | clockevents_notify(CLOCK_EVT_NOTIFY_SUSPEND, NULL); |
