KVM: x86: fix WARN_ON check of an unsigned less than zero
authorPaolo Bonzini <pbonzini@redhat.com>
Wed, 12 Feb 2020 12:27:10 +0000 (13:27 +0100)
committerPaolo Bonzini <pbonzini@redhat.com>
Wed, 12 Feb 2020 19:09:46 +0000 (20:09 +0100)
commit9446e6fce0ab9dfd44b96f630b4e3a0a0ab879fd
treef6c9f4f50e27e54732ba5767e6d29af1e02924a4
parentff479025349cef3106e165a761281851fd018282
KVM: x86: fix WARN_ON check of an unsigned less than zero

The check cpu->hv_clock.system_time < 0 is redundant since system_time
is a u64 and hence can never be less than zero.  But what was actually
meant is to check that the result is positive, since kernel_ns and
v->kvm->arch.kvmclock_offset are both s64.

Reported-by: Colin King <colin.king@canonical.com>
Suggested-by: Sean Christopherson <sean.j.christopherson@intel.com>
Addresses-Coverity: ("Macro compares unsigned to 0")
Reviewed-by: Miaohe Lin <linmiaohe@huawei.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
arch/x86/kvm/x86.c