Re: [PATCH v2 0/9] Various VPMU patches, version 2

[Suravee Suthikulanit <suravee.suthikulpanit@amd.com>]

Overall looks good. Tested.

Thanks,

Suravee

On 4/12/2013 12:59 PM, Boris Ostrovsky wrote:
> Version 2 of VPMU patches.
>
> Changes:
> * Load context on MSR read on SVM and then read MSR from HW (Suravee's suggestion)
> * Group VPMU state bits separate from feature bits (Dietmar's suggestion)
> * Clean up (int->unsigned int for loop control, trailing spaces)
> * Rename VPMU_STOPPED to VPMU_FROZEN (*slightly* more sensible since it is
> possible to be both STOPPED and RUNNING)
> * Add patch for dumping VPMU state on AMD
> * Add Reviewed-by/Tested-by/Acked-by tags that have been posted for v1
>
> -boris
>
>
> Boris Ostrovsky (9):
>    x86/AMD: Allow more fine-grained control of VMCB MSR Permission Map
>    x86/AMD: Do not intercept access to performance counters MSRs
>    x86/AMD: Load context when attempting to read VPMU MSRs
>    x86/AMD: Stop counters on VPMU save
>    x86/VPMU: Add Haswell support
>    x86/VPMU: Factor out VPMU common code
>    x86/VPMU: Save/restore VPMU only when necessary
>    x86/AMD: Clean up context_update() in AMD VPMU code
>    x86/AMD: Dump AMD VPMU info
>
>   xen/arch/x86/domain.c                    |  14 ++-
>   xen/arch/x86/hvm/svm/svm.c               |  19 ++-
>   xen/arch/x86/hvm/svm/vpmu.c              | 197 +++++++++++++++++++++++--------
>   xen/arch/x86/hvm/vmx/vmx.c               |   2 -
>   xen/arch/x86/hvm/vmx/vpmu_core2.c        |  48 ++++----
>   xen/arch/x86/hvm/vpmu.c                  | 114 ++++++++++++++++--
>   xen/include/asm-x86/hvm/svm/vmcb.h       |   8 +-
>   xen/include/asm-x86/hvm/vmx/vpmu_core2.h |   1 -
>   xen/include/asm-x86/hvm/vpmu.h           |  15 ++-
>   9 files changed, 312 insertions(+), 106 deletions(-)
>