On 10/04/2010 06:50 AM, Glauber Costa wrote:
Zach,

         vcpu->hv_clock.tsc_timestamp = tsc_timestamp;
         vcpu->hv_clock.system_time = kernel_ns + v->kvm->arch.kvmclock_offset;
         vcpu->last_kernel_ns = kernel_ns;<===== (1)
         vcpu->last_guest_tsc = tsc_timestamp;
         vcpu->hv_clock.flags = 0;

If I understand your intention corretly, you should add kvmclock_offset to (1).
Am I misinterpreting something?

No, last_kernel_ns is the last observed value of kernel_ns, not system time. The kvmclock_offset may be modified since observation (unlikely, yet possible), and should not be considered in this computation.
--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to