diff options
| author | Jiejing Zhang <jiejzhang@nvidia.com> | 2013-08-28 14:20:12 +0800 |
|---|---|---|
| committer | Dan Willemsen <dwillemsen@nvidia.com> | 2013-09-14 13:45:08 -0700 |
| commit | 8d2ead8cf6ad78b1ae0b4b4578525616cd25176b (patch) | |
| tree | 29fc3efd406e93f566296b0db449caeda8a1096e /include/trace | |
| parent | 04bcbd24d8f82a8728a553e0e129321bbe77f925 (diff) | |
tracing: remove clock lock/unlock event.
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>
Diffstat (limited to 'include/trace')
| -rw-r--r-- | include/trace/events/power.h | 14 |
1 files changed, 0 insertions, 14 deletions
diff --git a/include/trace/events/power.h b/include/trace/events/power.h index c2980e74011c..23ae177241ba 100644 --- a/include/trace/events/power.h +++ b/include/trace/events/power.h @@ -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), |