lkml.org 
[lkml]   [2015]   [Jan]   [28]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    Date
    From
    Subject[tip:perf/core] perf report: Get rid of report__inc_stat()
    Commit-ID:  590cd344e2099c7b040b29d3a711b4c26358def5
    Gitweb: http://git.kernel.org/tip/590cd344e2099c7b040b29d3a711b4c26358def5
    Author: Namhyung Kim <namhyung@kernel.org>
    AuthorDate: Mon, 22 Dec 2014 13:44:09 +0900
    Committer: Arnaldo Carvalho de Melo <acme@redhat.com>
    CommitDate: Wed, 21 Jan 2015 13:24:32 -0300

    perf report: Get rid of report__inc_stat()

    The report__inc_stat() function collects the number of hist entries in
    the session in order to calculate the max size of the progess bar.

    It'd be better if it does it during the addition of hist entries so that
    it can be used by other places too.

    Signed-off-by: Namhyung Kim <namhyung@kernel.org>
    Cc: Adrian Hunter <adrian.hunter@intel.com>
    Cc: David Ahern <dsahern@gmail.com>
    Cc: Ingo Molnar <mingo@kernel.org>
    Cc: Jiri Olsa <jolsa@redhat.com>
    Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
    Cc: Stephane Eranian <eranian@google.com>
    Link: http://lkml.kernel.org/r/1419223455-4362-2-git-send-email-namhyung@kernel.org
    Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
    ---
    tools/perf/builtin-report.c | 16 +++-------------
    tools/perf/util/hist.c | 2 ++
    2 files changed, 5 insertions(+), 13 deletions(-)

    diff --git a/tools/perf/builtin-report.c b/tools/perf/builtin-report.c
    index 072ae8a..2f91094 100644
    --- a/tools/perf/builtin-report.c
    +++ b/tools/perf/builtin-report.c
    @@ -86,17 +86,6 @@ static int report__config(const char *var, const char *value, void *cb)
    return perf_default_config(var, value, cb);
    }

    -static void report__inc_stats(struct report *rep, struct hist_entry *he)
    -{
    - /*
    - * The @he is either of a newly created one or an existing one
    - * merging current sample. We only want to count a new one so
    - * checking ->nr_events being 1.
    - */
    - if (he->stat.nr_events == 1)
    - rep->nr_entries++;
    -}
    -
    static int hist_iter__report_callback(struct hist_entry_iter *iter,
    struct addr_location *al, bool single,
    void *arg)
    @@ -108,8 +97,6 @@ static int hist_iter__report_callback(struct hist_entry_iter *iter,
    struct mem_info *mi;
    struct branch_info *bi;

    - report__inc_stats(rep, he);
    -
    if (!ui__has_annotation())
    return 0;

    @@ -499,6 +486,9 @@ static int __cmd_report(struct report *rep)

    report__warn_kptr_restrict(rep);

    + evlist__for_each(session->evlist, pos)
    + rep->nr_entries += evsel__hists(pos)->nr_entries;
    +
    if (use_browser == 0) {
    if (verbose > 3)
    perf_session__fprintf(session, stdout);
    diff --git a/tools/perf/util/hist.c b/tools/perf/util/hist.c
    index 038483a..e17163f 100644
    --- a/tools/perf/util/hist.c
    +++ b/tools/perf/util/hist.c
    @@ -429,6 +429,8 @@ static struct hist_entry *add_hist_entry(struct hists *hists,
    if (!he)
    return NULL;

    + hists->nr_entries++;
    +
    rb_link_node(&he->rb_node_in, parent, p);
    rb_insert_color(&he->rb_node_in, hists->entries_in);
    out:

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