Commit b19f4db4 authored by Thomas Gleixner's avatar Thomas Gleixner Committed by Ben Hutchings

time: Move common updates to a function

This is a backport of cc06268c

[John Stultz: While not a bugfix itself, it allows following fixes
 to backport in a more straightforward manner.]

CC: Thomas Gleixner <tglx@linutronix.de>
CC: Eric Dumazet <eric.dumazet@gmail.com>
CC: Richard Cochran <richardcochran@gmail.com>
Signed-off-by: default avatarThomas Gleixner <tglx@linutronix.de>
Cc: Prarit Bhargava <prarit@redhat.com>
Cc: Thomas Gleixner <tglx@linutronix.de>
Cc: Linux Kernel <linux-kernel@vger.kernel.org>
Signed-off-by: default avatarJohn Stultz <john.stultz@linaro.org>
Signed-off-by: default avatarBen Hutchings <ben@decadent.org.uk>
parent 09e66e8d
...@@ -166,6 +166,19 @@ static struct timespec total_sleep_time; ...@@ -166,6 +166,19 @@ static struct timespec total_sleep_time;
*/ */
static struct timespec raw_time; static struct timespec raw_time;
/* must hold write on xtime_lock */
static void timekeeping_update(bool clearntp)
{
if (clearntp) {
timekeeper.ntp_error = 0;
ntp_clear();
}
update_vsyscall(&xtime, &wall_to_monotonic,
timekeeper.clock, timekeeper.mult);
}
/* flag for if timekeeping is suspended */ /* flag for if timekeeping is suspended */
int __read_mostly timekeeping_suspended; int __read_mostly timekeeping_suspended;
...@@ -366,11 +379,7 @@ int do_settimeofday(const struct timespec *tv) ...@@ -366,11 +379,7 @@ int do_settimeofday(const struct timespec *tv)
xtime = *tv; xtime = *tv;
timekeeper.ntp_error = 0; timekeeping_update(true);
ntp_clear();
update_vsyscall(&xtime, &wall_to_monotonic, timekeeper.clock,
timekeeper.mult);
write_sequnlock_irqrestore(&xtime_lock, flags); write_sequnlock_irqrestore(&xtime_lock, flags);
...@@ -403,11 +412,7 @@ int timekeeping_inject_offset(struct timespec *ts) ...@@ -403,11 +412,7 @@ int timekeeping_inject_offset(struct timespec *ts)
xtime = timespec_add(xtime, *ts); xtime = timespec_add(xtime, *ts);
wall_to_monotonic = timespec_sub(wall_to_monotonic, *ts); wall_to_monotonic = timespec_sub(wall_to_monotonic, *ts);
timekeeper.ntp_error = 0; timekeeping_update(true);
ntp_clear();
update_vsyscall(&xtime, &wall_to_monotonic, timekeeper.clock,
timekeeper.mult);
write_sequnlock_irqrestore(&xtime_lock, flags); write_sequnlock_irqrestore(&xtime_lock, flags);
...@@ -636,10 +641,7 @@ void timekeeping_inject_sleeptime(struct timespec *delta) ...@@ -636,10 +641,7 @@ void timekeeping_inject_sleeptime(struct timespec *delta)
__timekeeping_inject_sleeptime(delta); __timekeeping_inject_sleeptime(delta);
timekeeper.ntp_error = 0; timekeeping_update(true);
ntp_clear();
update_vsyscall(&xtime, &wall_to_monotonic, timekeeper.clock,
timekeeper.mult);
write_sequnlock_irqrestore(&xtime_lock, flags); write_sequnlock_irqrestore(&xtime_lock, flags);
...@@ -1052,9 +1054,7 @@ static void update_wall_time(void) ...@@ -1052,9 +1054,7 @@ static void update_wall_time(void)
wall_to_monotonic.tv_sec -= leap; wall_to_monotonic.tv_sec -= leap;
} }
/* check to see if there is a new clocksource to use */ timekeeping_update(false);
update_vsyscall(&xtime, &wall_to_monotonic, timekeeper.clock,
timekeeper.mult);
} }
/** /**
......
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