diff options
author | Namhyung Kim <namhyung@kernel.org> | 2015-01-20 15:40:50 +0900 |
---|---|---|
committer | Arnaldo Carvalho de Melo <acme@redhat.com> | 2015-01-22 17:05:10 -0300 |
commit | 4397bd2f90459d550deca7f6ba32c12e382d8b57 (patch) | |
tree | 0fd3b8a6157994aa5e396f65db55a42b3e36a217 /tools/perf/ui/tui/helpline.c | |
parent | 605a3069161dd966d6cea795133c673fb6706e52 (diff) | |
download | op-kernel-dev-4397bd2f90459d550deca7f6ba32c12e382d8b57.zip op-kernel-dev-4397bd2f90459d550deca7f6ba32c12e382d8b57.tar.gz |
perf ui/tui: Show fatal error message only if exists
When perf exits with some error it shows the error message with
ui__error() or ui__warning() and then calls ui__exit() during
exit_browser().
On TUI, it then shows a window titled "Fatal Error" to inform user a
last message which might be related with this condition. However it
sometimes contains no message and just annoyes users.
The usual case for this is running perf top as normal user. (And
/proc/sys/kernel/perf_event_paranoid being 1).
Signed-off-by: Namhyung Kim <namhyung@kernel.org>
Acked-by: Jiri Olsa <jolsa@kernel.org>
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
Link: http://lkml.kernel.org/r/1421736050-5283-1-git-send-email-namhyung@kernel.org
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Diffstat (limited to 'tools/perf/ui/tui/helpline.c')
-rw-r--r-- | tools/perf/ui/tui/helpline.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/tools/perf/ui/tui/helpline.c b/tools/perf/ui/tui/helpline.c index 1c8b9af..88f5143 100644 --- a/tools/perf/ui/tui/helpline.c +++ b/tools/perf/ui/tui/helpline.c @@ -9,6 +9,7 @@ #include "../libslang.h" char ui_helpline__last_msg[1024]; +bool tui_helpline__set; static void tui_helpline__pop(void) { @@ -35,6 +36,8 @@ static int tui_helpline__show(const char *format, va_list ap) sizeof(ui_helpline__last_msg) - backlog, format, ap); backlog += ret; + tui_helpline__set = true; + if (ui_helpline__last_msg[backlog - 1] == '\n') { ui_helpline__puts(ui_helpline__last_msg); SLsmg_refresh(); |