perf tools: Factor sort elide
Get series with b4
b4 am [email protected]Versions newest first
- v1 Latest2 patches 1 replyIn review
Patches in v1 No applies recorded yet
| # | Subject and files | Size | Replies | Applied |
|---|---|---|---|---|
| 1/2 | perf tools: Remove elide setup for SORT_MODE__MEMORY modetools/ | +0 | 0 replies | |
| 2/2 | perf tools: Move elide bool into perf_hpp_fmt structtools/ tools/ tools/ +1 more | +67 | 0 replies |
Reviews trailers per patch
None of the 2 patches has a Reviewed-by or Acked-by of their own; the series has one for all of them.
| Patch | Namhyung Kim |
|---|---|
| Series | A-b A-b |
| 1/2 | |
| 2/2 |
Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.
Cover letter [RFC/PATCH 0/2] perf tools: Factor sort elide
hi,
factoring the elide code to make perf_hpp__should_skip function cheap call/check again.
Also I was wondering.. do we want to get rid of sort_entry structs in favor of perf_hpp__* stuff? Any plans?
I'm sending this a RFC, since any other such refactoring would change this stuff for sure as well ;-).
thanks,
jirka
Cc Arnaldo Carvalho de Melo <[email protected]>
Cc Corey Ashford <[email protected]>
Cc David Ahern <[email protected]>
Cc Frederic Weisbecker <[email protected]>
Cc Ingo Molnar <[email protected]>
Cc Namhyung Kim <[email protected]>
Cc Paul Mackerras <[email protected]>
Cc Peter Zijlstra <[email protected]>
Cc Stephane Eranian <[email protected]>
Signed-off-by Jiri Olsa <[email protected]>
--- tools/perf/ui/browsers/hists.c | 8 ++++---- tools/perf/util/hist.h | 8 +++++++- tools/perf/util/sort.c | 102 +++++++++++++++++++++++++++++++++++++++++++++++++++++++----------------------------------------------- tools/perf/util/sort.h | 2 +- 4 files changed, 67 insertions(+), 53 deletions(-)