lkml.org 
[lkml]   [2015]   [Apr]   [20]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 6/7] perf hists browser: Split popup menu actions
On Mon, Apr 20, 2015 at 11:46:07AM +0200, Jiri Olsa wrote:
> On Sun, Apr 19, 2015 at 01:04:14PM +0900, Namhyung Kim wrote:
>
> SNIP
>
> > continue;
> > case 's':
> > - if (is_report_browser(hbt))
> > - goto do_data_switch;
> > + if (is_report_browser(hbt)) {
> > + key = do_switch_data(opts, browser, hbt,
> > + fstack, env);
> > + if (key == K_SWITCH_INPUT_DATA)
> > + goto out_free_stack;
> > + }
> > continue;
> > case 'i':
> > /* env->arch is NULL for live-mode (i.e. perf top) */
> > @@ -1592,10 +1878,16 @@ static int perf_evsel__hists_browse(struct perf_evsel *evsel, int nr_events,
> > continue;
> > }
> > top = pstack__pop(fstack);
> > - if (top == &browser->hists->dso_filter)
> > - goto zoom_out_dso;
> > - if (top == &browser->hists->thread_filter)
> > - goto zoom_out_thread;
> > + if (top == &browser->hists->dso_filter) {
> > + perf_hpp__set_elide(HISTC_DSO, false);
> > + browser->hists->dso_filter = NULL;
> > + ui_helpline__pop();
> > + }
> > + if (top == &browser->hists->thread_filter) {
> > + perf_hpp__set_elide(HISTC_THREAD, false);
> > + thread__zput(browser->hists->thread_filter);
> > + ui_helpline__pop();
> > + }
>
> ui_helpline__pop could be called in heare and removed in above legs:
>
> ui_helpline__pop()
>
> also the original code calls following for zoom out:
>
> - hists__filter_by_dso(hists);
> - hist_browser__reset(browser);
>

Right. The intention was to call do_zoom_thread() or do_zoom_dso()
like in the next patch. Will fix this anyway.

Thanks for your careful review!
Namhyung


\
 
 \ /
  Last update: 2015-04-20 15:01    [W:0.080 / U:0.196 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site