Re: [PATCH v3 0/8] pvclock fixes - v3
On 05/06/2010 12:27 AM, Glauber Costa wrote: Hello, Avi, this version fixes the issues you raised in the last review. Hopte it is better now. Sorry about the late review. Looks reasonable, apart from a couple of minor points. A cpuid.txt wil follow as soon as we're set on everything, so I am not including it yet. Pity, for a reviewer it is a lot easier to review a spec, then the code's conformity to the spec, instead of having to reverse engineer the intent from the code. Please include it next time. -- error compiling committee.c: too many arguments to function -- To unsubscribe from this list: send the line unsubscribe kvm in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
[PATCH v3 0/8] pvclock fixes - v3
Hello, Avi, this version fixes the issues you raised in the last review. Hopte it is better now. A cpuid.txt wil follow as soon as we're set on everything, so I am not including it yet. Glauber Costa (8): Enable pvclock flags in vcpu_time_info structure Add a global synchronization point for pvclock change msr numbers for kvmclock add new KVMCLOCK cpuid feature export paravirtual cpuid flags in KVM_GET_SUPPORTED_CPUID Try using new kvm clock msrs don't compute pvclock adjustments if we trust the tsc Tell the guest we'll warn it about tsc stability arch/x86/include/asm/kvm_para.h| 13 arch/x86/include/asm/pvclock-abi.h |4 ++- arch/x86/include/asm/pvclock.h |1 + arch/x86/kernel/kvmclock.c | 56 ++- arch/x86/kernel/pvclock.c | 37 +++ arch/x86/kvm/x86.c | 40 +- 6 files changed, 128 insertions(+), 23 deletions(-) -- To unsubscribe from this list: send the line unsubscribe kvm in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html