lkml.org 
[lkml]   [2013]   [Jan]   [24]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 56/74] tools lib traceevent: Fix warning on '>=' operator
    Date
    From: Namhyung Kim <namhyung.kim@lge.com>

    Although the '>=' (and '<=') operator is handled properly in
    libtraceevent, it emitted following spurious warnings on perf test:

    $ perf test
    5: parse events tests :
    ...
    Warning: unknown op '>='
    Warning: unknown op '>='
    Warning: unknown op '>='
    Warning: unknown op '>='
    Warning: unknown op '>='
    Warning: unknown op '>='
    ...

    Add the operator to the checks.

    Signed-off-by: Namhyung Kim <namhyung@kernel.org>
    Acked-by: Steven Rostedt <rostedt@goodmis.org>
    Cc: Frederic Weisbecker <fweisbec@gmail.com>
    Cc: Ingo Molnar <mingo@kernel.org>
    Cc: Jiri Olsa <jolsa@redhat.com>
    Cc: Paul Mackerras <paulus@samba.org>
    Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
    Cc: Steven Rostedt <rostedt@goodmis.org>
    Link: http://lkml.kernel.org/r/1358236939-17393-1-git-send-email-namhyung@kernel.org
    Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
    ---
    tools/lib/traceevent/event-parse.c | 2 ++
    1 file changed, 2 insertions(+)

    diff --git a/tools/lib/traceevent/event-parse.c b/tools/lib/traceevent/event-parse.c
    index dc42e55..f504619 100644
    --- a/tools/lib/traceevent/event-parse.c
    +++ b/tools/lib/traceevent/event-parse.c
    @@ -1784,6 +1784,8 @@ process_op(struct event_format *event, struct print_arg *arg, char **tok)
    strcmp(token, "/") == 0 ||
    strcmp(token, "<") == 0 ||
    strcmp(token, ">") == 0 ||
    + strcmp(token, "<=") == 0 ||
    + strcmp(token, ">=") == 0 ||
    strcmp(token, "==") == 0 ||
    strcmp(token, "!=") == 0) {

    --
    1.8.1.1.361.gec3ae6e


    \
     
     \ /
      Last update: 2013-01-24 22:01    [W:3.535 / U:0.008 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site