lkml.org 
[lkml]   [2012]   [Aug]   [6]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 5/7] perf hists: Use hpp_functions->width to calculate the column widths
    Date
    From: Namhyung Kim <namhyung.kim@lge.com>

    Signed-off-by: Namhyung Kim <namhyung@kernel.org>
    ---
    tools/perf/ui/hist.c | 27 +++++++++++++++++++++++++++
    tools/perf/util/hist.c | 33 ---------------------------------
    2 files changed, 27 insertions(+), 33 deletions(-)

    diff --git a/tools/perf/ui/hist.c b/tools/perf/ui/hist.c
    index 01724a9905c0..1293546cf7f2 100644
    --- a/tools/perf/ui/hist.c
    +++ b/tools/perf/ui/hist.c
    @@ -835,6 +835,33 @@ out:
    return ret;
    }

    +/*
    + * See hists__fprintf to match the column widths
    + */
    +unsigned int hists__sort_list_width(struct hists *hists)
    +{
    + struct sort_entry *se;
    + int i, ret = 0;
    +
    + for (i = 0; i < PERF_HPP__MAX_INDEX; i++) {
    + if (!hpp_functions[i].cond)
    + continue;
    + if (i)
    + ret += 2;
    +
    + ret += hpp_functions[i].width(NULL);
    + }
    +
    + list_for_each_entry(se, &hist_entry__sort_list, list)
    + if (!se->elide)
    + ret += 2 + hists__col_len(hists, se->se_width_idx);
    +
    + if (verbose) /* Addr + origin */
    + ret += 3 + BITS_PER_LONG / 4;
    +
    + return ret;
    +}
    +
    size_t hists__fprintf_nr_events(struct hists *hists, FILE *fp)
    {
    int i;
    diff --git a/tools/perf/util/hist.c b/tools/perf/util/hist.c
    index b1817f15bb87..0ba65ad07cd1 100644
    --- a/tools/perf/util/hist.c
    +++ b/tools/perf/util/hist.c
    @@ -563,39 +563,6 @@ void hists__output_resort_threaded(struct hists *hists)
    return __hists__output_resort(hists, true);
    }

    -/*
    - * See hists__fprintf to match the column widths
    - */
    -unsigned int hists__sort_list_width(struct hists *hists)
    -{
    - struct sort_entry *se;
    - int ret = 9; /* total % */
    -
    - if (symbol_conf.show_cpu_utilization) {
    - ret += 7; /* count_sys % */
    - ret += 6; /* count_us % */
    - if (perf_guest) {
    - ret += 13; /* count_guest_sys % */
    - ret += 12; /* count_guest_us % */
    - }
    - }
    -
    - if (symbol_conf.show_nr_samples)
    - ret += 11;
    -
    - if (symbol_conf.show_total_period)
    - ret += 13;
    -
    - list_for_each_entry(se, &hist_entry__sort_list, list)
    - if (!se->elide)
    - ret += 2 + hists__col_len(hists, se->se_width_idx);
    -
    - if (verbose) /* Addr + origin */
    - ret += 3 + BITS_PER_LONG / 4;
    -
    - return ret;
    -}
    -
    static void hists__remove_entry_filter(struct hists *hists, struct hist_entry *h,
    enum hist_filter filter)
    {
    --
    1.7.11.2


    \
     
     \ /
      Last update: 2012-08-06 11:42    [W:2.180 / U:0.184 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site