perf ui: Serialize screen updates

The ui operations so far were used by just one thread, but 'perf top
--tui' now has two threads updating the screen, so we need to use a
mutex to avoid garbling the screen.

Cc: Frederic Weisbecker <fweisbec@gmail.com>
Cc: Ingo Molnar <mingo@elte.hu>
Cc: Mike Galbraith <efault@gmx.de>
Cc: Paul Mackerras <paulus@samba.org>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Stephane Eranian <eranian@google.com>
Cc: Tom Zanussi <tzanussi@gmail.com>
LKML-Reference: <new-submission>
Signed-off-by: default avatarArnaldo Carvalho de Melo <acme@redhat.com>
parent ba3dd36c
...@@ -637,6 +637,7 @@ else ...@@ -637,6 +637,7 @@ else
LIB_H += util/ui/libslang.h LIB_H += util/ui/libslang.h
LIB_H += util/ui/progress.h LIB_H += util/ui/progress.h
LIB_H += util/ui/util.h LIB_H += util/ui/util.h
LIB_H += util/ui/ui.h
endif endif
endif endif
......
#include "libslang.h" #include "libslang.h"
#include "ui.h"
#include <linux/compiler.h> #include <linux/compiler.h>
#include <linux/list.h> #include <linux/list.h>
#include <linux/rbtree.h> #include <linux/rbtree.h>
...@@ -178,6 +179,7 @@ int ui_browser__show(struct ui_browser *self, const char *title, ...@@ -178,6 +179,7 @@ int ui_browser__show(struct ui_browser *self, const char *title,
if (self->sb == NULL) if (self->sb == NULL)
return -1; return -1;
pthread_mutex_lock(&ui__lock);
SLsmg_gotorc(0, 0); SLsmg_gotorc(0, 0);
ui_browser__set_color(self, NEWT_COLORSET_ROOT); ui_browser__set_color(self, NEWT_COLORSET_ROOT);
slsmg_write_nstring(title, self->width); slsmg_write_nstring(title, self->width);
...@@ -188,25 +190,30 @@ int ui_browser__show(struct ui_browser *self, const char *title, ...@@ -188,25 +190,30 @@ int ui_browser__show(struct ui_browser *self, const char *title,
va_start(ap, helpline); va_start(ap, helpline);
ui_helpline__vpush(helpline, ap); ui_helpline__vpush(helpline, ap);
va_end(ap); va_end(ap);
pthread_mutex_unlock(&ui__lock);
return 0; return 0;
} }
void ui_browser__hide(struct ui_browser *self) void ui_browser__hide(struct ui_browser *self)
{ {
pthread_mutex_lock(&ui__lock);
newtFormDestroy(self->form); newtFormDestroy(self->form);
self->form = NULL; self->form = NULL;
ui_helpline__pop(); ui_helpline__pop();
pthread_mutex_unlock(&ui__lock);
} }
int ui_browser__refresh(struct ui_browser *self) int ui_browser__refresh(struct ui_browser *self)
{ {
int row; int row;
pthread_mutex_lock(&ui__lock);
newtScrollbarSet(self->sb, self->index, self->nr_entries - 1); newtScrollbarSet(self->sb, self->index, self->nr_entries - 1);
row = self->refresh(self); row = self->refresh(self);
ui_browser__set_color(self, HE_COLORSET_NORMAL); ui_browser__set_color(self, HE_COLORSET_NORMAL);
SLsmg_fill_region(self->y + row, self->x, SLsmg_fill_region(self->y + row, self->x,
self->height - row, self->width, ' '); self->height - row, self->width, ' ');
pthread_mutex_unlock(&ui__lock);
return 0; return 0;
} }
......
...@@ -5,6 +5,7 @@ ...@@ -5,6 +5,7 @@
#include "../debug.h" #include "../debug.h"
#include "helpline.h" #include "helpline.h"
#include "ui.h"
void ui_helpline__pop(void) void ui_helpline__pop(void)
{ {
...@@ -55,6 +56,7 @@ int ui_helpline__show_help(const char *format, va_list ap) ...@@ -55,6 +56,7 @@ int ui_helpline__show_help(const char *format, va_list ap)
int ret; int ret;
static int backlog; static int backlog;
pthread_mutex_lock(&ui__lock);
ret = vsnprintf(ui_helpline__last_msg + backlog, ret = vsnprintf(ui_helpline__last_msg + backlog,
sizeof(ui_helpline__last_msg) - backlog, format, ap); sizeof(ui_helpline__last_msg) - backlog, format, ap);
backlog += ret; backlog += ret;
...@@ -64,6 +66,7 @@ int ui_helpline__show_help(const char *format, va_list ap) ...@@ -64,6 +66,7 @@ int ui_helpline__show_help(const char *format, va_list ap)
newtRefresh(); newtRefresh();
backlog = 0; backlog = 0;
} }
pthread_mutex_unlock(&ui__lock);
return ret; return ret;
} }
...@@ -6,6 +6,9 @@ ...@@ -6,6 +6,9 @@
#include "../debug.h" #include "../debug.h"
#include "browser.h" #include "browser.h"
#include "helpline.h" #include "helpline.h"
#include "ui.h"
pthread_mutex_t ui__lock = PTHREAD_MUTEX_INITIALIZER;
static void newt_suspend(void *d __used) static void newt_suspend(void *d __used)
{ {
......
#ifndef _PERF_UI_H_
#define _PERF_UI_H_ 1
#include <pthread.h>
extern pthread_mutex_t ui__lock;
#endif /* _PERF_UI_H_ */
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment