From mboxrd@z Thu Jan 1 00:00:00 1970 From: Boris Ostrovsky Subject: Re: [PATCH v14 for-xen-4.5 00/21] x86/PMU: Xen PMU PV(H) support Date: Mon, 27 Oct 2014 09:47:59 -0400 Message-ID: <544E4D0F.20002@oracle.com> References: <1413580689-2750-1-git-send-email-boris.ostrovsky@oracle.com> <36292293.mK39DXonkm@amur> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii"; Format="flowed" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <36292293.mK39DXonkm@amur> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Dietmar Hahn , xen-devel@lists.xen.org Cc: kevin.tian@intel.com, keir@xen.org, JBeulich@suse.com, jun.nakajima@intel.com, andrew.cooper3@citrix.com, tim@xen.org, Aravind.Gopalakrishnan@amd.com, suravee.suthikulpanit@amd.com, dgdegra@tycho.nsa.gov List-Id: xen-devel@lists.xenproject.org On 10/27/2014 03:38 AM, Dietmar Hahn wrote: > Hi Boris, > > Am Freitag 17 Oktober 2014, 17:17:48 schrieb Boris Ostrovsky: >> Version 14 of PV(H) PMU patches. >> >> Changes in v14: >> >> * Moved struct xen_pmu_regs to pmu.h >> * Moved CHECK_pmu_* to an earlier patch (when structures are first introduced) >> * Added PMU_SAMPLE_REAL flag to indicate whether the sample was taken in real mode >> * Simplified slightly setting rules for xenpmu_data flags >> * Rewrote vpmu_force_context_switch() to again use continuations. (Returning EAGAIN >> to user would mean that VPMU mode may get into inconsistent state (across processors) >> and dealing with that is more compicated than I'd like). >> * Fixed msraddr_to_bitpos() and converted it into an inline >> * Replaced address range check in vmpu_do_interrupt() with guest_mode() >> * No error returns from __initcall >> * Rebased on top of recent VPMU changes >> * Various cleanups > I did some tests with the Intel part and all went fine, so > > Tested-by: Dietmar Hahn Excellent, thanks! > Btw. you use different names for the same structure in the linux kernel driver > part "struct xen_arch_pmu" and in the xen part "struct xen_pmu_arch". This is a > little bit confusing when searching through the code. So when possible ... ;-) > Many thanks! Linux driver had quite a few changes since I posted it last time (mostly due to requested API changes on Xen side) so the header files will need to be copied anew anyway. And structure names will be updated at that time. -boris