Mirror of the lttng-dev mailing list
 help / color / mirror / Atom feed
From: hgu1972@gmail.com (Harald Gustafsson)
Subject: [ltt-dev] OMAP3/4 trace clock and lockdep tracing
Date: Fri, 8 Apr 2011 10:55:54 +0200	[thread overview]
Message-ID: <BANLkTinVg3ZZKtRSEP7qMw546nBZVvAPnA@mail.gmail.com> (raw)
In-Reply-To: <BLU0-SMTP86985EF8EEA9698CAE849E96A40@phx.gbl>

I took a look at the Linaro lttng git, but they don't seem to have
adopted it more for the OMAP4 than in your git tree. There is a new
pm44xx.c file which don't contain any resync_trace_clock() calls which
should be needed and the trace-clock.c file is your ordinary one.

2011/4/7 Mathieu Desnoyers <compudj at krystal.dyndns.org>:
> Please note that you can be woken up by an interrupt, which means that
> an IRQ handler will be executed before you resume to execute the
> instruction that follows the WFI. Do you resync when woken up by these
> interrupts ?
No, which might be causing the issues, but do you resync in the IRQ
for OMAP3 I could not find any code that did that? rather the comment
saying that IRQ might have run before the call to the resync in the
pm34xx.c file.

>> ? ? ? clock_val = cnt32_to_63(clock32k->read(clock32k)) << TRACE_CLOCK_32K_BITSHIFT;
>
> You might want to use the trace clock 32 to 64 provided by LTTng instead
> of cnt_32_to_63. See the kernel/trace/trace-clock-32-to-64.c.

I did consider the trace-clock-32-to-64.c but decided that it was
overkill for extending a continuous running low rate clock, the
cnt32_to_63 macro in include/linux/cnt32_to_63.h is commonly used to
extend counters at least on ARM arch.

Harald



      parent reply	other threads:[~2011-04-08  8:55 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-04-05  8:54 Harald Gustafsson
2011-04-06  2:58 ` Mathieu Desnoyers
     [not found] ` <BLU0-SMTP50EBA923E8F17BCC6A80BD96A50@phx.gbl>
2011-04-06  8:32   ` Harald Gustafsson
2011-04-06 16:42     ` Mathieu Desnoyers
     [not found]     ` <BLU0-SMTP92A8ACAB7C397862CBB76D96A50@phx.gbl>
2011-04-07 12:57       ` Harald Gustafsson
2011-04-07 16:47         ` Mathieu Desnoyers
     [not found]         ` <BLU0-SMTP86985EF8EEA9698CAE849E96A40@phx.gbl>
2011-04-08  8:55           ` Harald Gustafsson [this message]

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=BANLkTinVg3ZZKtRSEP7qMw546nBZVvAPnA@mail.gmail.com \
    --to=hgu1972@gmail.com \
    /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