Re: [PATCH v3 0/8] pvclock fixes - v3

2010-05-11 Thread Avi Kivity

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

2010-05-05 Thread Glauber Costa
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