From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <linux-kernel-owner@vger.kernel.org>
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
        id S934725AbcJFUln (ORCPT <rfc822;w@1wt.eu>);
        Thu, 6 Oct 2016 16:41:43 -0400
Received: from mga05.intel.com ([192.55.52.43]:18041 "EHLO mga05.intel.com"
        rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP
        id S933011AbcJFUlf (ORCPT <rfc822;linux-kernel@vger.kernel.org>);
        Thu, 6 Oct 2016 16:41:35 -0400
X-ExtLoop1: 1
X-IronPort-AV: E=Sophos;i="5.31,454,1473145200"; 
   d="scan'208";a="1050453317"
Date: Thu, 6 Oct 2016 13:41:34 -0700
From: Andi Kleen <ak@linux.intel.com>
To: Arnaldo Carvalho de Melo <acme@kernel.org>
Cc: Andi Kleen <andi@firstfloor.org>, jolsa@kernel.org,
        linux-kernel@vger.kernel.org
Subject: Re: [PATCH 3/3] perf, tools: Fix completion script to handle comma
 list
Message-ID: <20161006204134.GE3078@tassilo.jf.intel.com>
References: <1475781807-6034-1-git-send-email-andi@firstfloor.org>
 <1475781807-6034-3-git-send-email-andi@firstfloor.org>
 <20161006200510.GC4809@kernel.org>
 <20161006200839.GD3078@tassilo.jf.intel.com>
 <20161006202944.GH4809@kernel.org>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20161006202944.GH4809@kernel.org>
User-Agent: Mutt/1.6.1 (2016-04-27)
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

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