[PATCH 4/5] tracing/workqueue: fix list header and function name format

From: Frederic Weisbecker
Date: Mon Jun 01 2009 - 13:18:40 EST


From: Zhaolei <zhaolei@xxxxxxxxxxxxxx>

Fix list header of workqueue trace,
and use %pf to print the function name only(ignore the offsets).

Before patch:
# CPU INSERTED EXECUTED MAX us AVG us TASKNAME:PID
# | | | | ` | -WORKFUNC
# | | | | | |
0 1 1 1053 1053 `-flush_async_commits+0x0/0x90
...

After patch:
# CPU INSERTED EXECUTED MAX us AVG us TASKNAME:PID
# | | | | | `-WORKFUNC
# | | | | | |
0 1 1 1155 1155 `-flush_async_commits
...

[ Impact: fix and clarify workqueue stats output ]

Signed-off-by: Zhao Lei <zhaolei@xxxxxxxxxxxxxx>
Signed-off-by: Frederic Weisbecker <fweisbec@xxxxxxxxx>
---
kernel/trace/trace_workqueue.c | 14 ++++++++------
1 files changed, 8 insertions(+), 6 deletions(-)

diff --git a/kernel/trace/trace_workqueue.c b/kernel/trace/trace_workqueue.c
index fa70ea2..cb6429e 100644
--- a/kernel/trace/trace_workqueue.c
+++ b/kernel/trace/trace_workqueue.c
@@ -402,7 +402,7 @@ static int workqueue_stat_show(struct seq_file *s, void *p)
avg_time = 0;
}

- seq_printf(s, " %3d %6d %6u %6llu %6llu %c-%pF\n",
+ seq_printf(s, " %3d %6d %6u %6llu %6llu %c-%pf\n",
cws->cpu,
wfstat->inserted,
wfstat->executed,
@@ -450,11 +450,13 @@ static void workqueue_stat_file_open(void)
/**/
static int workqueue_stat_headers(struct seq_file *s)
{
- seq_printf(s, "# CPU INSERTED EXECUTED MAX us AVG us"
- " TASKNAME:PID\n");
- seq_printf(s, "# | | | | ` |"
- " -WORKFUNC\n");
- seq_printf(s, "# | | | | | |\n");
+ seq_printf(s,
+ "# CPU INSERTED EXECUTED MAX us AVG us TASKNAME:PID\n");
+ seq_printf(s,
+ "# | | | | | `-WORKFUNC\n");
+ seq_printf(s,
+ "# | | | | | |\n");
+
return 0;
}

--
1.6.2.3

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/