Re: [PATCH 7/7] sparc64: Add function graph tracer support.

[Frederic Weisbecker <fweisbec@gmail.com>]

On Tue, Apr 13, 2010 at 03:39:21PM -0400, Rostedt wrote:
>
>
> On Apr 13, 2010, at 3:18 PM, Frederic Weisbecker <fweisbec@gmail.com>  
> wrote:
>
>> On Mon, Apr 12, 2010 at 11:43:00PM -0700, David Miller wrote:
>>>
>>> Signed-off-by: David S. Miller <davem@davemloft.net>
>>> ---
>>> arch/sparc/Kconfig              |    2 +
>>> arch/sparc/kernel/Makefile      |    1 +
>>> arch/sparc/kernel/ftrace.c      |   58 +++++++++++++++++++++++++++++ 
>>> +++++++
>>> arch/sparc/kernel/irq_64.c      |    3 +-
>>> arch/sparc/kernel/kgdb_64.c     |    3 +-
>>> arch/sparc/kernel/pcr.c         |    3 +-
>>> arch/sparc/kernel/smp_64.c      |   11 ++++---
>>> arch/sparc/kernel/time_64.c     |    3 +-
>>> arch/sparc/kernel/vmlinux.lds.S |    1 +
>>> arch/sparc/lib/mcount.S         |   62 +++++++++++++++++++++++++++++ 
>>> ++++++----
>>> 10 files changed, 132 insertions(+), 15 deletions(-)
>>
>>
>>
>> Works very well :)
>>
>> 63)               |  do_sparc64_fault() {
>> 63)   6.153 us    |    down_read_trylock();
>> 63)   5.164 us    |    find_vma();
>> 63)               |    handle_mm_fault() {
>> 63)   2.966 us    |      add_preempt_count();
>> 63)   2.856 us    |      sub_preempt_count();
>> 63)               |      _raw_spin_lock() {
>> 63)   2.857 us    |        add_preempt_count();
>> 63)   3.076 us    |        add_preempt_count();
>> 63)   2.527 us    |        sub_preempt_count();
>> 63) + 23.513 us   |      }
>>
>
> I just had to comment on the CPU count.
>
> Nice ;-)
>
> -- Steve


Heh yeah. The trace/trace_pipe in the main directory are basically
unusable there. Only per_cpu can make it :)