From mboxrd@z Thu Jan 1 00:00:00 1970 From: Boris Ostrovsky Subject: Re: [PATCH v5] x86/VPMU: implement ipc and arch filter flags Date: Wed, 6 Jan 2016 09:16:47 -0500 Message-ID: <568D21CF.3070104@oracle.com> References: <1451958218-4934-1-git-send-email-bgregg@netflix.com> <568D0BDA02000078000C3E32@prv-mh.provo.novell.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii"; Format="flowed" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <568D0BDA02000078000C3E32@prv-mh.provo.novell.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: Jan Beulich , Brendan Gregg Cc: andrew.cooper3@citrix.com, kevin.tian@intel.com, dietmar.hahn@ts.fujitsu.com, jun.nakajima@intel.com, xen-devel@lists.xen.org List-Id: xen-devel@lists.xenproject.org On 01/06/2016 06:43 AM, Jan Beulich wrote: >>>> On 05.01.16 at 02:43, wrote: >> This introduces a way to have a restricted VPMU, by specifying one of two >> predefined groups of PMCs to make available. For secure environments, this >> allows the VPMU to be used without needing to enable all PMCs. >> >> Signed-off-by: Brendan Gregg >> Reviewed-by: Boris Ostrovsky > Does this hold with ... > >> --- >> Changes in v5: >> * addressing review comments from Kevin: >> * simplified BTS DS_AREA filter >> * improvements to comments, incl. matching Intel SDM descriptions > ... all of these? > Yes. Note that this patch will conflict with http://lists.xenproject.org/archives/html/xen-devel/2015-12/msg02753.html (for 'case MSR_IA32_PEBS_ENABLE' in core2_vpmu_do_wrmsr()). I can rebase it on top of Brendan's if you prefer. -boris