perf timechart: Use NSEC_PER_U?SEC

Following kernel practices, using linux/time64.h

Cc: Adrian Hunter <adrian.hunter@intel.com>
Cc: Arjan van de Ven <arjan@linux.intel.com>
Cc: David Ahern <dsahern@gmail.com>
Cc: Jiri Olsa <jolsa@kernel.org>
Cc: Namhyung Kim <namhyung@kernel.org>
Cc: Stanislav Fomichev <stfomichev@yandex-team.ru>
Cc: Steven Rostedt <rostedt@goodmis.org>
Cc: Wang Nan <wangnan0@huawei.com>
Link: http://lkml.kernel.org/n/tip-5l1md8lsdhfnrlsqyejzo9w2@git.kernel.orgSigned-off-by: default avatarArnaldo Carvalho de Melo <acme@redhat.com>
parent 4fc76e49
...@@ -24,6 +24,7 @@ ...@@ -24,6 +24,7 @@
#include "util/evlist.h" #include "util/evlist.h"
#include "util/evsel.h" #include "util/evsel.h"
#include <linux/rbtree.h> #include <linux/rbtree.h>
#include <linux/time64.h>
#include "util/symbol.h" #include "util/symbol.h"
#include "util/callchain.h" #include "util/callchain.h"
#include "util/strlist.h" #include "util/strlist.h"
...@@ -1288,9 +1289,9 @@ static void draw_process_bars(struct timechart *tchart) ...@@ -1288,9 +1289,9 @@ static void draw_process_bars(struct timechart *tchart)
if (c->comm) { if (c->comm) {
char comm[256]; char comm[256];
if (c->total_time > 5000000000) /* 5 seconds */ if (c->total_time > 5000000000) /* 5 seconds */
sprintf(comm, "%s:%i (%2.2fs)", c->comm, p->pid, c->total_time / 1000000000.0); sprintf(comm, "%s:%i (%2.2fs)", c->comm, p->pid, c->total_time / (double)NSEC_PER_SEC);
else else
sprintf(comm, "%s:%i (%3.1fms)", c->comm, p->pid, c->total_time / 1000000.0); sprintf(comm, "%s:%i (%3.1fms)", c->comm, p->pid, c->total_time / (double)NSEC_PER_MSEC);
svg_text(Y, c->start_time, comm); svg_text(Y, c->start_time, comm);
} }
...@@ -1637,7 +1638,7 @@ static int __cmd_timechart(struct timechart *tchart, const char *output_name) ...@@ -1637,7 +1638,7 @@ static int __cmd_timechart(struct timechart *tchart, const char *output_name)
write_svg_file(tchart, output_name); write_svg_file(tchart, output_name);
pr_info("Written %2.1f seconds of trace to %s.\n", pr_info("Written %2.1f seconds of trace to %s.\n",
(tchart->last_time - tchart->first_time) / 1000000000.0, output_name); (tchart->last_time - tchart->first_time) / (double)NSEC_PER_SEC, output_name);
out_delete: out_delete:
perf_session__delete(session); perf_session__delete(session);
return ret; return ret;
...@@ -1901,10 +1902,10 @@ parse_time(const struct option *opt, const char *arg, int __maybe_unused unset) ...@@ -1901,10 +1902,10 @@ parse_time(const struct option *opt, const char *arg, int __maybe_unused unset)
if (sscanf(arg, "%" PRIu64 "%cs", value, &unit) > 0) { if (sscanf(arg, "%" PRIu64 "%cs", value, &unit) > 0) {
switch (unit) { switch (unit) {
case 'm': case 'm':
*value *= 1000000; *value *= NSEC_PER_MSEC;
break; break;
case 'u': case 'u':
*value *= 1000; *value *= NSEC_PER_USEC;
break; break;
case 'n': case 'n':
break; break;
...@@ -1928,7 +1929,7 @@ int cmd_timechart(int argc, const char **argv, ...@@ -1928,7 +1929,7 @@ int cmd_timechart(int argc, const char **argv,
.ordered_events = true, .ordered_events = true,
}, },
.proc_num = 15, .proc_num = 15,
.min_time = 1000000, .min_time = NSEC_PER_MSEC,
.merge_dist = 1000, .merge_dist = 1000,
}; };
const char *output_name = "output.svg"; const char *output_name = "output.svg";
......
...@@ -18,6 +18,7 @@ ...@@ -18,6 +18,7 @@
#include <unistd.h> #include <unistd.h>
#include <string.h> #include <string.h>
#include <linux/bitmap.h> #include <linux/bitmap.h>
#include <linux/time64.h>
#include "perf.h" #include "perf.h"
#include "svghelper.h" #include "svghelper.h"
...@@ -274,14 +275,14 @@ static char *time_to_string(u64 duration) ...@@ -274,14 +275,14 @@ static char *time_to_string(u64 duration)
text[0] = 0; text[0] = 0;
if (duration < 1000) /* less than 1 usec */ if (duration < NSEC_PER_USEC) /* less than 1 usec */
return text; return text;
if (duration < 1000 * 1000) { /* less than 1 msec */ if (duration < NSEC_PER_MSEC) { /* less than 1 msec */
sprintf(text, "%.1f us", duration / 1000.0); sprintf(text, "%.1f us", duration / (double)NSEC_PER_USEC);
return text; return text;
} }
sprintf(text, "%.1f ms", duration / 1000.0 / 1000); sprintf(text, "%.1f ms", duration / (double)NSEC_PER_MSEC);
return text; return text;
} }
...@@ -297,7 +298,7 @@ void svg_waiting(int Yslot, int cpu, u64 start, u64 end, const char *backtrace) ...@@ -297,7 +298,7 @@ void svg_waiting(int Yslot, int cpu, u64 start, u64 end, const char *backtrace)
style = "waiting"; style = "waiting";
if (end-start > 10 * 1000000) /* 10 msec */ if (end-start > 10 * NSEC_PER_MSEC) /* 10 msec */
style = "WAITING"; style = "WAITING";
text = time_to_string(end-start); text = time_to_string(end-start);
......
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