public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Tim Bird <tim.bird@am.sony.com>
To: Frederic Weisbecker <fweisbec@gmail.com>
Cc: "Uwe �" <u.kleine-koenig@pengutronix.de>,
	"linux kernel" <linux-kernel@vger.kernel.org>,
	"Steven Rostedt" <rostedt@goodmis.org>,
	"Ingo Molnar" <mingo@elte.hu>,
	linux-arm-kernel <linux-arm-kernel@lists.arm.linux.org.uk>,
	"Russell King" <rmk@arm.linux.org.uk>
Subject: Re: [PATCH] Add function graph tracer support for ARM
Date: Thu, 23 Apr 2009 15:40:58 -0700	[thread overview]
Message-ID: <49F0EE7A.8090400@am.sony.com> (raw)
In-Reply-To: <20090423221516.GB5976@nowhere>

Frederic Weisbecker wrote:
> On Thu, Apr 23, 2009 at 02:49:56PM -0700, Tim Bird wrote:
>> Uwe � wrote:
>>>> Signed-off-by: <tim.bird@am.sony.com>
>>> According to Documentation/SubmittingPatches you need to provide your
>>> real name in the S-o-b line.
>> OK -that's embarrassing - I'll fix this.
>> scripts/checkpatch.pl didn't catch this.  I may look at adding
>> something to checkpatch.pl to catch empty names.
>>
>>> For the lazy of us, can you point me^Uus to some documentation how to
>>> use the graph tracer?
>> It looks like Documentation/ftrace.txt is missing anything about
>> function graph tracing.
>>
>> Should I add a section?
> 
> 
> 
> There is already a section in Documentation/frace.txt as well in 2.6.30 :-)

LOL. :-)

> Oh, that's the first function graph trace from Arm I can see.
> /me feels a bit moved...
> 
> Just one childish caprice...it would be nice to see a part of a
> trace which shows specific Arm arch functions in the next changelog :-)

Will do.  Here's something to tide you over.  I have pages and
pages of the following (repeating over and over):

 0)               |  omap_pm_idle() {
 0) + 30.518 us   |    omap_dma_running();
 0)               |    __exception_text_start() {
 0)               |      irq_enter() {
 0)   0.000 us    |        idle_cpu();
 0)   0.000 us    |      }
 0)   0.000 us    |      irq_to_desc();
 0)               |      handle_level_irq() {
 0) + 30.517 us   |        omap_mask_irq();
 0)               |        omap_mask_ack_irq() {
 0)   0.000 us    |          omap_mask_irq();
 0) + 30.518 us   |        }
 0)               |        handle_IRQ_event() {
 0)               |          omap_mpu_timer1_interrupt() {
 0)               |            tick_handle_periodic() {
 0)               |              tick_periodic() {
 0)               |                do_timer() {
 0)               |                  update_wall_time() {
 0)   0.000 us    |                    mpu_read();
 0)   0.000 us    |                    update_xtime_cache();
 0) + 30.517 us   |                    clocksource_get_next();
 0) ! 122.070 us  |                  }
 0) ! 183.106 us  |                }
 0)               |                update_process_times() {
 0)               |                  account_process_tick() {
 0) + 30.517 us   |                    account_idle_time();
 0) + 30.517 us   |                  }
 0)               |                  run_local_timers() {
 0)               |                    hrtimer_run_queues() {
 0)   0.000 us    |                      current_kernel_time();
 0) + 30.518 us   |                    }
 0)   0.000 us    |                    raise_softirq();
 0)               |                    softlockup_tick() {
 0)               |                      get_timestamp() {
 0)   0.000 us    |                        sched_clock();
 0)   0.000 us    |                      }
 0) + 61.035 us   |                    }
 0) ! 122.071 us  |                  }
 0)               |                  rcu_pending() {
 0)   0.000 us    |                    __rcu_pending();
 0)   0.000 us    |                    __rcu_pending();
 0) + 61.035 us   |                  }
 0)   0.000 us    |                  printk_tick();
 0)               |                  scheduler_tick() {
 0)   0.000 us    |                    sched_clock();
 0) + 30.518 us   |                    task_tick_idle();
 0) + 61.035 us   |                  }
 0)   0.000 us    |                  run_posix_cpu_timers();
 0) ! 305.176 us  |                }
 0)   0.000 us    |                profile_tick();
 0) ! 518.799 us  |              }
 0) ! 518.799 us  |            }
 0) ! 549.316 us  |          }
 0) ! 549.316 us  |        }
 0)   0.000 us    |        note_interrupt();
 0)   0.000 us    |        omap_unmask_irq();
 0) ! 671.386 us  |      }
 0)               |      irq_exit() {
 0)               |        __do_softirq() {
 0)               |          run_timer_softirq() {
 0)   0.000 us    |            hrtimer_run_pending();
 0) + 30.518 us   |          }
 0)   0.000 us    |          _local_bh_enable();
 0) + 61.036 us   |        }
 0) + 91.553 us   |      }
 0) ! 823.975 us  |    }
 0) ! 9979.248 us |  }

Enjoy :-)
 -- Tim

=============================
Tim Bird
Architecture Group Chair, CE Linux Forum
Senior Staff Engineer, Sony Corporation of America
=============================


  reply	other threads:[~2009-04-23 22:41 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-04-23 18:08 [PATCH] Add function graph tracer support for ARM Tim Bird
2009-04-23 18:39 ` Uwe Kleine-König
2009-04-23 21:49   ` Tim Bird
2009-04-23 22:15     ` Frederic Weisbecker
2009-04-23 22:40       ` Tim Bird [this message]
2009-04-24  6:44     ` Uwe Kleine-König
2009-04-24 18:00       ` Tim Bird
2009-04-23 19:20 ` Frederic Weisbecker
2009-04-23 22:33   ` Tim Bird
2009-04-23 23:42     ` Frederic Weisbecker
2009-04-24  6:32     ` Uwe Kleine-König

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=49F0EE7A.8090400@am.sony.com \
    --to=tim.bird@am.sony.com \
    --cc=fweisbec@gmail.com \
    --cc=linux-arm-kernel@lists.arm.linux.org.uk \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mingo@elte.hu \
    --cc=rmk@arm.linux.org.uk \
    --cc=rostedt@goodmis.org \
    --cc=u.kleine-koenig@pengutronix.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox