On 20/01/2017 13:20, Marcelo Tosatti wrote: > + ret = kvm_hypercall2(KVM_HC_CLOCK_PAIRING, > + clock_off_gpa, > + KVM_CLOCK_PAIRING_WALLCLOCK); > + if (ret != 0) { > + pr_err("clock offset hypercall ret %lu\n", ret); > + spin_unlock(&kvm_ptp_lock); > + preempt_enable_notrace(); > + return -EOPNOTSUPP; > + } > + Is it worth making this hypercall, or even all of ptp_kvm_get_time_fn, a pv_ops entry? But this looks good already, apart from my different preference on emulate_ptp_sys_offset_mean. Paolo -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html