lkml.org 
[lkml]   [2013]   [Sep]   [23]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [Xen-devel] [PATCH v1 3/5] xen/PMU: Initialization code for Xen PMU
On 09/23/2013 10:18 AM, Boris Ostrovsky wrote:
> On 09/23/2013 09:26 AM, Konrad Rzeszutek Wilk wrote:
>> On Tue, Sep 10, 2013 at 11:31:48AM -0400, Boris Ostrovsky wrote:
>>> Map shared data structure that will hold CPU registers, VPMU context,
>>> VCPU/PCPI IDs of the VCPU interrupted by PMU interrupt. Hypervisor
>>> fills this information in its handler and passes it to the guest for
>>> further processing.
>>>
>>> Set up PMU VIRQ.
>>>
>>> Signed-off-by: Boris Ostrovsky <boris.ostrovsky@oracle.com>
>>> ---
>>> arch/x86/xen/Makefile | 2 +-
>>> arch/x86/xen/pmu.c | 122
>>> +++++++++++++++++++++++++++++++++++++++++
>>> arch/x86/xen/pmu.h | 12 ++++
>>> arch/x86/xen/smp.c | 31 ++++++++++-
>>> include/xen/interface/xen.h | 1 +
>>> include/xen/interface/xenpmu.h | 77 ++++++++++++++++++++++++++
>>> 6 files changed, 243 insertions(+), 2 deletions(-)
>>> create mode 100644 arch/x86/xen/pmu.c
>>> create mode 100644 arch/x86/xen/pmu.h
>>>
>>> diff --git a/arch/x86/xen/Makefile b/arch/x86/xen/Makefile
>>> index 96ab2c0..b187df5 100644
>>> --- a/arch/x86/xen/Makefile
>>> +++ b/arch/x86/xen/Makefile
>>> @@ -13,7 +13,7 @@ CFLAGS_mmu.o := $(nostackp)
>>> obj-y := enlighten.o setup.o multicalls.o mmu.o irq.o \
>>> time.o xen-asm.o xen-asm_$(BITS).o \
>>> grant-table.o suspend.o platform-pci-unplug.o \
>>> - p2m.o
>>> + p2m.o pmu.o
>> Perhaps guard the build of this based on CONFIG_PERF_EVENTS?
>>
>> That would of course mean you also have to create in xenpmu.h
>> static inline empy functions for xen_pmu_finish and xen_pmu_init in case
>> CONFIG_PERF_EVENTS is not set.
>
> This is interface header, am I allowed to do those sorts of things there?

Never mind that, I was thinking of a different file. Sorry.

>
> Also, *theoretically* other performance-measuring tools can use this
> framework, expect for perf-specific things like callbacks and the
> handler call. So maybe I should put CONFIG_PERF_EVENTS in pmu.c?
>

-boris



\
 
 \ /
  Last update: 2013-09-23 16:41    [W:0.038 / U:0.152 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site