From: Avi Kivity <avi@redhat.com>
Date: Tue, 4 May 2010 12:00:37 +0000 (+0300)
Subject: KVM: Fix wallclock version writing race
X-Git-Tag: firefly_0821_release~10186^2~1443
X-Git-Url: http://demsky.eecs.uci.edu/git/?a=commitdiff_plain;h=0890bb8d955da47bd51bc823f5080ec0c2664b3f;p=firefly-linux-kernel-4.4.55.git

KVM: Fix wallclock version writing race

Wallclock writing uses an unprotected global variable to hold the version;
this can cause one guest to interfere with another if both write their
wallclock at the same time.

Acked-by: Glauber Costa <glommer@redhat.com>
Signed-off-by: Avi Kivity <avi@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
(cherry picked from commit 9ed3c444ab8987c7b219173a2f7807e3f71e234e)
---

diff --git a/arch/x86/kvm/x86.c b/arch/x86/kvm/x86.c
index 3f83a30d1802..802c7bd2cef2 100644
--- a/arch/x86/kvm/x86.c
+++ b/arch/x86/kvm/x86.c
@@ -554,14 +554,22 @@ static int do_set_msr(struct kvm_vcpu *vcpu, unsigned index, u64 *data)
 
 static void kvm_write_wall_clock(struct kvm *kvm, gpa_t wall_clock)
 {
-	static int version;
+	int version;
+	int r;
 	struct pvclock_wall_clock wc;
 	struct timespec boot;
 
 	if (!wall_clock)
 		return;
 
-	version++;
+	r = kvm_read_guest(kvm, wall_clock, &version, sizeof(version));
+	if (r)
+		return;
+
+	if (version & 1)
+		++version;  /* first time write, random junk */
+
+	++version;
 
 	kvm_write_guest(kvm, wall_clock, &version, sizeof(version));