Skip to contentSkip to conversation

perf hists browser: Do not exit when 'f' is pressed in 'report' mode

tip:perf/core tip-bot for Arnaldo Carvalho de Melo 1 participant 1 message Jun 19, 2015 perf events

lore
Share & tools
lore
#0 lore raw
Commit-ID:  516e536849d26a2bcacd9050f4a7222a14c0f9e4
Gitweb:     http://git.kernel.org/tip/516e536849d26a2bcacd9050f4a7222a14c0f9e4
Author:     Arnaldo Carvalho de Melo <[email protected]>
AuthorDate: Fri, 19 Jun 2015 16:59:43 -0300
Committer:  Arnaldo Carvalho de Melo <[email protected]>
CommitDate: Fri, 19 Jun 2015 16:59:43 -0300

perf hists browser: Do not exit when 'f' is pressed in 'report' mode

The 'f' hotkey is only used when in 'top', dynamic mode, to enable/disable events, currently not making sense in the 'report', static mode, where we can't go from showing the histogram entries created from a perf.data file to adding more events after recreating the evlist created from the perf.data file, albeit possible, this is not implemented right now.

Cc Adrian Hunter <[email protected]>
Cc Borislav Petkov <[email protected]>
Cc David Ahern <[email protected]>
Cc Don Zickus <[email protected]>
Cc Frederic Weisbecker <[email protected]>
Cc Jiri Olsa <[email protected]>
Cc Namhyung Kim <[email protected]>
Cc Stephane Eranian <[email protected]>
---
 tools/perf/ui/browsers/hists.c | 3 +++
 1 file changed, 3 insertions(+)
tools/perf/ui/browsers/hists.c +3 -0
index 764f7ca..79a093a7 100644
@@ -1901,7 +1901,10 @@ static int perf_evsel__hists_browse(struct perf_evsel *evsel, int nr_events,
 			/* Fall thru */
 		case 'q':
 		case CTRL('c'):
+			goto out_free_stack;
 		case 'f':
+			if (is_report_browser(hbt))
+				continue;
 			goto out_free_stack;
 		default:
 			continue;

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in Please read the FAQ at http://www.tux.org/lkml/