From a9eb302abaccdd3609d81ba75434cccaed49f4fd Mon Sep 17 00:00:00 2001 From: David Garcia Quintas Date: Mon, 27 Apr 2015 19:17:20 -0700 Subject: STAP annotation working. Test annotations made to client_channel.c (start_rpc) and channel_create.c (grpc_channel_create). --- test/core/profiling/mark_timings.stp | 32 ++++++++++++++++++++++++++++++++ 1 file changed, 32 insertions(+) create mode 100644 test/core/profiling/mark_timings.stp (limited to 'test/core') diff --git a/test/core/profiling/mark_timings.stp b/test/core/profiling/mark_timings.stp new file mode 100644 index 0000000000..a7ccc83c61 --- /dev/null +++ b/test/core/profiling/mark_timings.stp @@ -0,0 +1,32 @@ +global starts, times, times_per_tag + +probe process.mark("timing_ns_begin") { + starts[$arg1, tid()] = gettimeofday_ns(); +} + +probe process.mark("timing_ns_end") { + tag = $arg1 + t = gettimeofday_ns(); + if (s = starts[tag, tid()]) { + times[tag, tid()] <<< t-s; + delete starts[tag, tid()]; + } +} + +probe end { + printf("%15s %9s %10s %10s %10s %10s\n", "tag", "tid", "count", + "min(ns)", "avg(ns)", "max(ns)"); + foreach ([tag+, tid] in times) { + printf("%15X %9d %10d %10d %10d %10d\n", tag, tid, @count(times[tag, tid]), + @min(times[tag, tid]), @avg(times[tag, tid]), @max(times[tag, tid])); + } + + printf("Per tag average of averages\n"); + foreach ([tag+, tid] in times) { + times_per_tag[tag] <<< @avg(times[tag, tid]); + } + printf("%15s %10s %10s\n", "tag", "count", "avg(ns)"); + foreach ([tag+] in times_per_tag) { + printf("%15X %10d %10d\n", tag, @count(times_per_tag[tag]), @avg(times_per_tag[tag])); + } +} -- cgit v1.2.3 From 735987bea0e48cfe579b48a969d71298ae973c31 Mon Sep 17 00:00:00 2001 From: David Garcia Quintas Date: Wed, 29 Apr 2015 10:41:38 -0700 Subject: Added documentation to stap script. --- test/core/profiling/mark_timings.stp | 14 ++++++++++---- 1 file changed, 10 insertions(+), 4 deletions(-) (limited to 'test/core') diff --git a/test/core/profiling/mark_timings.stp b/test/core/profiling/mark_timings.stp index a7ccc83c61..45d5e0d217 100644 --- a/test/core/profiling/mark_timings.stp +++ b/test/core/profiling/mark_timings.stp @@ -1,10 +1,15 @@ -global starts, times, times_per_tag +/* This script requires a command line argument, to be used in the "process" + * probe definition. + * + * For a statically build binary, that'd be the name of the binary itself. + * For dinamically built ones, point to the location of the libgprc.so being + * used. */ -probe process.mark("timing_ns_begin") { +probe process(@1).mark("timing_ns_begin") { starts[$arg1, tid()] = gettimeofday_ns(); } -probe process.mark("timing_ns_end") { +probe process(@1).mark("timing_ns_end") { tag = $arg1 t = gettimeofday_ns(); if (s = starts[tag, tid()]) { @@ -27,6 +32,7 @@ probe end { } printf("%15s %10s %10s\n", "tag", "count", "avg(ns)"); foreach ([tag+] in times_per_tag) { - printf("%15X %10d %10d\n", tag, @count(times_per_tag[tag]), @avg(times_per_tag[tag])); + printf("%15X %10d %10d\n", tag, @count(times_per_tag[tag]), + @avg(times_per_tag[tag])); } } -- cgit v1.2.3 From b220c1d21342fb995881c39efb6b4b8c55e89e6b Mon Sep 17 00:00:00 2001 From: David Garcia Quintas Date: Wed, 29 Apr 2015 14:26:08 -0700 Subject: Added missing globals declaration to STAP script. --- test/core/profiling/mark_timings.stp | 2 ++ 1 file changed, 2 insertions(+) (limited to 'test/core') diff --git a/test/core/profiling/mark_timings.stp b/test/core/profiling/mark_timings.stp index 45d5e0d217..0c0a417faf 100644 --- a/test/core/profiling/mark_timings.stp +++ b/test/core/profiling/mark_timings.stp @@ -5,6 +5,8 @@ * For dinamically built ones, point to the location of the libgprc.so being * used. */ +global starts, times, times_per_tag + probe process(@1).mark("timing_ns_begin") { starts[$arg1, tid()] = gettimeofday_ns(); } -- cgit v1.2.3 From 0e67cb2c3958097786d2391c6da86657ceedc4e4 Mon Sep 17 00:00:00 2001 From: David Garcia Quintas Date: Thu, 30 Apr 2015 09:03:00 -0700 Subject: Fixed missing name change from timers.h refactor. --- test/core/profiling/timers_test.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'test/core') diff --git a/test/core/profiling/timers_test.c b/test/core/profiling/timers_test.c index 55e59c969e..12b08c115e 100644 --- a/test/core/profiling/timers_test.c +++ b/test/core/profiling/timers_test.c @@ -76,8 +76,8 @@ void test_log_events(int num_seqs) { int main(int argc, char **argv) { grpc_test_init(argc, argv); - grpc_timers_log_global_init(); + grpc_timers_global_init(); test_log_events(1000000); - grpc_timers_log_global_destroy(); + grpc_timers_global_destroy(); return 0; } -- cgit v1.2.3