lkml.org 
[lkml]   [2021]   [Jan]   [29]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH v2 1/2] perf util: Change intlist int to unsigned long
    Date
    This is to let intlist support address.

    One potential problem is it can't support negative number. But
    so far, there is no such kind of use case.

    Signed-off-by: Jin Yao <yao.jin@linux.intel.com>
    ---
    v2:
    New in v2.

    tools/perf/util/intlist.c | 27 ++++++++++++++++-----------
    tools/perf/util/intlist.h | 10 +++++-----
    tools/perf/util/probe-event.c | 2 +-
    3 files changed, 22 insertions(+), 17 deletions(-)

    diff --git a/tools/perf/util/intlist.c b/tools/perf/util/intlist.c
    index 84e5304e151a..934092199f89 100644
    --- a/tools/perf/util/intlist.c
    +++ b/tools/perf/util/intlist.c
    @@ -13,7 +13,7 @@
    static struct rb_node *intlist__node_new(struct rblist *rblist __maybe_unused,
    const void *entry)
    {
    - int i = (int)((long)entry);
    + unsigned long i = (unsigned long)entry;
    struct rb_node *rc = NULL;
    struct int_node *node = malloc(sizeof(*node));

    @@ -41,15 +41,20 @@ static void intlist__node_delete(struct rblist *rblist __maybe_unused,

    static int intlist__node_cmp(struct rb_node *rb_node, const void *entry)
    {
    - int i = (int)((long)entry);
    + unsigned long i = (unsigned long)entry;
    struct int_node *node = container_of(rb_node, struct int_node, rb_node);

    - return node->i - i;
    + if (node->i > i)
    + return 1;
    + else if (node->i < i)
    + return -1;
    +
    + return 0;
    }

    -int intlist__add(struct intlist *ilist, int i)
    +int intlist__add(struct intlist *ilist, unsigned long i)
    {
    - return rblist__add_node(&ilist->rblist, (void *)((long)i));
    + return rblist__add_node(&ilist->rblist, (void *)i);
    }

    void intlist__remove(struct intlist *ilist, struct int_node *node)
    @@ -58,7 +63,7 @@ void intlist__remove(struct intlist *ilist, struct int_node *node)
    }

    static struct int_node *__intlist__findnew(struct intlist *ilist,
    - int i, bool create)
    + unsigned long i, bool create)
    {
    struct int_node *node = NULL;
    struct rb_node *rb_node;
    @@ -67,9 +72,9 @@ static struct int_node *__intlist__findnew(struct intlist *ilist,
    return NULL;

    if (create)
    - rb_node = rblist__findnew(&ilist->rblist, (void *)((long)i));
    + rb_node = rblist__findnew(&ilist->rblist, (void *)i);
    else
    - rb_node = rblist__find(&ilist->rblist, (void *)((long)i));
    + rb_node = rblist__find(&ilist->rblist, (void *)i);

    if (rb_node)
    node = container_of(rb_node, struct int_node, rb_node);
    @@ -77,12 +82,12 @@ static struct int_node *__intlist__findnew(struct intlist *ilist,
    return node;
    }

    -struct int_node *intlist__find(struct intlist *ilist, int i)
    +struct int_node *intlist__find(struct intlist *ilist, unsigned long i)
    {
    return __intlist__findnew(ilist, i, false);
    }

    -struct int_node *intlist__findnew(struct intlist *ilist, int i)
    +struct int_node *intlist__findnew(struct intlist *ilist, unsigned long i)
    {
    return __intlist__findnew(ilist, i, true);
    }
    @@ -93,7 +98,7 @@ static int intlist__parse_list(struct intlist *ilist, const char *s)
    int err;

    do {
    - long value = strtol(s, &sep, 10);
    + unsigned long value = strtol(s, &sep, 10);
    err = -EINVAL;
    if (*sep != ',' && *sep != '\0')
    break;
    diff --git a/tools/perf/util/intlist.h b/tools/perf/util/intlist.h
    index 5c19ee001299..e336b174d0c7 100644
    --- a/tools/perf/util/intlist.h
    +++ b/tools/perf/util/intlist.h
    @@ -9,7 +9,7 @@

    struct int_node {
    struct rb_node rb_node;
    - int i;
    + unsigned long i;
    void *priv;
    };

    @@ -21,13 +21,13 @@ struct intlist *intlist__new(const char *slist);
    void intlist__delete(struct intlist *ilist);

    void intlist__remove(struct intlist *ilist, struct int_node *in);
    -int intlist__add(struct intlist *ilist, int i);
    +int intlist__add(struct intlist *ilist, unsigned long i);

    struct int_node *intlist__entry(const struct intlist *ilist, unsigned int idx);
    -struct int_node *intlist__find(struct intlist *ilist, int i);
    -struct int_node *intlist__findnew(struct intlist *ilist, int i);
    +struct int_node *intlist__find(struct intlist *ilist, unsigned long i);
    +struct int_node *intlist__findnew(struct intlist *ilist, unsigned long i);

    -static inline bool intlist__has_entry(struct intlist *ilist, int i)
    +static inline bool intlist__has_entry(struct intlist *ilist, unsigned long i)
    {
    return intlist__find(ilist, i) != NULL;
    }
    diff --git a/tools/perf/util/probe-event.c b/tools/perf/util/probe-event.c
    index 8eae2afff71a..137f19c5b686 100644
    --- a/tools/perf/util/probe-event.c
    +++ b/tools/perf/util/probe-event.c
    @@ -1074,7 +1074,7 @@ static int __show_line_range(struct line_range *lr, const char *module,
    }

    intlist__for_each_entry(ln, lr->line_list) {
    - for (; ln->i > l; l++) {
    + for (; ln->i > (unsigned long)l; l++) {
    ret = show_one_line(fp, l - lr->offset);
    if (ret < 0)
    goto end;
    --
    2.17.1
    \
     
     \ /
      Last update: 2021-01-29 08:12    [W:2.694 / U:0.144 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site