2012-11-14 07:46:34

by Arnaldo Carvalho de Melo

[permalink] [raw]
Subject: [tip:perf/core] tools lib traceevent: Avoid comparisions between signed/unsigned

Commit-ID: 8a38cce4c900307e947eda774f844caf5b9a7083
Gitweb: http://git.kernel.org/tip/8a38cce4c900307e947eda774f844caf5b9a7083
Author: Arnaldo Carvalho de Melo <[email protected]>
AuthorDate: Fri, 9 Nov 2012 15:32:00 -0300
Committer: Arnaldo Carvalho de Melo <[email protected]>
CommitDate: Fri, 9 Nov 2012 17:34:58 -0300

tools lib traceevent: Avoid comparisions between signed/unsigned

Fixing this warning-as-error on f14 32-bit:

tools/lib/traceevent/event-parse.c:5564:17: error: comparison between signed and unsigned integer expressions
tools/lib/traceevent/event-parse.c:5586:17: error: comparison between signed and unsigned integer expressions

Cc: David Ahern <[email protected]>
Cc: Frederic Weisbecker <[email protected]>
Cc: Mike Galbraith <[email protected]>
Cc: Paul Mackerras <[email protected]>
Cc: Peter Zijlstra <[email protected]>
Cc: Stephane Eranian <[email protected]>
Cc: Steven Rostedt <[email protected]>
Link: http://lkml.kernel.org/n/[email protected]
Signed-off-by: Arnaldo Carvalho de Melo <[email protected]>
---
tools/lib/traceevent/event-parse.c | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/tools/lib/traceevent/event-parse.c b/tools/lib/traceevent/event-parse.c
index 0dcc18c..6b647c1 100644
--- a/tools/lib/traceevent/event-parse.c
+++ b/tools/lib/traceevent/event-parse.c
@@ -5561,7 +5561,7 @@ void pevent_free(struct pevent *pevent)
}

if (pevent->func_map) {
- for (i = 0; i < pevent->func_count; i++) {
+ for (i = 0; i < (int)pevent->func_count; i++) {
free(pevent->func_map[i].func);
free(pevent->func_map[i].mod);
}
@@ -5583,7 +5583,7 @@ void pevent_free(struct pevent *pevent)
}

if (pevent->printk_map) {
- for (i = 0; i < pevent->printk_count; i++)
+ for (i = 0; i < (int)pevent->printk_count; i++)
free(pevent->printk_map[i].printk);
free(pevent->printk_map);
}