kvm/x86: Skip SynIC vector check for QEMU side
authorAndrey Smetanin <asmetanin@virtuozzo.com>
Mon, 28 Dec 2015 15:27:23 +0000 (18:27 +0300)
committerPaolo Bonzini <pbonzini@redhat.com>
Fri, 8 Jan 2016 18:04:42 +0000 (19:04 +0100)
QEMU zero-inits Hyper-V SynIC vectors. We should allow that,
and don't reject zero values if set by the host.

Signed-off-by: Andrey Smetanin <asmetanin@virtuozzo.com>
Reviewed-by: Roman Kagan <rkagan@virtuozzo.com>
CC: Gleb Natapov <gleb@kernel.org>
CC: Paolo Bonzini <pbonzini@redhat.com>
CC: Roman Kagan <rkagan@virtuozzo.com>
CC: Denis V. Lunev <den@openvz.org>
CC: qemu-devel@nongnu.org
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
arch/x86/kvm/hyperv.c

index abfb9208ae531999cbd9d82618a43f3ddba43466..ddae13eed075544234789f22c31b1813de39d14c 100644 (file)
@@ -72,12 +72,13 @@ static bool synic_has_vector_auto_eoi(struct kvm_vcpu_hv_synic *synic,
        return false;
 }
 
-static int synic_set_sint(struct kvm_vcpu_hv_synic *synic, int sint, u64 data)
+static int synic_set_sint(struct kvm_vcpu_hv_synic *synic, int sint,
+                         u64 data, bool host)
 {
        int vector;
 
        vector = data & HV_SYNIC_SINT_VECTOR_MASK;
-       if (vector < 16)
+       if (vector < 16 && !host)
                return 1;
        /*
         * Guest may configure multiple SINTs to use the same vector, so
@@ -247,7 +248,7 @@ static int synic_set_msr(struct kvm_vcpu_hv_synic *synic,
                break;
        }
        case HV_X64_MSR_SINT0 ... HV_X64_MSR_SINT15:
-               ret = synic_set_sint(synic, msr - HV_X64_MSR_SINT0, data);
+               ret = synic_set_sint(synic, msr - HV_X64_MSR_SINT0, data, host);
                break;
        default:
                ret = 1;