Commit ea92ed5a authored by Ingo Molnar's avatar Ingo Molnar

perf sched: Add runtime stats

Extend the latency tracking structure with scheduling atom
runtime info - and sum it up during per task display.

(Also clean up a few details.)

Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
Cc: Mike Galbraith <efault@gmx.de>
Cc: Paul Mackerras <paulus@samba.org>
Cc: Arnaldo Carvalho de Melo <acme@redhat.com>
Cc: Frederic Weisbecker <fweisbec@gmail.com>
LKML-Reference: <new-submission>
Signed-off-by: default avatarIngo Molnar <mingo@elte.hu>
parent d9340c1d
...@@ -877,6 +877,7 @@ struct lat_snapshot { ...@@ -877,6 +877,7 @@ struct lat_snapshot {
enum thread_state state; enum thread_state state;
u64 wake_up_time; u64 wake_up_time;
u64 sched_in_time; u64 sched_in_time;
u64 runtime;
}; };
struct thread_latency { struct thread_latency {
...@@ -951,6 +952,7 @@ latency_fork_event(struct trace_fork_event *fork_event __used, ...@@ -951,6 +952,7 @@ latency_fork_event(struct trace_fork_event *fork_event __used,
/* should insert the newcomer */ /* should insert the newcomer */
} }
__used
static char sched_out_state(struct trace_switch_event *switch_event) static char sched_out_state(struct trace_switch_event *switch_event)
{ {
const char *str = TASK_STATE_TO_CHAR_STR; const char *str = TASK_STATE_TO_CHAR_STR;
...@@ -960,17 +962,15 @@ static char sched_out_state(struct trace_switch_event *switch_event) ...@@ -960,17 +962,15 @@ static char sched_out_state(struct trace_switch_event *switch_event)
static void static void
lat_sched_out(struct thread_latency *lat, lat_sched_out(struct thread_latency *lat,
struct trace_switch_event *switch_event) struct trace_switch_event *switch_event __used, u64 delta)
{ {
struct lat_snapshot *snapshot; struct lat_snapshot *snapshot;
if (sched_out_state(switch_event) == 'R')
return;
snapshot = calloc(sizeof(*snapshot), 1); snapshot = calloc(sizeof(*snapshot), 1);
if (!snapshot) if (!snapshot)
die("Non memory"); die("Non memory");
snapshot->runtime = delta;
list_add_tail(&snapshot->list, &lat->snapshot_list); list_add_tail(&snapshot->list, &lat->snapshot_list);
} }
...@@ -997,16 +997,31 @@ lat_sched_in(struct thread_latency *lat, u64 timestamp) ...@@ -997,16 +997,31 @@ lat_sched_in(struct thread_latency *lat, u64 timestamp)
snapshot->sched_in_time = timestamp; snapshot->sched_in_time = timestamp;
} }
static void static void
latency_switch_event(struct trace_switch_event *switch_event, latency_switch_event(struct trace_switch_event *switch_event,
struct event *event __used, struct event *event __used,
int cpu __used, int cpu,
u64 timestamp, u64 timestamp,
struct thread *thread __used) struct thread *thread __used)
{ {
struct thread_latency *out_lat, *in_lat; struct thread_latency *out_lat, *in_lat;
struct thread *sched_out, *sched_in; struct thread *sched_out, *sched_in;
u64 timestamp0;
s64 delta;
if (cpu >= MAX_CPUS || cpu < 0)
return;
timestamp0 = cpu_last_switched[cpu];
cpu_last_switched[cpu] = timestamp;
if (timestamp0)
delta = timestamp - timestamp0;
else
delta = 0;
if (delta < 0)
die("hm, delta: %Ld < 0 ?\n", delta);
sched_out = threads__findnew(switch_event->prev_pid, &threads, &last_match); sched_out = threads__findnew(switch_event->prev_pid, &threads, &last_match);
sched_in = threads__findnew(switch_event->next_pid, &threads, &last_match); sched_in = threads__findnew(switch_event->next_pid, &threads, &last_match);
...@@ -1028,7 +1043,7 @@ latency_switch_event(struct trace_switch_event *switch_event, ...@@ -1028,7 +1043,7 @@ latency_switch_event(struct trace_switch_event *switch_event,
} }
lat_sched_in(in_lat, timestamp); lat_sched_in(in_lat, timestamp);
lat_sched_out(out_lat, switch_event); lat_sched_out(out_lat, switch_event, delta);
} }
static void static void
...@@ -1080,8 +1095,11 @@ static void output_lat_thread(struct thread_latency *lat) ...@@ -1080,8 +1095,11 @@ static void output_lat_thread(struct thread_latency *lat)
int ret; int ret;
u64 max = 0, avg; u64 max = 0, avg;
u64 total = 0, delta; u64 total = 0, delta;
u64 total_runtime = 0;
list_for_each_entry(shot, &lat->snapshot_list, list) { list_for_each_entry(shot, &lat->snapshot_list, list) {
total_runtime += shot->runtime;
if (shot->state != THREAD_SCHED_IN) if (shot->state != THREAD_SCHED_IN)
continue; continue;
...@@ -1104,7 +1122,7 @@ static void output_lat_thread(struct thread_latency *lat) ...@@ -1104,7 +1122,7 @@ static void output_lat_thread(struct thread_latency *lat)
avg = total / count; avg = total / count;
printf("|%9.3f ms |%9d | avg:%9.3f ms | max:%9.3f ms |\n", printf("|%9.3f ms |%9d | avg:%9.3f ms | max:%9.3f ms |\n",
0.0, count, (double)avg/1e9, (double)max/1e9); (double)total_runtime/1e9, count, (double)avg/1e9, (double)max/1e9);
} }
static void __cmd_lat(void) static void __cmd_lat(void)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment