Skip to contentSkip to conversation

perf ordered_events: Clear the progress bar at the end of a flush

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

lore
Share & tools
lore
#0 lore raw
Commit-ID:  5c9ce1e644c1919ac4bff4394a4c372f47a89b4e
Gitweb:     http://git.kernel.org/tip/5c9ce1e644c1919ac4bff4394a4c372f47a89b4e
Author:     Arnaldo Carvalho de Melo <[email protected]>
AuthorDate: Mon, 24 Aug 2015 17:16:22 -0300
Committer:  Arnaldo Carvalho de Melo <[email protected]>
CommitDate: Mon, 24 Aug 2015 17:16:22 -0300

perf ordered_events: Clear the progress bar at the end of a flush

We were depending on the next screen operation after a flush() being one that would redraw the whole screen so that the progress bar would be overwritten, when that didn't happen a screen artifact of, say, a error dialog window would be overlaid on top of the progress bar, fix it by calling ui_browser__finish(), that now has a TUI implementation.

Cc Adrian Hunter <[email protected]>
Cc Borislav Petkov <[email protected]>
Cc David Ahern <[email protected]>
Cc Frederic Weisbecker <[email protected]>
Cc Jiri Olsa <[email protected]>
Cc Namhyung Kim <[email protected]>
Cc Stephane Eranian <[email protected]>
---
 tools/perf/util/ordered-events.c | 3 +++
 1 file changed, 3 insertions(+)
tools/perf/util/ordered-events.c +3 -0
index 52be201..b1b9e23 100644
@@ -220,6 +220,9 @@ static int __ordered_events__flush(struct ordered_events *oe)
 	else if (last_ts <= limit)
 		oe->last = list_entry(head->prev, struct ordered_event, list);
 
+	if (show_progress)
+		ui_progress__finish();
+
 	return 0;
 }