Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753431AbZJYNhH (ORCPT ); Sun, 25 Oct 2009 09:37:07 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1753138AbZJYNhG (ORCPT ); Sun, 25 Oct 2009 09:37:06 -0400 Received: from mail-bw0-f227.google.com ([209.85.218.227]:60463 "EHLO mail-bw0-f227.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752807AbZJYNhF (ORCPT ); Sun, 25 Oct 2009 09:37:05 -0400 DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :cc:content-type; b=Yz067+QliCEouoiKnqzUI1IVoAeEPZDPr+6rei/Ul3mgGm34L1tWxMp4hAWTFp6G3r QqyOO9h8hFfWqfBhzZxVZREiq+4HSB19jojfJkGsQJ6MLGw1Wzm9CPoG1bcWq9OzHOeM waeQE/Giwij67YJvsh44xQRtxpFObwKs+5oOQ= MIME-Version: 1.0 In-Reply-To: <1256467717.6143.13.camel@falcon> References: <028867b99ec532b84963a35e7d552becc783cafc.1256135456.git.wuzhangjin@gmail.com> <2f73eae542c47ac5bbb9f7280e6c0271d193e90d.1256135456.git.wuzhangjin@gmail.com> <3f0d3515f74a58f4cfd11e61b62a129fdc21e3a7.1256135456.git.wuzhangjin@gmail.com> <96110ea5dd4d3d54eb97d0bb708a5bd81c7a50b5.1256135456.git.wuzhangjin@gmail.com> <53bdfdd95ec4fa00d4cc505bb5972cf21243a14d.1256135456.git.wuzhangjin@gmail.com> <26008418.post@talk.nabble.com> <1256467717.6143.13.camel@falcon> Date: Sun, 25 Oct 2009 14:37:07 +0100 Message-ID: <7a39cd850910250637h200e2dd2i4f725d55bb88c212@mail.gmail.com> Subject: Re: [PATCH -v4 9/9] tracing: add function graph tracer support for MIPS From: Patrik Kluba To: wuzhangjin@gmail.com Cc: linux-kernel@vger.kernel.org, GCC Patches , Adam Nemet , rostedt@goodmis.org, linux-mips@linux-mips.org, Thomas Gleixner , Ralf Baechle , Nicholas Mc Guire , Richard Sandiford , David Daney Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1780 Lines: 37 On Sun, Oct 25, 2009 at 11:48 AM, Wu Zhangjin wrote: > > do you mean if enabling PROFILE_BEFORE_PROLOGUE, there will be some > problems with module support using -mlong-calls? > No, there are no problems. I've tested it on friday, and function graph tracing was working correctly. I meant to say that 4.2.1 we use does not generate correct profile calls from kernel modules. Maybe this issue was fixed in newer releases, I did not check. I've applied a patch (don't remember where have I found that, maybe it was created by you) to our toolchain several months ago. I was thinking about dynamic tracing, and I think a toolchain patch can be avoided completely. We only need to make difference between "jal _mcount" and "jalr v1"-style mcount calls when replacing them with "nop" instructions in the code-patching function called by ftrace_convert_nops(). This can be done in 2 ways: 1) keeping old instructions - takes extra memory, not an option 2) using 2 separate instructions to replace with. One of them could be the normal NOP instruction, which expands to "sll r0, r0, 0". For the other we could use "sll r0, r0, 1" but as it has already special meaning (SSNOP) a better candidate could be something like "sll r1, r1, 0". This way we can decide which instruction to patch in when tracing is enabled for a function, eg. when the code patcher encounters a "sll r0, r0, 0" it emits a function call using JAL and when it encounters "sll r1, r1, 0" it emits a function call using "JALR v1". Regards, Patrik -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/