17 Jul, 2014

1 commit

  • And use verbose as an global object in following functions:

    __map_groups__fprintf_maps
    __map_groups__fprintf_removed_maps
    map_groups__fprintf_maps
    map_groups__fprintf

    Also making map_groups__fprintf_maps static.

    Signed-off-by: Jiri Olsa
    Cc: Corey Ashford
    Cc: David Ahern
    Cc: Frederic Weisbecker
    Cc: Ingo Molnar
    Cc: Namhyung Kim
    Cc: Paul Mackerras
    Cc: Peter Zijlstra
    Link: http://lkml.kernel.org/r/1405374411-29012-2-git-send-email-jolsa@kernel.org
    Signed-off-by: Arnaldo Carvalho de Melo

    Jiri Olsa
     

08 Jul, 2014

1 commit

  • Adding ui.show-headers config file option to define if the histogram
    entries headers will start visible or not.

    Currently columns headers are displayed by default, following
    lines in ~/.perfconfig file will disable that:

    [ui]
    show-headers = false

    Signed-off-by: Jiri Olsa
    Cc: Corey Ashford
    Cc: David Ahern
    Cc: Frederic Weisbecker
    Cc: Ingo Molnar
    Cc: Namhyung Kim
    Cc: Paul Mackerras
    Cc: Peter Zijlstra
    Link: http://lkml.kernel.org/r/1403886418-5556-4-git-send-email-jolsa@kernel.org
    [ renamed symbol_conf.show_headers to .show_hist_headers ]
    Signed-off-by: Arnaldo Carvalho de Melo

    Jiri Olsa
     

07 Jul, 2014

6 commits

  • Displaying columns header text whenever 'H' is pressed,
    and hiding it on on another press.

    Signed-off-by: Jiri Olsa
    Cc: Arnaldo Carvalho de Melo
    Cc: Corey Ashford
    Cc: David Ahern
    Cc: Frederic Weisbecker
    Cc: Ingo Molnar
    Cc: Namhyung Kim
    Cc: Paul Mackerras
    Cc: Peter Zijlstra
    Link: http://lkml.kernel.org/n/tip-w9pcqpum5erza2a05ysvollz@git.kernel.org
    Signed-off-by: Arnaldo Carvalho de Melo

    Jiri Olsa
     
  • Open up space to show a one-line header text whenever 'H' is pressed,
    hide it on another key press.

    Follow up patch will format this line from the set of headers used.

    Cc: Corey Ashford
    Cc: David Ahern
    Cc: Frederic Weisbecker
    Cc: Ingo Molnar
    Cc: Jiri Olsa
    Cc: Namhyung Kim
    Cc: Paul Mackerras
    Cc: Peter Zijlstra
    Link: http://lkml.kernel.org/n/tip-m894d6qk30h3qofw4k8neq4q@git.kernel.org
    Signed-off-by: Arnaldo Carvalho de Melo

    Arnaldo Carvalho de Melo
     
  • This requires some more work so that we can really just use the width of
    current entries when we want to partition the screen.

    Right now its just a prep patch so that we can have where to update
    ui_browser->rows when introducing the column headers line, that will be
    togglable, so we need to update it everytime we refresh the dimensions
    of the browser.

    Cc: Adrian Hunter
    Cc: David Ahern
    Cc: Don Zickus
    Cc: Frederic Weisbecker
    Cc: Jiri Olsa
    Cc: Mike Galbraith
    Cc: Namhyung Kim
    Cc: Paul Mackerras
    Cc: Peter Zijlstra
    Cc: Stephane Eranian
    Link: http://lkml.kernel.org/n/tip-ovk654rx525b4657y0mh6ku9@git.kernel.org
    Signed-off-by: Arnaldo Carvalho de Melo

    Arnaldo Carvalho de Melo
     
  • That will allow us to add a row offset to open up space for the column
    headers.

    Cc: Adrian Hunter
    Cc: David Ahern
    Cc: Don Zickus
    Cc: Frederic Weisbecker
    Cc: Jiri Olsa
    Cc: Mike Galbraith
    Cc: Namhyung Kim
    Cc: Paul Mackerras
    Cc: Peter Zijlstra
    Cc: Stephane Eranian
    Link: http://lkml.kernel.org/n/tip-otc3ployokfci5qi81o7jo22@git.kernel.org
    Signed-off-by: Arnaldo Carvalho de Melo

    Arnaldo Carvalho de Melo
     
  • Some browsers, like the hist_browser, may want to be notified everytime
    a refresh_dimensions is needed, so that it can reset ui_browser->rows,
    for instance, or do some other related reaction to screen resizings.

    Cc: Adrian Hunter
    Cc: David Ahern
    Cc: Don Zickus
    Cc: Frederic Weisbecker
    Cc: Jiri Olsa
    Cc: Mike Galbraith
    Cc: Namhyung Kim
    Cc: Paul Mackerras
    Cc: Peter Zijlstra
    Cc: Stephane Eranian
    Link: http://lkml.kernel.org/n/tip-ielvluuemzn30bneh0zk3twi@git.kernel.org
    Signed-off-by: Arnaldo Carvalho de Melo

    Arnaldo Carvalho de Melo
     
  • The ui_browser->height is about the whole browser "window", including
    any header, status lines or any other space needed for some "Yes", "No",
    etc buttons a descendent browser, like hist_browser, may have.

    Since the navigation is done mostly on the ui_browser methods, it needs
    to know how many rows are on the screen, while details about what other
    components are, say, if a header (that may be composed of multiple
    lines, etc) is present.

    Besides this we'll need to add a ui_browser->refresh_dimensions() hook
    so that browsers like hist_browser can update ->rows in response to
    screen resizes, this will come in a follow up patch.

    This patch just adds ->rows and updates it when updating ->height, keeps
    using ->height for the only other widget that can come with ui_browser,
    the scrollbar, that goes on using all the height on the rightmost column
    in the screen, using ->rows for the keyboard navigation needs.

    Cc: Adrian Hunter
    Cc: David Ahern
    Cc: Don Zickus
    Cc: Frederic Weisbecker
    Cc: Jiri Olsa
    Cc: Mike Galbraith
    Cc: Namhyung Kim
    Cc: Paul Mackerras
    Cc: Peter Zijlstra
    Cc: Stephane Eranian
    Link: http://lkml.kernel.org/n/tip-xexmwg1mv7u03j5imn66jdak@git.kernel.org
    Signed-off-by: Arnaldo Carvalho de Melo

    Arnaldo Carvalho de Melo
     

25 Jun, 2014

1 commit

  • …/acme/linux into perf/core

    Pull perf/core improvements and fixes from Arnaldo Carvalho de Melo:

    User visible changes:

    * Add --repeat global option to 'perf bench' to be used in benchmarks
    such as the existing 'futex' one, that was modified to use it instead
    of a local option. (Davidlohr Bueso)

    * Fix fd -> pathname resolution in 'trace', be it using /proc or
    a vfs_getname probe point. (Arnaldo Carvalho de Melo)

    * Add suggestion of how to set perf_event_paranoid sysctl, to help
    non-root users trying tools like 'trace' to get a working environment.
    (Arnaldo Carvalho de Melo)

    Fixes:

    * Fix memory leak in the 'sched-messaging' perf bench test. (Davidlohr Bueso)

    * The -o and -n 'perf bench mem' options are mutually exclusive, emit error
    when both are specified. (Davidlohr Bueso)

    * Fix scrollbar refresh row index in the ui browser, problem exposed now
    that headers will be added and will be allowed to be switched on/off.
    (Jiri Olsa)

    Cleanups:

    * Remove needless reassignments in 'trace' (Arnaldo Carvalho de Melo)

    * Cache the is_exit syscall test in 'trace) (Arnaldo Carvalho de Melo)

    * No need to reimplement err() in 'perf bench sched-messaging', drop barf().
    (Davidlohr Bueso).

    * Remove ev_name argument from perf_evsel__hists_browse, can be obtained
    from the other parameters. (Jiri Olsa)

    Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
    Signed-off-by: Ingo Molnar <mingo@kernel.org>

    Ingo Molnar
     

20 Jun, 2014

3 commits

  • When cumulative callchain mode is on, we could get samples with
    with no actual hits. This breaks the assumption of the annotation
    code, that each sample has annotation counts allocated and leads
    to segfault.

    Fixing this by additional checks for annotation stats.

    Acked-by: Namhyung Kim
    Acked-by: Arnaldo Carvalho de Melo
    Cc: Arnaldo Carvalho de Melo
    Cc: Corey Ashford
    Cc: David Ahern
    Cc: Frederic Weisbecker
    Cc: Ingo Molnar
    Cc: Namhyung Kim
    Cc: Paul Mackerras
    Cc: Peter Zijlstra
    Link: http://lkml.kernel.org/r/1402821332-12419-1-git-send-email-jolsa@kernel.org
    Signed-off-by: Jiri Olsa

    Jiri Olsa
     
  • The ui_browser__gotorc function needs offset from 'y' member, so the row
    index has to begin with 0, which happens by accident in current code,
    because we display only one header line.

    The bug shows when we want to display more than 1 header lines like
    columns headers in following patches.

    Signed-off-by: Jiri Olsa
    Cc: Corey Ashford
    Cc: David Ahern
    Cc: Frederic Weisbecker
    Cc: Ingo Molnar
    Cc: Namhyung Kim
    Cc: Paul Mackerras
    Cc: Peter Zijlstra
    Link: http://lkml.kernel.org/r/1403178076-14072-4-git-send-email-jolsa@kernel.org
    Signed-off-by: Arnaldo Carvalho de Melo

    Jiri Olsa
     
  • Removing ev_name argument from perf_evsel__hists_browse function,
    because it's not needed. We can get the name out of the 'struct
    perf_evsel' which is passed as argument as well.

    Signed-off-by: Jiri Olsa
    Cc: Corey Ashford
    Cc: David Ahern
    Cc: Frederic Weisbecker
    Cc: Ingo Molnar
    Cc: Namhyung Kim
    Cc: Paul Mackerras
    Cc: Peter Zijlstra
    Link: http://lkml.kernel.org/r/1403178076-14072-3-git-send-email-jolsa@kernel.org
    Signed-off-by: Arnaldo Carvalho de Melo

    Jiri Olsa
     

04 Jun, 2014

2 commits

  • After output/sort fields refactoring, it's expensive
    to check the elide bool in its current location inside
    the 'struct sort_entry'.

    The perf_hpp__should_skip function gets highly noticable in
    workloads with high number of output/sort fields, like for:

    $ perf report -i perf-test.data -F overhead,sample,period,comm,pid,dso,symbol,cpu --stdio

    Performance report:
    9.70% perf [.] perf_hpp__should_skip

    Moving the elide bool into the 'struct perf_hpp_fmt', which
    makes the perf_hpp__should_skip just single struct read.

    Got speedup of around 22% for my test perf.data workload.
    The change should not harm any other workload types.

    Performance counter stats for (10 runs):
    before:
    358,319,732,626 cycles ( +- 0.55% )
    467,129,581,515 instructions # 1.30 insns per cycle ( +- 0.00% )

    150.943975206 seconds time elapsed ( +- 0.62% )

    now:
    278,785,972,990 cycles ( +- 0.12% )
    370,146,797,640 instructions # 1.33 insns per cycle ( +- 0.00% )

    116.416670507 seconds time elapsed ( +- 0.31% )

    Acked-by: Namhyung Kim
    Cc: Arnaldo Carvalho de Melo
    Cc: Corey Ashford
    Cc: David Ahern
    Cc: Frederic Weisbecker
    Cc: Ingo Molnar
    Cc: Namhyung Kim
    Cc: Paul Mackerras
    Cc: Peter Zijlstra
    Cc: Stephane Eranian
    Link: http://lkml.kernel.org/r/20140601142622.GA9131@krava.brq.redhat.com
    Signed-off-by: Jiri Olsa

    Jiri Olsa
     
  • Convert "==" into "=" in ui_browser__warning assignment.

    Bug description: https://bugzilla.kernel.org/show_bug.cgi?id=76751

    Reported-by: David Binderman
    Signed-off-by: Dianfang Zhang
    Acked-by: Arnaldo Carvalho de Melo
    Cc: Jean Delvare
    Link: http://lkml.kernel.org/r/20140530154709.GC1202@kernel.org
    [ changed the changelog a bit ]
    Signed-off-by: Jiri Olsa

    zhangdianfang
     

01 Jun, 2014

8 commits

  • With current output field change, GTK browser cannot display callchain
    information correctly since it couldn't determine where the symbol
    column is. This is a problem - just for now I changed to use the last
    column since it'll work for most cases.

    Also it has a same problem of the percentage as stdio code.

    Signed-off-by: Namhyung Kim
    Cc: Arun Sharma
    Cc: Frederic Weisbecker
    Link: http://lkml.kernel.org/r/1401335910-16832-25-git-send-email-namhyung@kernel.org
    Signed-off-by: Jiri Olsa

    Namhyung Kim
     
  • On stdio, there's a problem that it shows invalid values for
    callchains in cumulated hist entries. It's because it only cares
    about the self period. But with --children behavior, we always add
    callchain info to the cumulated entries so it should use the value in
    that case.

    Before:

    # Children Self Command Shared Object Symbol
    # ........ ........ ....... ................. ................
    #
    61.22% 0.32% swapper [kernel.kallsyms] [k] cpu_idle
    |
    --- cpu_idle
    |
    |--16530.76%-- start_secondary
    |
    |--2758.70%-- rest_init
    | start_kernel
    | x86_64_start_reservations
    | x86_64_start_kernel
    --6837850969203030.00%-- [...]

    After:

    # Children Self Command Shared Object Symbol
    # ........ ........ ....... ................. ................
    #
    61.22% 0.32% swapper [kernel.kallsyms] [k] cpu_idle
    |
    --- cpu_idle
    |
    |--85.70%-- start_secondary
    |
    --14.30%-- rest_init
    start_kernel
    x86_64_start_reservations
    x86_64_start_kernel

    Signed-off-by: Namhyung Kim
    Cc: Arun Sharma
    Cc: Frederic Weisbecker
    Link: http://lkml.kernel.org/r/1401335910-16832-24-git-send-email-namhyung@kernel.org
    Signed-off-by: Jiri Olsa

    Namhyung Kim
     
  • Depending on the configuration perf inserts/removes the Children
    column in the output automatically. But it might not be what user
    wants if [s]he give --fields option explicitly.

    Signed-off-by: Namhyung Kim
    Tested-by: Rodrigo Campos
    Cc: Arun Sharma
    Cc: Frederic Weisbecker
    Link: http://lkml.kernel.org/r/1401335910-16832-18-git-send-email-namhyung@kernel.org
    Signed-off-by: Jiri Olsa

    Namhyung Kim
     
  • Sometimes it needs to disable some columns at runtime. Add help
    functions to support that.

    Signed-off-by: Namhyung Kim
    Tested-by: Arun Sharma
    Tested-by: Rodrigo Campos
    Cc: Frederic Weisbecker
    Link: http://lkml.kernel.org/r/1401335910-16832-15-git-send-email-namhyung@kernel.org
    Signed-off-by: Jiri Olsa

    Namhyung Kim
     
  • If -g cumulative option is given, it needs to show entries which don't
    have self overhead. So apply percent-limit to accumulated overhead
    percentage in this case.

    Signed-off-by: Namhyung Kim
    Tested-by: Arun Sharma
    Tested-by: Rodrigo Campos
    Cc: Frederic Weisbecker
    Link: http://lkml.kernel.org/r/1401335910-16832-14-git-send-email-namhyung@kernel.org
    Signed-off-by: Jiri Olsa

    Namhyung Kim
     
  • Print accumulated stat of a hist entry if requested.

    Signed-off-by: Namhyung Kim
    Tested-by: Arun Sharma
    Tested-by: Rodrigo Campos
    Cc: Frederic Weisbecker
    Link: http://lkml.kernel.org/r/1401335910-16832-13-git-send-email-namhyung@kernel.org
    Signed-off-by: Jiri Olsa

    Namhyung Kim
     
  • Print accumulated stat of a hist entry if requested.

    Signed-off-by: Namhyung Kim
    Tested-by: Arun Sharma
    Tested-by: Rodrigo Campos
    Cc: Frederic Weisbecker
    Link: http://lkml.kernel.org/r/1401335910-16832-12-git-send-email-namhyung@kernel.org
    Signed-off-by: Jiri Olsa

    Namhyung Kim
     
  • Print accumulated stat of a hist entry if requested.

    To do that, add new HPP_PERCENT_ACC_FNS macro and generate a
    perf_hpp_fmt using it. The __hpp__sort_acc() function sorts entries
    by accumulated period value. When accumulated periods of two entries
    are same (i.e. single path callchain) put the caller above since
    accumulation tends to put callers on higher position for obvious
    reason.

    Also add "overhead_children" output field to be selected by user.

    Signed-off-by: Namhyung Kim
    Tested-by: Arun Sharma
    Tested-by: Rodrigo Campos
    Cc: Frederic Weisbecker
    Link: http://lkml.kernel.org/r/1401335910-16832-11-git-send-email-namhyung@kernel.org
    Signed-off-by: Jiri Olsa

    Namhyung Kim
     

21 May, 2014

13 commits

  • The reset_output_field() function is for clearing output field
    settings and will be used for test code in later patch.

    Signed-off-by: Namhyung Kim
    Link: http://lkml.kernel.org/r/1400480762-22852-19-git-send-email-namhyung@kernel.org
    Signed-off-by: Jiri Olsa

    Namhyung Kim
     
  • Now we moved to the perf_hpp_[_sort]_list so no need to keep the old
    hist_entry__sort_list and sort__first_dimension. Also the
    hist_entry__sort_snprintf() can be gone as hist_entry__snprintf()
    provides the functionality.

    Signed-off-by: Namhyung Kim
    Link: http://lkml.kernel.org/r/1400480762-22852-18-git-send-email-namhyung@kernel.org
    Signed-off-by: Jiri Olsa

    Namhyung Kim
     
  • Some fields missed to set default column length so it broke align in
    --stdio output. Add perf_hpp__reset_width() to set it to a sane
    default value.

    Note that this change will ignore -w/--column-widths option for now.

    Before:
    $ perf report -F cpu,comm,overhead --stdio
    ...
    # CPU Command Overhead
    # ............... ........
    #
    0 firefox 2.65%
    0 kworker/0:0 1.45%
    0 swapper 5.52%
    0 synergys 0.92%
    1 firefox 4.54%

    After:
    # CPU Command Overhead
    # ... ............... ........
    #
    0 firefox 2.65%
    0 kworker/0:0 1.45%
    0 swapper 5.52%
    0 synergys 0.92%
    1 firefox 4.54%

    Signed-off-by: Namhyung Kim
    Link: http://lkml.kernel.org/r/1400480762-22852-17-git-send-email-namhyung@kernel.org
    Signed-off-by: Jiri Olsa

    Namhyung Kim
     
  • When it converted sort entries to hpp formats, it missed se->elide
    handling, so add it for compatibility.

    Signed-off-by: Namhyung Kim
    Link: http://lkml.kernel.org/r/1400480762-22852-16-git-send-email-namhyung@kernel.org
    Signed-off-by: Jiri Olsa

    Namhyung Kim
     
  • The hists__filter_entries() function is called when down arrow key is
    pressed for navigating through the entries in TUI. It has a check for
    filtering out entries that have very small overhead (under min_pcnt).

    However it just assumed the entries are sorted by the overhead so when
    it saw such a small overheaded entry, it just stopped navigating as an
    optimization. But it's not true anymore due to new --fields and
    --sort optoin behavior and this case users cannot go down to a next
    entry if ther's an entry with small overhead in-between.

    Signed-off-by: Namhyung Kim
    Link: http://lkml.kernel.org/r/1400480762-22852-14-git-send-email-namhyung@kernel.org
    Signed-off-by: Jiri Olsa

    Namhyung Kim
     
  • The -F/--fields option is to allow user setup output field in any
    order. It can receive any sort keys and following (hpp) fields:

    overhead, overhead_sys, overhead_us, sample and period

    If guest profiling is enabled, overhead_guest_{sys,us} will be
    available too.

    The output fields also affect sort order unless you give -s/--sort
    option. And any keys specified on -s option, will also be added to
    the output field list automatically.

    $ perf report -F sym,sample,overhead
    ...
    # Symbol Samples Overhead
    # .......................... ............ ........
    #
    [.] __cxa_atexit 2 2.50%
    [.] __libc_csu_init 4 5.00%
    [.] __new_exitfn 3 3.75%
    [.] _dl_check_map_versions 1 1.25%
    [.] _dl_name_match_p 4 5.00%
    [.] _dl_setup_hash 1 1.25%
    [.] _dl_sysdep_start 1 1.25%
    [.] _init 5 6.25%
    [.] _setjmp 6 7.50%
    [.] a 8 10.00%
    [.] b 8 10.00%
    [.] brk 1 1.25%
    [.] c 8 10.00%

    Note that, the example output above is captured after applying next
    patch which fixes sort/comparing behavior.

    Requested-by: Ingo Molnar
    Signed-off-by: Namhyung Kim
    Acked-by: Ingo Molnar
    Link: http://lkml.kernel.org/r/1400480762-22852-12-git-send-email-namhyung@kernel.org
    Signed-off-by: Jiri Olsa

    Namhyung Kim
     
  • So that it can be set properly prior to set up output fields. That
    makes easy to handle/warn errors during the setup since it doesn't
    need to be bothered with the GUI.

    Signed-off-by: Namhyung Kim
    Link: http://lkml.kernel.org/r/1400480762-22852-11-git-send-email-namhyung@kernel.org
    Signed-off-by: Jiri Olsa

    Namhyung Kim
     
  • Add overhead{,_sys,_us,_guest_sys,_guest_us}, sample and period sort
    keys so that they can be selected with --sort/-s option.

    $ perf report -s period,comm --stdio
    ...
    # Overhead Period Command
    # ........ ............ ...............
    #
    47.06% 152 swapper
    13.93% 45 qemu-system-arm
    12.38% 40 synergys
    3.72% 12 firefox
    2.48% 8 xchat

    Signed-off-by: Namhyung Kim
    Acked-by: Ingo Molnar
    Link: http://lkml.kernel.org/r/1400480762-22852-9-git-send-email-namhyung@kernel.org
    Signed-off-by: Jiri Olsa

    Namhyung Kim
     
  • The callback was used by TUI for determining color of folded sign
    using percent of first field/column. But it cannot be used anymore
    since it now support dynamic reordering of output field.

    So move the logic to the hist_browser__show_entry().

    Signed-off-by: Namhyung Kim
    Acked-by: Ingo Molnar
    Link: http://lkml.kernel.org/r/1400480762-22852-8-git-send-email-namhyung@kernel.org
    Signed-off-by: Jiri Olsa

    Namhyung Kim
     
  • Until now the hpp and sort functions do similar jobs different ways.
    Since the sort functions converted/wrapped to hpp formats it can do
    the job in a uniform way.

    The perf_hpp__sort_list has a list of hpp formats to sort entries and
    the perf_hpp__list has a list of hpp formats to print output result.

    To have a backward compatibility, it automatically adds 'overhead'
    field in front of sort list. And then all of fields in sort list
    added to the output list (if it's not already there).

    Signed-off-by: Namhyung Kim
    Acked-by: Ingo Molnar
    Link: http://lkml.kernel.org/n/tip-7g3h86woz2sckg3h1lj42ygj@git.kernel.org
    Signed-off-by: Jiri Olsa

    Namhyung Kim
     
  • Move logic of hist_entry__sort_on_period to __hpp__sort() in order to
    support event group report.

    Signed-off-by: Namhyung Kim
    Acked-by: Ingo Molnar
    Link: http://lkml.kernel.org/r/1400480762-22852-5-git-send-email-namhyung@kernel.org
    Signed-off-by: Jiri Olsa

    Namhyung Kim
     
  • This is a preparation of consolidating management of output field and
    sort keys.

    Signed-off-by: Namhyung Kim
    Acked-by: Ingo Molnar
    Link: http://lkml.kernel.org/r/1400480762-22852-3-git-send-email-namhyung@kernel.org
    Signed-off-by: Jiri Olsa

    Namhyung Kim
     
  • Those function pointers will be used to sort report output based on
    the selected fields. This is a preparation of later change.

    Signed-off-by: Namhyung Kim
    Acked-by: Ingo Molnar
    Link: http://lkml.kernel.org/r/1400480762-22852-2-git-send-email-namhyung@kernel.org
    Signed-off-by: Jiri Olsa

    Namhyung Kim
     

02 May, 2014

1 commit

  • Combine all definitions into a common tools/include/linux/types.h and
    kill the wild growth elsewhere. Move DECLARE_BITMAP to its proper
    bitmap.h header.

    Signed-off-by: Borislav Petkov
    Acked-by: Rusty Russell
    Link: http://lkml.kernel.org/n/tip-azczs7qcv6h9xek9od10hiv2@git.kernel.org
    Signed-off-by: Jiri Olsa

    Borislav Petkov
     

28 Apr, 2014

1 commit

  • Moving towards sharing map groups within a process threads.

    Because of this we need the map groups to be dynamically allocated. No
    other functional change is intended in here.

    Based on a patch by Jiri Olsa, but this time _just_ making the
    conversion from statically allocating thread->mg to turning it into a
    pointer and instead of initializing it at thread's constructor,
    introduce a constructor/destructor for the map_groups class and
    call at thread creation time.

    Later we will introduce the get/put methods when we move to sharing
    those map_groups, when the get/put refcounting semantics will be needed.

    Signed-off-by: Arnaldo Carvalho de Melo
    Acked-by: Namhyung Kim
    Cc: Adrian Hunter
    Cc: Corey Ashford
    Cc: David Ahern
    Cc: Don Zickus
    Cc: Frederic Weisbecker
    Cc: Ingo Molnar
    Cc: Mike Galbraith
    Cc: Namhyung Kim
    Cc: Paul Mackerras
    Cc: Peter Zijlstra
    Cc: Stephane Eranian
    Link: http://lkml.kernel.org/r/1397490723-1992-3-git-send-email-jolsa@redhat.com
    Signed-off-by: Jiri Olsa

    Arnaldo Carvalho de Melo
     

24 Apr, 2014

3 commits

  • When TUI hist browser expands/collapses callchains it accounted number
    of callchain nodes into total entries to show. However this code
    ignores filtering so that it can make the cursor go to out of screen.

    Thanks to Jiri Olsa for pointing out a bug (and a fix) in the code.

    Signed-off-by: Namhyung Kim
    Link: http://lkml.kernel.org/r/1398327843-31845-12-git-send-email-namhyung@kernel.org
    Signed-off-by: Jiri Olsa

    Namhyung Kim
     
  • The hist_browser__reset() is only called right after a filter is
    applied so it needs to udpate browser->nr_entries properly. We cannot
    use hists->nr_non_filtered_entreis directly since it's possible that
    such entries are also filtered out by minimum percentage limit.

    In addition when a filter is used for perf top, hist browser's
    nr_entries field was not updated after applying the filter. But it
    needs to be updated as new samples are coming.

    Signed-off-by: Namhyung Kim
    Link: http://lkml.kernel.org/r/1398327843-31845-11-git-send-email-namhyung@kernel.org
    Signed-off-by: Jiri Olsa

    Namhyung Kim
     
  • Rename ->nr_pcnt_entries and hist_browser__update_pcnt_entries() to
    ->nr_non_filtered_entries and hist_browser__update_nr_entries() since
    it's now used for filtering as well.

    Signed-off-by: Namhyung Kim
    Link: http://lkml.kernel.org/r/1398327843-31845-10-git-send-email-namhyung@kernel.org
    Signed-off-by: Jiri Olsa

    Namhyung Kim