Re: [PATCH] basic perf support for sparc
From: Ingo Molnar
Date: Fri Sep 04 2009 - 01:03:18 EST
* David Miller <davem@xxxxxxxxxxxxx> wrote:
> From: Jens Axboe <jens.axboe@xxxxxxxxxx>
> Date: Mon, 17 Aug 2009 08:48:51 +0200
> > sparc: add basic support for 'perf'
> > This wires up the perf_counter_open() syscall so that basic
> > software support for perf is working.
> > Signed-off-by: Jens Axboe <jens.axboe@xxxxxxxxxx>
> Does this build for you without adding an
> arch/sparc/include/asm/perf_counter.h that looks
> something like the following?
> #ifndef _ASM_SPARC_PERF_COUNTER_H
> #define _ASM_SPARC_PERF_COUNTER_H
> #define PERF_COUNTER_INDEX_OFFSET 0
> Or is this somehow now required in the -tip trees?
This used to be required but i recently fixed this (and that fix is
upstream as well) via:
f738eb1: perf_counter: Fix the PARISC build
there's now a default define of 0 so you dont have to define it and
can leave out this chunk.
( That index is only interesting if the architecture has a way to
allow unprivileged user-space to access counter registers
directly. In that case the index reflects the offset from the
(constantly changing) dynamix index which we put into the mmap
header. With Sparc not having a hw-PMU implementation this index
is entirely uninteresting at this stage. )
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/