From: "Steven Rostedt (Red Hat)" <[email protected]>

Instead of using 1000000, define a USECS_PER_SEC macro and use that instead.

Signed-off-by: Steven Rostedt <[email protected]>
---
 tools/lib/traceevent/event-parse.c | 4 ++--
 tools/lib/traceevent/event-parse.h | 2 ++
 2 files changed, 4 insertions(+), 2 deletions(-)

diff --git a/tools/lib/traceevent/event-parse.c 
b/tools/lib/traceevent/event-parse.c
index a6f1ce779e05..f49b266b3978 100644
--- a/tools/lib/traceevent/event-parse.c
+++ b/tools/lib/traceevent/event-parse.c
@@ -5388,8 +5388,8 @@ void pevent_print_event(struct pevent *pevent, struct 
trace_seq *s,
                } else {
                        usecs = (nsecs + 500) / NSECS_PER_USEC;
                        /* To avoid usecs larger than 1 sec */
-                       if (usecs >= 1000000) {
-                               usecs -= 1000000;
+                       if (usecs >= USECS_PER_SEC) {
+                               usecs -= USECS_PER_SEC;
                                secs++;
                        }
                        p = 6;
diff --git a/tools/lib/traceevent/event-parse.h 
b/tools/lib/traceevent/event-parse.h
index 706d9bc24066..4373c5190076 100644
--- a/tools/lib/traceevent/event-parse.h
+++ b/tools/lib/traceevent/event-parse.h
@@ -175,6 +175,8 @@ struct pevent_plugin_option {
 #define NSECS_PER_SEC          1000000000ULL
 #define NSECS_PER_USEC         1000ULL
 
+#define USECS_PER_SEC          1000000ULL
+
 enum format_flags {
        FIELD_IS_ARRAY          = 1,
        FIELD_IS_POINTER        = 2,
-- 
2.6.4


Reply via email to