timekeeping: Update timekeeper before updating vsyscall and pvclock
authorThomas Gleixner <tglx@linutronix.de>
Sat, 6 Sep 2014 10:24:49 +0000 (12:24 +0200)
committerThomas Gleixner <tglx@linutronix.de>
Sat, 6 Sep 2014 10:58:18 +0000 (12:58 +0200)
commit9bf2419fa7bffa16ce58a4d5c20399eff8c970c9
tree23415c1be550979e70c5ecabff7a76aff47ee378
parent849151dd5481bc8acb1d287a299b5d6a4ca9f1c3
timekeeping: Update timekeeper before updating vsyscall and pvclock

The update_walltime() code works on the shadow timekeeper to make the
seqcount protected region as short as possible. But that update to the
shadow timekeeper does not update all timekeeper fields because it's
sufficient to do that once before it becomes life. One of these fields
is tkr.base_mono. That stays stale in the shadow timekeeper unless an
operation happens which copies the real timekeeper to the shadow.

The update function is called after the update calls to vsyscall and
pvclock. While not correct, it did not cause any problems because none
of the invoked update functions used base_mono.

commit cbcf2dd3b3d4 (x86: kvm: Make kvm_get_time_and_clockread()
nanoseconds based) changed that in the kvm pvclock update function, so
the stale mono_base value got used and caused kvm-clock to malfunction.

Put the update where it belongs and fix the issue.

Reported-by: Chris J Arges <chris.j.arges@canonical.com>
Reported-by: Paolo Bonzini <pbonzini@redhat.com>
Cc: Gleb Natapov <gleb@kernel.org>
Cc: John Stultz <john.stultz@linaro.org>
Link: http://lkml.kernel.org/r/alpine.DEB.2.10.1409050000570.3333@nanos
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
kernel/time/timekeeping.c