Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753775AbZJUOfs (ORCPT ); Wed, 21 Oct 2009 10:35:48 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1753758AbZJUOfq (ORCPT ); Wed, 21 Oct 2009 10:35:46 -0400 Received: from mail-fx0-f218.google.com ([209.85.220.218]:64251 "EHLO mail-fx0-f218.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753751AbZJUOfp (ORCPT ); Wed, 21 Oct 2009 10:35:45 -0400 DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=from:to:cc:subject:date:message-id:x-mailer:in-reply-to:references; b=Exox2RPYx2E4ohm++AA1OVjyXCypROZY4P9YXWtVY4m4/72FuEMX8lKFi3jsv2DPcT BM2LVozSc3IAj0/S+QFGLAfZFiDw7OE/F51LGlR7CJnM1SYw9kAFtZCeL6lrnP2pLrjf d9uDvE8k9rDlZNSGVMF3boyz5cysYlzrcXFtQ= From: Wu Zhangjin To: linux-kernel@vger.kernel.org, linux-mips@linux-mips.org Cc: Wu Zhangjin , Thomas Gleixner , Ralf Baechle , Nicholas Mc Guire , Steven Rostedt Subject: [PATCH -v4 8/9] tracing: not trace mips_timecounter_init() in MIPS Date: Wed, 21 Oct 2009 22:35:02 +0800 Message-Id: X-Mailer: git-send-email 1.6.2.1 In-Reply-To: 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> <5dda13e8e3a9c9dba4bb7179183941bda502604f.1256135456.git.wuzhangjin@gmail.com> In-Reply-To: References: Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3274 Lines: 96 We use mips_timecounter_init() to get the timestamp in MIPS, so, it's better to not trace it, otherwise, it will goto recursion(hang) when using function graph tracer. but this patch have touched the common part in kernel/time/clocksource.c and include/linux/clocksource.h, so it is not good, perhaps the better solution is moving the whole mips_timecounter_init() implementation into arch/mips, but that will introduce source code duplication. any other solution? Signed-off-by: Wu Zhangjin --- arch/mips/include/asm/time.h | 2 +- arch/mips/kernel/csrc-r4k.c | 4 ++-- include/linux/clocksource.h | 2 +- kernel/time/clocksource.c | 4 ++-- 4 files changed, 6 insertions(+), 6 deletions(-) diff --git a/arch/mips/include/asm/time.h b/arch/mips/include/asm/time.h index b8af7fa..e1c660f 100644 --- a/arch/mips/include/asm/time.h +++ b/arch/mips/include/asm/time.h @@ -77,7 +77,7 @@ extern int init_r4k_timecounter(void); extern u64 r4k_timecounter_read(void); #endif -static inline u64 mips_timecounter_read(void) +static inline u64 notrace mips_timecounter_read(void) { #ifdef CONFIG_CSRC_R4K return r4k_timecounter_read(); diff --git a/arch/mips/kernel/csrc-r4k.c b/arch/mips/kernel/csrc-r4k.c index 4e7705f..0690bea 100644 --- a/arch/mips/kernel/csrc-r4k.c +++ b/arch/mips/kernel/csrc-r4k.c @@ -42,7 +42,7 @@ static struct timecounter r4k_tc = { .cc = NULL, }; -static cycle_t r4k_cc_read(const struct cyclecounter *cc) +static cycle_t notrace r4k_cc_read(const struct cyclecounter *cc) { return read_c0_count(); } @@ -66,7 +66,7 @@ int __init init_r4k_timecounter(void) return 0; } -u64 r4k_timecounter_read(void) +u64 notrace r4k_timecounter_read(void) { u64 clock; diff --git a/include/linux/clocksource.h b/include/linux/clocksource.h index 83d2fbd..2a02992 100644 --- a/include/linux/clocksource.h +++ b/include/linux/clocksource.h @@ -73,7 +73,7 @@ struct timecounter { * XXX - This could use some mult_lxl_ll() asm optimization. Same code * as in cyc2ns, but with unsigned result. */ -static inline u64 cyclecounter_cyc2ns(const struct cyclecounter *cc, +static inline u64 notrace cyclecounter_cyc2ns(const struct cyclecounter *cc, cycle_t cycles) { u64 ret = (u64)cycles; diff --git a/kernel/time/clocksource.c b/kernel/time/clocksource.c index 5e18c6a..9ce9d02 100644 --- a/kernel/time/clocksource.c +++ b/kernel/time/clocksource.c @@ -52,7 +52,7 @@ EXPORT_SYMBOL(timecounter_init); * The first call to this function for a new time counter initializes * the time tracking and returns an undefined result. */ -static u64 timecounter_read_delta(struct timecounter *tc) +static u64 notrace timecounter_read_delta(struct timecounter *tc) { cycle_t cycle_now, cycle_delta; u64 ns_offset; @@ -72,7 +72,7 @@ static u64 timecounter_read_delta(struct timecounter *tc) return ns_offset; } -u64 timecounter_read(struct timecounter *tc) +u64 notrace timecounter_read(struct timecounter *tc) { u64 nsec; -- 1.6.2.1 -- 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/