linux-mips
[Top] [All Lists]

Re: [PATCH v2 2/6] mips dynamic function tracer support

To: wu zhangjin <wuzhangjin@gmail.com>
Subject: Re: [PATCH v2 2/6] mips dynamic function tracer support
From: Wang Liming <liming.wang@windriver.com>
Date: Wed, 03 Jun 2009 14:05:18 +0800
Cc: linux-mips@linux-mips.org, linux-kernel@vger.kernel.org, Wu Zhangjin <wuzj@lemote.com>, Steven Rostedt <rostedt@goodmis.org>, Ralf Baechle <ralf@linux-mips.org>, Ingo Molnar <mingo@elte.hu>, Andrew Morton <akpm@linux-foundation.org>, Frederic Weisbecker <fweisbec@gmail.com>, Thomas Gleixner <tglx@linutronix.de>, Nicholas Mc Guire <der.herr@hofr.at>
In-reply-to: <b00321320906020915n7ba241eqb3cb0de877af514d@mail.gmail.com>
Original-recipient: rfc822;linux-mips@linux-mips.org
References: <cover.1243604390.git.wuzj@lemote.com> <a00a91f6fc79b7d20b5b2193086e879dcafded46.1243604390.git.wuzj@lemote.com> <4A22281B.7020908@windriver.com> <b00321320906020915n7ba241eqb3cb0de877af514d@mail.gmail.com>
Sender: linux-mips-bounce@linux-mips.org
User-agent: Thunderbird 2.0.0.21 (Windows/20090302)
wu zhangjin wrote:
hi,

sorry, I'm so late to reply your E-mail, a little busy these days.

}
----------arch/mips/kernel/module.c:apply_r_mips_26_rel()-------------------

v is kernel _mcount's address, location is the address of the instrution
that should be relocated;

To resolve this problem, we may need to do more work, either on gcc or on
the kernel. So I want to hear your test result and if you have solution,
please let me know.


yes, current version of mips-specific dynamic ftrace not support modules yet.

there is similar solution implemented in PowerPC(something named trampoline),
although I did not look into it, but I'm sure we can implement the
mips-specific one
via imitating it.
Good hit. I may have a look on Powerpc implementation.


and currently, I'm planning to fix it in the -v3 of mips-specific
ftrace support.
OK. I'm looking forward to your -v3 patches.

Thanks.
Liming Wang

best wishes,
Wu Zhangjin



<Prev in Thread] Current Thread [Next in Thread>