lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Thu, 10 Jul 2014 10:50:22 -0300 From: Arnaldo Carvalho de Melo <acme@...nel.org> To: Jiri Olsa <jolsa@...hat.com> Cc: Ingo Molnar <mingo@...nel.org>, linux-kernel@...r.kernel.org, Adrian Hunter <adrian.hunter@...el.com>, David Ahern <dsahern@...il.com>, Don Zickus <dzickus@...hat.com>, Frederic Weisbecker <fweisbec@...il.com>, Mike Galbraith <efault@....de>, Namhyung Kim <namhyung@...nel.org>, Paul Mackerras <paulus@...ba.org>, Peter Zijlstra <peterz@...radead.org>, Stephane Eranian <eranian@...gle.com> Subject: Re: [PATCH 04/16] perf hists browser: Override ui_browser refresh_dimensions method Em Thu, Jul 10, 2014 at 12:48:49AM +0200, Jiri Olsa escreveu: > On Tue, Jul 08, 2014 at 04:02:52PM -0300, Arnaldo Carvalho de Melo wrote: > > From: Arnaldo Carvalho de Melo <acme@...hat.com> > > > > SNIP > > > index ca63564d203a..ef7abf896f5a 100644 > > --- a/tools/perf/ui/browsers/hists.c > > +++ b/tools/perf/ui/browsers/hists.c > > @@ -56,11 +56,19 @@ static u32 hist_browser__nr_entries(struct hist_browser *hb) > > return nr_entries + hb->nr_callchain_rows; > > } > > > > -static void hist_browser__refresh_dimensions(struct hist_browser *browser) > > +static void hist_browser__refresh_dimensions(struct ui_browser *browser) > > { > > + struct hist_browser *hb = container_of(browser, struct hist_browser, b); > > + > > /* 3 == +/- toggle symbol before actual hist_entry rendering */ > > - browser->b.width = 3 + (hists__sort_list_width(browser->hists) + > > - sizeof("[k]")); > > + browser->width = 3 + (hists__sort_list_width(hb->hists) + sizeof("[k]")); > > why not remove this completely? it'll get overwritten in the > following ui_browser__refresh_dimensions call anyway.. I thought I left a comment below it explaining that this would get fixed at a later patch, but yeah, we can comment it out, leaving it just as a hint at what needs to be done if we ever want to partition the screen horizontally. > > + /* > > + * FIXME: Just keeping existing behaviour, but this really should be > > + * before updating browser->width, as it will invalidate the > > + * calculation above. Fix this and the fallout in another > > + * changeset. > > + */ > > extra whitespace above Gack, thought I had that fixed, will check my pre-commit hooks... now: [acme@zoo linux]$ ls -la .git/hooks/pre-commit.sample -rwxrwxr-x. 1 acme acme 1704 Abr 16 2013 .git/hooks/pre-commit.sample [acme@zoo linux]$ mv .git/hooks/pre-commit.sample .git/hooks/pre-commit [acme@zoo linux]$ vim .git/hooks/pre-commit [acme@zoo linux]$ tail -3 .git/hooks/pre-commit # If there are whitespace errors, print the offending file names and # fail. exec git diff-index --check --cached $against -- [acme@zoo linux] I guess I should catch those on this machine from now on. - Arnaldo -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists