On 8/11/15 08:21, Paolo Bonzini wrote:
On 07/08/2015 21:53, Wei Huang wrote:These two trivial patches are related to x86 vPMU code. They were actually suggested by Andrew Jones while he was reviewing the last big vPMU patch set. These patches have been compiled and tested on AMD system using a 64-bit guest VM with various perf commands (e.g. bench, test, top, stat). No obvious problems were found. Thanks, -Wei Wei Huang (2): KVM: x86/vPMU: Move the definition of kvm_pmu_ops to arch-specific files KVM: x86/vPMU: Fix unnecessary signed extesion for AMD PERFCTRn arch/x86/kvm/pmu.h | 2 -- arch/x86/kvm/pmu_amd.c | 2 -- arch/x86/kvm/svm.c | 1 + arch/x86/kvm/vmx.c | 1 + 4 files changed, 2 insertions(+), 4 deletions(-)Applied patch 2. For patch 1 I'm not sure, because I do not really like 1) externs in .c files; 2) globals with no declarations in a .h file. So I'm leaving it out while I think more about it.
Thanks. The first one is minor anyway. I won't complain about it. :-) -Wei
Paolo
-- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
