2017-04-06 19:11+0200, Radim Krčmář:
> 2017-04-06 11:08+0300, Denis Plotnikov:
>> diff --git a/arch/x86/kvm/x86.c b/arch/x86/kvm/x86.c
>> @@ -1455,15 +1455,23 @@ void kvm_write_tsc(struct kvm_vcpu *vcpu, struct msr_data *msr)
>> elapsed = ns - kvm->arch.last_tsc_nsec;
>>
>> if (vcpu->arch.virtual_tsc_khz) {
>> - u64 tsc_exp = kvm->arch.last_tsc_write +
>> - nsec_to_cycles(vcpu, elapsed);
>> - u64 tsc_hz = vcpu->arch.virtual_tsc_khz * 1000LL;
>> - /*
>> - * Special case: TSC write with a small delta (1 second) of virtual
>> - * cycle time against real time is interpreted as an attempt to
>> - * synchronize the CPU.
>> - */
>> - synchronizing = data < tsc_exp + tsc_hz && data > tsc_exp - tsc_hz;
>> + if ((data == 0) && msr->host_initiated) {
And while I'm nitpicking ... the code would also be nicer without the
parentheses around data.
