Re: [GIT PULL] Vendor events file/dir names

From: Ingo Molnar
Date: Wed Oct 19 2016 - 11:24:24 EST



* Arnaldo Carvalho de Melo <acme@xxxxxxxxxx> wrote:

> Em Wed, Oct 19, 2016 at 03:38:33PM +0200, Ingo Molnar escreveu:
> > * Arnaldo Carvalho de Melo <acme@xxxxxxxxxx> wrote:
> > > Em Tue, Oct 18, 2016 at 09:59:01AM +0200, Ingo Molnar escreveu:
> > > > * Arnaldo Carvalho de Melo <acme@xxxxxxxxxx> wrote:
> > > > > Ingo pointed out to me that in the kernel sources we do not use
> > > > > file/dir names with uppercase chars (look, for instance, at arch/), so I
> > > > > mostly scripted a conversion to lowercase and what I got is at:
> > >
> > > > > git://git.kernel.org/pub/scm/linux/kernel/git/acme/linux.git perf/vendor_events
> > >
> > > > > Please let me know if you have any technical argument against
> > > > > this move,
> > > >
> > > > Looks good to me!
> > >
> > > So I've made this signed tag available with just what is in this branch,
> > > that was based off tip/perf/urgent, please pull it into the most convenient
> > > branch at this time,
> > >
> > > Thanks,
> > >
> > > - Arnaldo
> > >
> > > tag perf-vendor_events-for-mingo-20161018
> >
> > Ok, I tried this out, and I like it mostly - the event files hierarchy looks good
> > and the 'perf list' output looks good, but I found a couple of usability problems
> > when trying to actually navigate and search the new hw events:
>
> Ok, but can't we work on these usability glitches with patches on top of
> what we have now?

Yeah, sure - and it seems like we are in agreement about the usability details, so
I've pulled your branch into tip:perf/core.

Thanks,

Ingo