lkml.org 
[lkml]   [2010]   [Feb]   [22]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH -tip v3 3/9] perf probe: Fix bugs in line range finder
    Date
    Fix find_line_range_by_line() to init line_list and remove
    misconseptional found marking which should be done when
    real lines are found (if there is no lines probe-able,
    find_line_range() should return 0).

    Signed-off-by: Masami Hiramatsu <mhiramat@redhat.com>
    Cc: Ingo Molnar <mingo@elte.hu>
    Cc: Frederic Weisbecker <fweisbec@gmail.com>
    Cc: Arnaldo Carvalho de Melo <acme@redhat.com>
    Cc: Peter Zijlstra <peterz@infradead.org>
    Cc: Paul Mackerras <paulus@samba.org>
    Cc: Mike Galbraith <efault@gmx.de>
    Cc: K.Prasad <prasad@linux.vnet.ibm.com>
    ---

    tools/perf/util/probe-finder.c | 3 +--
    1 files changed, 1 insertions(+), 2 deletions(-)

    diff --git a/tools/perf/util/probe-finder.c b/tools/perf/util/probe-finder.c
    index 1b2124d..3e10dbe 100644
    --- a/tools/perf/util/probe-finder.c
    +++ b/tools/perf/util/probe-finder.c
    @@ -788,6 +788,7 @@ static void find_line_range_by_line(struct line_finder *lf)
    Dwarf_Addr addr;
    int ret;

    + INIT_LIST_HEAD(&lf->lr->line_list);
    ret = dwarf_srclines(lf->cu_die, &lines, &cnt, &__dw_error);
    DIE_IF(ret != DW_DLV_OK);

    @@ -848,8 +849,6 @@ static int linefunc_callback(struct die_link *dlink, void *data)
    lr->start = lf->lno_s;
    lr->end = lf->lno_e;
    find_line_range_by_line(lf);
    - /* If we find a target function, this should be end. */
    - lf->found = 1;
    return 1;
    }
    return 0;

    --
    Masami Hiramatsu

    Software Engineer
    Hitachi Computer Products (America), Inc.
    Software Solutions Division

    e-mail: mhiramat@redhat.com


    \
     
     \ /
      Last update: 2010-02-23 03:09    [W:0.020 / U:2.380 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site