diff options
| author | John Stultz <johnstul@us.ibm.com> | 2012-07-17 13:33:52 -0400 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2012-07-19 11:58:37 -0400 |
| commit | 62b787f886e2d96cc7c5428aeee05dbe32a9531b (patch) | |
| tree | efde1d722f2151d394f693da0790afd16cf6dc1a /include/linux | |
| parent | c7e2580578671c4d19a1a83e6fdb2482cc136283 (diff) | |
hrtimer: Provide clock_was_set_delayed()
This is a backport of f55a6faa384304c89cfef162768e88374d3312cb
clock_was_set() cannot be called from hard interrupt context because
it calls on_each_cpu().
For fixing the widely reported leap seconds issue it is necessary to
call it from hard interrupt context, i.e. the timer tick code, which
does the timekeeping updates.
Provide a new function which denotes it in the hrtimer cpu base
structure of the cpu on which it is called and raise the hrtimer
softirq. We then execute the clock_was_set() notificiation from
softirq context in run_hrtimer_softirq(). The hrtimer softirq is
rarely used, so polling the flag there is not a performance issue.
[ tglx: Made it depend on CONFIG_HIGH_RES_TIMERS. We really should get
rid of all this ifdeffery ASAP ]
Signed-off-by: John Stultz <johnstul@us.ibm.com>
Reported-by: Jan Engelhardt <jengelh@inai.de>
Reviewed-by: Ingo Molnar <mingo@kernel.org>
Acked-by: Peter Zijlstra <a.p.zijlstra@chello.nl>
Acked-by: Prarit Bhargava <prarit@redhat.com>
Link: http://lkml.kernel.org/r/1341960205-56738-2-git-send-email-johnstul@us.ibm.com
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Cc: Prarit Bhargava <prarit@redhat.com>
Cc: Thomas Gleixner <tglx@linutronix.de>
Signed-off-by: John Stultz <johnstul@us.ibm.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'include/linux')
| -rw-r--r-- | include/linux/hrtimer.h | 9 |
1 files changed, 8 insertions, 1 deletions
diff --git a/include/linux/hrtimer.h b/include/linux/hrtimer.h index fd0dc30c9f1..c9ec9400ee5 100644 --- a/include/linux/hrtimer.h +++ b/include/linux/hrtimer.h | |||
| @@ -165,6 +165,7 @@ enum hrtimer_base_type { | |||
| 165 | * @lock: lock protecting the base and associated clock bases | 165 | * @lock: lock protecting the base and associated clock bases |
| 166 | * and timers | 166 | * and timers |
| 167 | * @active_bases: Bitfield to mark bases with active timers | 167 | * @active_bases: Bitfield to mark bases with active timers |
| 168 | * @clock_was_set: Indicates that clock was set from irq context. | ||
| 168 | * @expires_next: absolute time of the next event which was scheduled | 169 | * @expires_next: absolute time of the next event which was scheduled |
| 169 | * via clock_set_next_event() | 170 | * via clock_set_next_event() |
| 170 | * @hres_active: State of high resolution mode | 171 | * @hres_active: State of high resolution mode |
| @@ -177,7 +178,8 @@ enum hrtimer_base_type { | |||
| 177 | */ | 178 | */ |
| 178 | struct hrtimer_cpu_base { | 179 | struct hrtimer_cpu_base { |
| 179 | raw_spinlock_t lock; | 180 | raw_spinlock_t lock; |
| 180 | unsigned long active_bases; | 181 | unsigned int active_bases; |
| 182 | unsigned int clock_was_set; | ||
| 181 | #ifdef CONFIG_HIGH_RES_TIMERS | 183 | #ifdef CONFIG_HIGH_RES_TIMERS |
| 182 | ktime_t expires_next; | 184 | ktime_t expires_next; |
| 183 | int hres_active; | 185 | int hres_active; |
| @@ -286,6 +288,8 @@ extern void hrtimer_peek_ahead_timers(void); | |||
| 286 | # define MONOTONIC_RES_NSEC HIGH_RES_NSEC | 288 | # define MONOTONIC_RES_NSEC HIGH_RES_NSEC |
| 287 | # define KTIME_MONOTONIC_RES KTIME_HIGH_RES | 289 | # define KTIME_MONOTONIC_RES KTIME_HIGH_RES |
| 288 | 290 | ||
| 291 | extern void clock_was_set_delayed(void); | ||
| 292 | |||
| 289 | #else | 293 | #else |
| 290 | 294 | ||
| 291 | # define MONOTONIC_RES_NSEC LOW_RES_NSEC | 295 | # define MONOTONIC_RES_NSEC LOW_RES_NSEC |
| @@ -306,6 +310,9 @@ static inline int hrtimer_is_hres_active(struct hrtimer *timer) | |||
| 306 | { | 310 | { |
| 307 | return 0; | 311 | return 0; |
| 308 | } | 312 | } |
| 313 | |||
| 314 | static inline void clock_was_set_delayed(void) { } | ||
| 315 | |||
| 309 | #endif | 316 | #endif |
| 310 | 317 | ||
| 311 | extern void clock_was_set(void); | 318 | extern void clock_was_set(void); |
