Commit 53c54019 authored by Arnaldo Carvalho de Melo's avatar Arnaldo Carvalho de Melo Committed by Ingo Molnar

perf report: Add a popup menu to ask what operation is to be performed

Right now it presents a menu with these options:

 +------------------------------+
 | Annotate CURRENT_SYMBOL_NAME |
 | Exit                         |
 +------------------------------+

If the highlighted (current) symbol is not annotatable only the
"Exit" option will appear.

Also add a confirmation dialog when ESC is pressed on the top
level to avoid exiting the application by pressing one too many
ESC key.

To get to the menu just press the -> (Right navigation key), to
exit just press ESC.
Signed-off-by: default avatarArnaldo Carvalho de Melo <acme@redhat.com>
Cc: Frédéric Weisbecker <fweisbec@gmail.com>
Cc: Mike Galbraith <efault@gmx.de>
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
Cc: Paul Mackerras <paulus@samba.org>
LKML-Reference: <1269459619-982-1-git-send-email-acme@infradead.org>
Signed-off-by: default avatarIngo Molnar <mingo@elte.hu>
parent d814f301
...@@ -28,6 +28,47 @@ static newtComponent newt_form__new(void) ...@@ -28,6 +28,47 @@ static newtComponent newt_form__new(void)
return self; return self;
} }
static int popup_menu(int argc, const char *argv[])
{
struct newtExitStruct es;
int i, rc = -1, max_len = 5;
newtComponent listbox, form = newt_form__new();
if (form == NULL)
return -1;
listbox = newtListbox(0, 0, argc, NEWT_FLAG_RETURNEXIT);
if (listbox == NULL)
goto out_destroy_form;
newtFormAddComponents(form, listbox, NULL);
for (i = 0; i < argc; ++i) {
int len = strlen(argv[i]);
if (len > max_len)
max_len = len;
if (newtListboxAddEntry(listbox, argv[i], (void *)(long)i))
goto out_destroy_form;
}
newtCenteredWindow(max_len, argc, NULL);
newtFormRun(form, &es);
rc = newtListboxGetCurrent(listbox) - NULL;
if (es.reason == NEWT_EXIT_HOTKEY)
rc = -1;
newtPopWindow();
out_destroy_form:
newtFormDestroy(form);
return rc;
}
static bool dialog_yesno(const char *msg)
{
/* newtWinChoice should really be accepting const char pointers... */
char yes[] = "Yes", no[] = "No";
return newtWinChoice(NULL, no, yes, (char *)msg) == 2;
}
/* /*
* When debugging newt problems it was useful to be able to "unroll" * When debugging newt problems it was useful to be able to "unroll"
* the calls to newtCheckBoxTreeAdd{Array,Item}, so that we can generate * the calls to newtCheckBoxTreeAdd{Array,Item}, so that we can generate
...@@ -309,6 +350,19 @@ static void symbol__annotate_browser(const struct symbol *self) ...@@ -309,6 +350,19 @@ static void symbol__annotate_browser(const struct symbol *self)
free(str); free(str);
} }
static const void *newt__symbol_tree_get_current(newtComponent self)
{
if (symbol_conf.use_callchain)
return newtCheckboxTreeGetCurrent(self);
return newtListboxGetCurrent(self);
}
static void perf_session__selection(newtComponent self, void *data)
{
const struct symbol **symbol_ptr = data;
*symbol_ptr = newt__symbol_tree_get_current(self);
}
void perf_session__browse_hists(struct rb_root *hists, u64 session_total, void perf_session__browse_hists(struct rb_root *hists, u64 session_total,
const char *helpline) const char *helpline)
{ {
...@@ -322,6 +376,7 @@ void perf_session__browse_hists(struct rb_root *hists, u64 session_total, ...@@ -322,6 +376,7 @@ void perf_session__browse_hists(struct rb_root *hists, u64 session_total,
char str[1024]; char str[1024];
newtComponent form, tree; newtComponent form, tree;
struct newtExitStruct es; struct newtExitStruct es;
const struct symbol *selection;
snprintf(str, sizeof(str), "Samples: %Ld", session_total); snprintf(str, sizeof(str), "Samples: %Ld", session_total);
newtDrawRootText(0, 0, str); newtDrawRootText(0, 0, str);
...@@ -336,6 +391,8 @@ void perf_session__browse_hists(struct rb_root *hists, u64 session_total, ...@@ -336,6 +391,8 @@ void perf_session__browse_hists(struct rb_root *hists, u64 session_total,
tree = newtListbox(0, 0, rows - 5, (NEWT_FLAG_SCROLL | tree = newtListbox(0, 0, rows - 5, (NEWT_FLAG_SCROLL |
NEWT_FLAG_RETURNEXIT)); NEWT_FLAG_RETURNEXIT));
newtComponentAddCallback(tree, perf_session__selection, &selection);
list_for_each_entry(se, &hist_entry__sort_list, list) { list_for_each_entry(se, &hist_entry__sort_list, list) {
if (se->elide) if (se->elide)
continue; continue;
...@@ -377,19 +434,42 @@ void perf_session__browse_hists(struct rb_root *hists, u64 session_total, ...@@ -377,19 +434,42 @@ void perf_session__browse_hists(struct rb_root *hists, u64 session_total,
form = newt_form__new(); form = newt_form__new();
newtFormAddHotKey(form, 'A'); newtFormAddHotKey(form, 'A');
newtFormAddHotKey(form, 'a'); newtFormAddHotKey(form, 'a');
newtFormAddHotKey(form, NEWT_KEY_RIGHT);
newtFormAddComponents(form, tree, NULL); newtFormAddComponents(form, tree, NULL);
selection = newt__symbol_tree_get_current(tree);
while (1) { while (1) {
const struct symbol *selection; char annotate[512];
const char *options[2];
int nr_options = 0, choice;
newtFormRun(form, &es); newtFormRun(form, &es);
if (es.reason == NEWT_EXIT_HOTKEY && if (es.reason == NEWT_EXIT_HOTKEY) {
toupper(es.u.key) != 'A') if (toupper(es.u.key) == 'A') {
symbol__annotate_browser(selection);
continue;
}
if (es.u.key == NEWT_KEY_ESCAPE ||
toupper(es.u.key) == 'Q' ||
es.u.key == CTRL('c')) {
if (dialog_yesno("Do you really want to exit?"))
break; break;
if (!symbol_conf.use_callchain)
selection = newtListboxGetCurrent(tree);
else else
selection = newtCheckboxTreeGetCurrent(tree); continue;
}
}
if (selection != NULL) {
snprintf(annotate, sizeof(annotate),
"Annotate %s", selection->name);
options[nr_options++] = annotate;
}
options[nr_options++] = "Exit";
choice = popup_menu(nr_options, options);
if (choice == nr_options - 1)
break;
else if (selection != NULL && choice >= 0)
symbol__annotate_browser(selection); symbol__annotate_browser(selection);
} }
......
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