Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1496860

Re: [PATCH 3/3] perf, tools: Fix completion script to handle comma list

From Andi Kleen <ak@linux.intel.com>
Newsgroups linux.kernel
Subject Re: [PATCH 3/3] perf, tools: Fix completion script to handle comma list
Date 2016-10-06 22:50 +0200
Message-ID <spnM6-5ZP-7@gated-at.bofh.it> (permalink)
References <spmwF-5ch-5@gated-at.bofh.it> <spmwF-5ch-13@gated-at.bofh.it> <spn9n-5J3-3@gated-at.bofh.it> <spn9n-5J3-7@gated-at.bofh.it> <spnCq-5Wo-15@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


On Thu, Oct 06, 2016 at 05:29:44PM -0300, Arnaldo Carvalho de Melo wrote:
> Em Thu, Oct 06, 2016 at 01:08:39PM -0700, Andi Kleen escreveu:
> > > perf stat -e CPL_CYCLES.RING0
> > > 
> > > but if I try:
> > > 
> > > perf stat -e cycles,CPL<TAB>
> > > 
> > > I get nothing :-\
> > > 
> > > Is this working with you?
> > 
> > You can't mix cases like this, the upper case patch doesn't support that.
> > It's either all upper or all lower case.
> > 
> > If you continue with lower case it should work.
> 
> I understand that this should be a limitation of the current patch, but
> if we decide to support completion after a comma, I think people will
> expect that all events will be available :-\

I don't see a simple way to do it.  I'm not gonna write a complex parser in shell.

If that really is a requirement the patch has to be withdrawn.

-Andi

Back to linux.kernel | Previous | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

[PATCH 1/3] perf, tools: Add support for perf list json Andi Kleen <andi@firstfloor.org> - 2016-10-06 21:30 +0200
  [PATCH 3/3] perf, tools: Fix completion script to handle comma list Andi Kleen <andi@firstfloor.org> - 2016-10-06 21:30 +0200
    Re: [PATCH 3/3] perf, tools: Fix completion script to handle comma  list Arnaldo Carvalho de Melo <acme@kernel.org> - 2016-10-06 22:10 +0200
      Re: [PATCH 3/3] perf, tools: Fix completion script to handle comma  list Andi Kleen <ak@linux.intel.com> - 2016-10-06 22:10 +0200
        Re: [PATCH 3/3] perf, tools: Fix completion script to handle comma  list Arnaldo Carvalho de Melo <acme@kernel.org> - 2016-10-06 22:40 +0200
          Re: [PATCH 3/3] perf, tools: Fix completion script to handle comma  list Andi Kleen <ak@linux.intel.com> - 2016-10-06 22:50 +0200
  Re: [PATCH 1/3] perf, tools: Add support for perf list json Arnaldo Carvalho de Melo <acme@kernel.org> - 2016-10-06 22:10 +0200

csiph-web