]> git.baikalelectronics.ru Git - kernel.git/commit
KVM: arm/arm64: Handle forward time correction gracefully
authorMarc Zyngier <marc.zyngier@arm.com>
Wed, 6 Apr 2016 08:37:22 +0000 (09:37 +0100)
committerChristoffer Dall <christoffer.dall@linaro.org>
Wed, 6 Apr 2016 11:17:54 +0000 (13:17 +0200)
commit1c743f947d1c11fdd408fdb2bcef0d01c3ee6482
tree28b5238d600d497cd3b6bdbac1f80708bc07a864
parentf6a6d33ab3966d2e69071c844820182541e32262
KVM: arm/arm64: Handle forward time correction gracefully

On a host that runs NTP, corrections can have a direct impact on
the background timer that we program on the behalf of a vcpu.

In particular, NTP performing a forward correction will result in
a timer expiring sooner than expected from a guest point of view.
Not a big deal, we kick the vcpu anyway.

But on wake-up, the vcpu thread is going to perform a check to
find out whether or not it should block. And at that point, the
timer check is going to say "timer has not expired yet, go back
to sleep". This results in the timer event being lost forever.

There are multiple ways to handle this. One would be record that
the timer has expired and let kvm_cpu_has_pending_timer return
true in that case, but that would be fairly invasive. Another is
to check for the "short sleep" condition in the hrtimer callback,
and restart the timer for the remaining time when the condition
is detected.

This patch implements the latter, with a bit of refactoring in
order to avoid too much code duplication.

Cc: <stable@vger.kernel.org>
Reported-by: Alexander Graf <agraf@suse.de>
Reviewed-by: Alexander Graf <agraf@suse.de>
Signed-off-by: Marc Zyngier <marc.zyngier@arm.com>
Signed-off-by: Christoffer Dall <christoffer.dall@linaro.org>
virt/kvm/arm/arch_timer.c