Re: [PATCH 1/1] ARM: oprofile: add A5/A7/A15 entries in op_perf_name

2012-11-23 Thread Robert Richter
On 20.11.12 17:08:17, Will Deacon wrote: > > The advantage of a solution where userland updates cpu_type is that we > > never need to update the kernel anymore. This means, cpu detection can > > be part of the tools. > > True, but the kernel-side perf code still needs updating to support the new

Re: [PATCH 1/1] ARM: oprofile: add A5/A7/A15 entries in op_perf_name

2012-11-23 Thread Robert Richter
On 20.11.12 17:08:17, Will Deacon wrote: The advantage of a solution where userland updates cpu_type is that we never need to update the kernel anymore. This means, cpu detection can be part of the tools. True, but the kernel-side perf code still needs updating to support the new CPU so

Re: [PATCH 1/1] ARM: oprofile: add A5/A7/A15 entries in op_perf_name

2012-11-20 Thread Will Deacon
On Tue, Nov 20, 2012 at 05:06:01PM +, Robert Richter wrote: > On 20.11.12 16:55:17, Will Deacon wrote: > > On Tue, Nov 20, 2012 at 04:31:58PM +, Robert Richter wrote: > > > I am thinking of the following: > > > > > > # cat /root/cpu_type > > > arm/armv7-ca5 > > > # cat

Re: [PATCH 1/1] ARM: oprofile: add A5/A7/A15 entries in op_perf_name

2012-11-20 Thread Robert Richter
On 20.11.12 16:55:17, Will Deacon wrote: > On Tue, Nov 20, 2012 at 04:31:58PM +, Robert Richter wrote: > > I am thinking of the following: > > > > # cat /root/cpu_type > > arm/armv7-ca5 > > # cat /dev/oprofile/cpu_type > > unknown > > # mount --bind /root/cpu_type /dev/oprofile/cpu_type

Re: [PATCH 1/1] ARM: oprofile: add A5/A7/A15 entries in op_perf_name

2012-11-20 Thread Will Deacon
On Tue, Nov 20, 2012 at 04:31:58PM +, Robert Richter wrote: > On 20.11.12 15:57:17, Will Deacon wrote: > > On Tue, Nov 20, 2012 at 12:17:47PM +, Robert Richter wrote: > > > > > > since this is just an update of cpu detection I would be willing to > > > include this into kernel code

Re: [PATCH 1/1] ARM: oprofile: add A5/A7/A15 entries in op_perf_name

2012-11-20 Thread Robert Richter
On 20.11.12 15:57:17, Will Deacon wrote: > On Tue, Nov 20, 2012 at 12:17:47PM +, Robert Richter wrote: > > Will, > > > > On 05.11.12 11:31:03, Will Deacon wrote: > > > > diff --git a/arch/arm/oprofile/common.c b/arch/arm/oprofile/common.c > > > > index 99c63d4b..ec10db1 100644 > > > > ---

Re: [PATCH 1/1] ARM: oprofile: add A5/A7/A15 entries in op_perf_name

2012-11-20 Thread Will Deacon
On Tue, Nov 20, 2012 at 12:17:47PM +, Robert Richter wrote: > Will, > > On 05.11.12 11:31:03, Will Deacon wrote: > > > diff --git a/arch/arm/oprofile/common.c b/arch/arm/oprofile/common.c > > > index 99c63d4b..ec10db1 100644 > > > --- a/arch/arm/oprofile/common.c > > > +++

Re: [PATCH 1/1] ARM: oprofile: add A5/A7/A15 entries in op_perf_name

2012-11-20 Thread Robert Richter
Will, On 05.11.12 11:31:03, Will Deacon wrote: > > diff --git a/arch/arm/oprofile/common.c b/arch/arm/oprofile/common.c > > index 99c63d4b..ec10db1 100644 > > --- a/arch/arm/oprofile/common.c > > +++ b/arch/arm/oprofile/common.c > > @@ -37,8 +37,11 @@ static struct op_perf_name { > > {

Re: [PATCH 1/1] ARM: oprofile: add A5/A7/A15 entries in op_perf_name

2012-11-20 Thread Robert Richter
Will, On 05.11.12 11:31:03, Will Deacon wrote: diff --git a/arch/arm/oprofile/common.c b/arch/arm/oprofile/common.c index 99c63d4b..ec10db1 100644 --- a/arch/arm/oprofile/common.c +++ b/arch/arm/oprofile/common.c @@ -37,8 +37,11 @@ static struct op_perf_name { { xscale1,

Re: [PATCH 1/1] ARM: oprofile: add A5/A7/A15 entries in op_perf_name

2012-11-20 Thread Will Deacon
On Tue, Nov 20, 2012 at 12:17:47PM +, Robert Richter wrote: Will, On 05.11.12 11:31:03, Will Deacon wrote: diff --git a/arch/arm/oprofile/common.c b/arch/arm/oprofile/common.c index 99c63d4b..ec10db1 100644 --- a/arch/arm/oprofile/common.c +++ b/arch/arm/oprofile/common.c @@

Re: [PATCH 1/1] ARM: oprofile: add A5/A7/A15 entries in op_perf_name

2012-11-20 Thread Robert Richter
On 20.11.12 15:57:17, Will Deacon wrote: On Tue, Nov 20, 2012 at 12:17:47PM +, Robert Richter wrote: Will, On 05.11.12 11:31:03, Will Deacon wrote: diff --git a/arch/arm/oprofile/common.c b/arch/arm/oprofile/common.c index 99c63d4b..ec10db1 100644 ---

Re: [PATCH 1/1] ARM: oprofile: add A5/A7/A15 entries in op_perf_name

2012-11-20 Thread Will Deacon
On Tue, Nov 20, 2012 at 04:31:58PM +, Robert Richter wrote: On 20.11.12 15:57:17, Will Deacon wrote: On Tue, Nov 20, 2012 at 12:17:47PM +, Robert Richter wrote: since this is just an update of cpu detection I would be willing to include this into kernel code anyway.

Re: [PATCH 1/1] ARM: oprofile: add A5/A7/A15 entries in op_perf_name

2012-11-20 Thread Robert Richter
On 20.11.12 16:55:17, Will Deacon wrote: On Tue, Nov 20, 2012 at 04:31:58PM +, Robert Richter wrote: I am thinking of the following: # cat /root/cpu_type arm/armv7-ca5 # cat /dev/oprofile/cpu_type unknown # mount --bind /root/cpu_type /dev/oprofile/cpu_type # cat

Re: [PATCH 1/1] ARM: oprofile: add A5/A7/A15 entries in op_perf_name

2012-11-20 Thread Will Deacon
On Tue, Nov 20, 2012 at 05:06:01PM +, Robert Richter wrote: On 20.11.12 16:55:17, Will Deacon wrote: On Tue, Nov 20, 2012 at 04:31:58PM +, Robert Richter wrote: I am thinking of the following: # cat /root/cpu_type arm/armv7-ca5 # cat /dev/oprofile/cpu_type unknown

Re: [PATCH 1/1] ARM: oprofile: add A5/A7/A15 entries in op_perf_name

2012-11-05 Thread Will Deacon
On Fri, Nov 02, 2012 at 10:43:36AM +, jgq...@gmail.com wrote: > From: Xiao Jiang > > Add related name for A5/A7/A15 which are consistent with the OProfile > user ABI. > > Signed-off-by: Xiao Jiang > --- > arch/arm/oprofile/common.c |3 +++ > 1 files changed, 3 insertions(+), 0

Re: [PATCH 1/1] ARM: oprofile: add A5/A7/A15 entries in op_perf_name

2012-11-05 Thread Will Deacon
On Fri, Nov 02, 2012 at 10:43:36AM +, jgq...@gmail.com wrote: From: Xiao Jiang jgq...@gmail.com Add related name for A5/A7/A15 which are consistent with the OProfile user ABI. Signed-off-by: Xiao Jiang jgq...@gmail.com --- arch/arm/oprofile/common.c |3 +++ 1 files changed, 3

[PATCH 1/1] ARM: oprofile: add A5/A7/A15 entries in op_perf_name

2012-11-02 Thread jgq516
From: Xiao Jiang Add related name for A5/A7/A15 which are consistent with the OProfile user ABI. Signed-off-by: Xiao Jiang --- arch/arm/oprofile/common.c |3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git a/arch/arm/oprofile/common.c b/arch/arm/oprofile/common.c index

[PATCH 1/1] ARM: oprofile: add A5/A7/A15 entries in op_perf_name

2012-11-02 Thread jgq516
From: Xiao Jiang jgq...@gmail.com Add related name for A5/A7/A15 which are consistent with the OProfile user ABI. Signed-off-by: Xiao Jiang jgq...@gmail.com --- arch/arm/oprofile/common.c |3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git a/arch/arm/oprofile/common.c