linux-mips.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Wu Zhangjin <wuzhangjin@gmail.com>
To: Ralf Baechle <ralf@linux-mips.org>
Cc: linux-mips <linux-mips@linux-mips.org>,
	David Daney <david.s.daney@gmail.com>,
	Wu Zhangjin <wuzhangjin@gmail.com>
Subject: [PATCH v7 0/9] tracing: MIPS: add misc fixups and cleanups
Date: Fri, 14 May 2010 19:08:25 +0800	[thread overview]
Message-ID: <cover.1273834561.git.wuzhangjin@gmail.com> (raw)

From: Wu Zhangjin <wuzhangjin@gmail.com>

This patchset adds misc fixups and cleanups for Ftrace of MIPS:

  + Fix the support of 32bit support with -mmcount-ra-address of gcc 4.5
    o tracing: MIPS: mcount.S: Fix the argument passing of the 32bit support with
      gcc 4.5, The argument is passed by $12 in 32bit, not t0.
    o tracing: MIPS: Fixup of the 32bit support with -mmcount-ra-address
      For 32bit kernel, the offset of "b 1f" should be 5 instructions, not only 4.
 
  + Speedup the dynamic function tracer
    o tracing: MIPS: Reduce the overhead of dynamic Function Tracer
      In the old implementation, we have encode the 'nop' instruction and the
      instruction of calling to mcount at run-time, which may add some
      overhead.  We reduce this overhead via encoding them when initializing
      the dynamic function tracer.

  + Lots of cleanups
    o The other patches.

----------------
v6 -> v7:

  o Apply the feedback from David Daney:
  define a macro MCOUNT_RA_ADDRESS_REG instead of the magic number $12 for the patch
  "tracing: MIPS: mcount.S: Fix the argument passing of the 32bit
    support with gcc 4.5"

v5 -> v6:

  o splits up the old v5 revision into several patches to make the maintainer
  happier to review it.

Regards,
        Wu Zhangjin


Wu Zhangjin (9):
  tracing: MIPS: mcount.S: merge the same continuous #ifdefs
  tracing: MIPS: mcount.S: cleanup the arguments of
    prepare_ftrace_return
  tracing: MIPS: mcount.S: cleanup of the comments
  tracing: MIPS: mcount.S: Fix the argument passing of the 32bit
    support with gcc 4.5
  tracing: MIPS: Fixup of the 32bit support with -mmcount-ra-address
  tracing: MIPS: cleanup of the instructions
  tracing: MIPS: Reduce the overhead of dynamic Function Tracer
  tracing: MIPS: cleanup of function graph tracer
  tracing: MIPS: cleanup of the address space checking

 arch/mips/kernel/ftrace.c |  184 +++++++++++++++++++++++++++------------------
 arch/mips/kernel/mcount.S |   55 ++++++++-----
 2 files changed, 146 insertions(+), 93 deletions(-)

             reply	other threads:[~2010-05-14 11:09 UTC|newest]

Thread overview: 23+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-05-14 11:08 Wu Zhangjin [this message]
2010-05-14 11:08 ` [PATCH 1/9] tracing: MIPS: mcount.S: merge the same continuous #ifdefs Wu Zhangjin
2010-05-27 11:27   ` Ralf Baechle
2010-05-14 11:08 ` [PATCH 2/9] tracing: MIPS: mcount.S: cleanup the arguments of prepare_ftrace_return Wu Zhangjin
2010-05-27 11:27   ` Ralf Baechle
2010-05-14 11:08 ` [PATCH 3/9] tracing: MIPS: mcount.S: cleanup of the comments Wu Zhangjin
2010-05-27 11:27   ` Ralf Baechle
2010-05-14 11:08 ` [PATCH 4/9] tracing: MIPS: mcount.S: Fix the argument passing of the 32bit support with gcc 4.5 Wu Zhangjin
2010-05-27 11:27   ` Ralf Baechle
2010-05-14 11:08 ` [PATCH 5/9] tracing: MIPS: Fixup of the 32bit support with -mmcount-ra-address Wu Zhangjin
2010-05-27 11:27   ` Ralf Baechle
2010-05-14 11:08 ` [PATCH 6/9] tracing: MIPS: cleanup of the instructions Wu Zhangjin
2010-05-27 11:27   ` Ralf Baechle
2010-05-14 11:08 ` [PATCH 7/9] tracing: MIPS: Reduce the overhead of dynamic Function Tracer Wu Zhangjin
2010-05-14 17:15   ` David Daney
2010-05-15  2:43     ` Wu Zhangjin
2010-05-16 17:04       ` David Daney
2010-05-27 11:22         ` Ralf Baechle
2010-05-27 11:28           ` Ralf Baechle
2010-05-14 11:08 ` [PATCH 8/9] tracing: MIPS: cleanup of function graph tracer Wu Zhangjin
2010-05-27 11:28   ` Ralf Baechle
2010-05-14 11:08 ` [PATCH 9/9] tracing: MIPS: cleanup of the address space checking Wu Zhangjin
2010-05-27 11:28   ` Ralf Baechle

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=cover.1273834561.git.wuzhangjin@gmail.com \
    --to=wuzhangjin@gmail.com \
    --cc=david.s.daney@gmail.com \
    --cc=linux-mips@linux-mips.org \
    --cc=ralf@linux-mips.org \
    /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;
as well as URLs for NNTP newsgroup(s).