From: Philippe Gerum <rpm@xenomai.org>
To: Jan Kiszka <jan.kiszka@domain.hid>
Cc: adeos-main <adeos-main@gna.org>
Subject: Re: [Adeos-main] DYNAMIC_FTRACE vs. I-pipe tracer
Date: Mon, 29 Jun 2009 17:39:07 +0200 [thread overview]
Message-ID: <1246289947.4040.20.camel@domain.hid> (raw)
In-Reply-To: <4A48D9A1.40509@domain.hid>
On Mon, 2009-06-29 at 17:11 +0200, Jan Kiszka wrote:
> Hi Philippe,
>
> I saw this for 2.6.30, but it obviously also made it into 2.6.29:
>
> http://git.denx.de/?p=ipipe-2.6.git;a=commitdiff;h=d302f7638b9b52ea90e3c042edd940449cedb7a1
>
> What precise problem does it try to solve? Is it a 2.6.30-only problem?
> The point is that, last time I checked, the dynamic ftrace approach was
> by far not I-pipe compatible. So I suspect we rather need a fix for the
> core issue, but keep DYNAMIC_FTRACE off.
There are compatible for basic usage at least, but the way the tracer
forced activation of the ftrace layer in 2.6.29+ was wrong in the first
place, and this basically wrecked the ppc64 boot sequence. But now that
it is right (at least non-x86s are happy with this now), we need
DYNAMIC_FTRACE to enable the tracepoints; otherwise we would only get a
few xenomai symbols in the trace log (seen on x86*/ppc*). As it is, we
do get meaningful traces via /proc/ipipe/trace, but I would not assume
that we could not break the kernel when fiddling with ftrace's debugfs
interface.
Quite frankly, moving the tracer over ftrace brought quite a lot of
issues to the non-x86 ports unfortunately (including significant
overhead issues on ppc64); the fact that ftrace is a moving target did
not help either. This integration probably requires more work.
>
> Jan
>
--
Philippe.
next prev parent reply other threads:[~2009-06-29 15:39 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-06-29 15:11 [Adeos-main] DYNAMIC_FTRACE vs. I-pipe tracer Jan Kiszka
2009-06-29 15:39 ` Philippe Gerum [this message]
2009-06-29 15:53 ` Jan Kiszka
2009-06-29 17:28 ` Philippe Gerum
2009-06-29 18:07 ` Jan Kiszka
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=1246289947.4040.20.camel@domain.hid \
--to=rpm@xenomai.org \
--cc=adeos-main@gna.org \
--cc=jan.kiszka@domain.hid \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.