tracing: remove clock lock/unlock event.
Jiejing Zhang [Wed, 28 Aug 2013 06:20:12 +0000 (14:20 +0800)]
This patch is removed  clock lock/unlock event.

For those reasons:
1. events are *too* much, because clock tree, one lock operation
   will invoke too much clock_lock & clock_unlock event.

compare these two traces:
trace-cmd record -e regulator -e power  16000 -s 1000000 sleep 10
3162112 byte

trace-cmd record -e regulator -e power -v -e "clock_*lock" -b 16000 -s 1000000 sleep 10
585726 byte

The trace size was 1/6 between with lock & without lock events.

2. the debug function of adding this event is not clear.

Change-Id: I6e8d93487c8cc472990fd6a21d7e1f4c63e24556
Signed-off-by: Jiejing Zhang <jiejzhang@nvidia.com>
Signed-off-by: Eric Miao <emiao@nvidia.com>
Reviewed-on: http://git-master/r/269324
GVS: Gerrit_Virtual_Submit
Reviewed-by: Bharat Nihalani <bnihalani@nvidia.com>

arch/arm/mach-tegra/clock.h
include/trace/events/power.h

index d692ba3..e79ba01 100644 (file)
@@ -413,8 +413,6 @@ static inline bool clk_cansleep(struct clk *c)
 
 static inline void clk_lock_save(struct clk *c, unsigned long *flags)
 {
-       trace_clock_lock(c->name, c->rate, raw_smp_processor_id());
-
        if (clk_cansleep(c)) {
                *flags = 0;
                mutex_lock(&c->mutex);
@@ -434,8 +432,6 @@ static inline void clk_unlock_restore(struct clk *c, unsigned long *flags)
        } else {
                spin_unlock_irqrestore(&c->spinlock, *flags);
        }
-
-       trace_clock_unlock(c->name, c->rate, raw_smp_processor_id());
 }
 
 static inline int tegra_clk_prepare_enable(struct clk *c)
index c2980e7..23ae177 100644 (file)
@@ -247,20 +247,6 @@ TRACE_EVENT(clock_set_parent,
        TP_printk("%s parent=%s", __get_str(name), __get_str(parent_name))
 );
 
-DEFINE_EVENT(clock, clock_lock,
-
-       TP_PROTO(const char *name, unsigned int state, unsigned int cpu_id),
-
-       TP_ARGS(name, state, cpu_id)
-);
-
-DEFINE_EVENT(clock, clock_unlock,
-
-       TP_PROTO(const char *name, unsigned int state, unsigned int cpu_id),
-
-       TP_ARGS(name, state, cpu_id)
-);
-
 DEFINE_EVENT(clock, clock_set_start,
 
        TP_PROTO(const char *name, unsigned int state, unsigned int cpu_id),