From mboxrd@z Thu Jan 1 00:00:00 1970 From: Suravee Suthikulanit Subject: Re: [PATCH v2 0/9] Various VPMU patches, version 2 Date: Mon, 15 Apr 2013 13:04:02 -0500 Message-ID: <516C4112.3000204@amd.com> References: <1365789566-3733-1-git-send-email-boris.ostrovsky@oracle.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii"; Format="flowed" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1365789566-3733-1-git-send-email-boris.ostrovsky@oracle.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Boris Ostrovsky Cc: jun.nakajima@intel.com, haitao.shan@intel.com, jacob.shin@amd.com, dietmar.hahn@ts.fujitsu.com, xen-devel@lists.xen.org, JBeulich@suse.com List-Id: xen-devel@lists.xenproject.org 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(-) >