perf ui tui progress: Implement the ui_progress_ops->finish() method
authorArnaldo Carvalho de Melo <acme@redhat.com>
Mon, 24 Aug 2015 19:18:26 +0000 (16:18 -0300)
committerArnaldo Carvalho de Melo <acme@redhat.com>
Mon, 24 Aug 2015 19:18:26 +0000 (16:18 -0300)
So that we can erase the progress bar after we're done with it, avoiding
things like:

-------------------------------------------------------------------

          ┌─Error:──────────────────────────────────────────────────────┐
          │Can't annotate unmapped_area_topdown:                        │
          │                                                             │
          │No vmlinux file with build id a826726b5ddacfab1f0bade868f1a79
          │was found in the path.                                       │
          │                                                             │
          │Note that annotation using /proc/kcore requires CAP_SYS_RAWIO│
┌Processin│                                                             │──┐
│         │Please use:                                                  │  │
└─────────│                                                             │──┘
          │  perf buildid-cache -vu vmlinux                             │
          │                                                             │
          │or:                                                          │
          │                                                             │
          │  --vmlinux vmlinux                                          │
          │                                                             │
          │                                                             │
          │Press any key...                                             │
          └─────────────────────────────────────────────────────────────┘

Can't annotate unmapped_area_topdown:
-------------------------------------------------------------------

I.e. that finished progress bar behind the error window. It is not a
problem when we end up redrawing the whole screen, but its ugly when
we present such error windows, provide a TUI method so that code like
the above may avoid this situation, as will be done with the annotation
code in the next cset.

Cc: Adrian Hunter <adrian.hunter@intel.com>
Cc: Borislav Petkov <bp@suse.de>
Cc: David Ahern <dsahern@gmail.com>
Cc: Frederic Weisbecker <fweisbec@gmail.com>
Cc: Jiri Olsa <jolsa@redhat.com>
Cc: Namhyung Kim <namhyung@kernel.org>
Cc: Stephane Eranian <eranian@google.com>
Link: http://lkml.kernel.org/n/tip-qvktnojzwwe37pweging058t@git.kernel.org
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
tools/perf/ui/tui/progress.c

index c61d14b101e037fb9cea4e7188f5745b16d5206b..c4b99008e2c91d4e2d6fbade5c50ba836c2f3668 100644 (file)
@@ -33,9 +33,26 @@ static void tui_progress__update(struct ui_progress *p)
        pthread_mutex_unlock(&ui__lock);
 }
 
+static void tui_progress__finish(void)
+{
+       int y;
+
+       if (use_browser <= 0)
+               return;
+
+       ui__refresh_dimensions(false);
+       pthread_mutex_lock(&ui__lock);
+       y = SLtt_Screen_Rows / 2 - 2;
+       SLsmg_set_color(0);
+       SLsmg_fill_region(y, 0, 3, SLtt_Screen_Cols, ' ');
+       SLsmg_refresh();
+       pthread_mutex_unlock(&ui__lock);
+}
+
 static struct ui_progress_ops tui_progress__ops =
 {
-       .update         = tui_progress__update,
+       .update = tui_progress__update,
+       .finish = tui_progress__finish,
 };
 
 void tui_progress__init(void)