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 S1753672AbcBWPer (ORCPT <rfc822;w@1wt.eu>);
	Tue, 23 Feb 2016 10:34:47 -0500
Received: from mail.kernel.org ([198.145.29.136]:52021 "EHLO mail.kernel.org"
	rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP
	id S1752981AbcBWPeq (ORCPT <rfc822;linux-kernel@vger.kernel.org>);
	Tue, 23 Feb 2016 10:34:46 -0500
Date: Tue, 23 Feb 2016 12:34:42 -0300
From: Arnaldo Carvalho de Melo <acme@kernel.org>
To: Jiri Olsa <jolsa@redhat.com>
Cc: Jiri Olsa <jolsa@kernel.org>, lkml <linux-kernel@vger.kernel.org>,
        David Ahern <dsahern@gmail.com>, Ingo Molnar <mingo@kernel.org>,
        Namhyung Kim <namhyung@kernel.org>,
        Peter Zijlstra <a.p.zijlstra@chello.nl>,
        Stephane Eranian <eranian@google.com>, Andi Kleen <ak@linux.intel.com>
Subject: Re: [PATCH 06/23] perf mem: Check for memory events support
Message-ID: <20160223153442.GE15284@kernel.org>
References: <1455525293-8671-1-git-send-email-jolsa@kernel.org>
 <1455525293-8671-7-git-send-email-jolsa@kernel.org>
 <20160223151814.GB15284@kernel.org>
 <20160223152957.GA9102@krava.redhat.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20160223152957.GA9102@krava.redhat.com>
X-Url: http://acmel.wordpress.com
User-Agent: Mutt/1.5.24 (2015-08-30)
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Em Tue, Feb 23, 2016 at 04:29:57PM +0100, Jiri Olsa escreveu:
> On Tue, Feb 23, 2016 at 12:18:14PM -0300, Arnaldo Carvalho de Melo wrote:
> > Em Mon, Feb 15, 2016 at 09:34:36AM +0100, Jiri Olsa escreveu:
> > > Check if current kernel support available memory events
> > > and display the status within -e  list option:
> > > 
> > >   $ perf mem record -e list
> > >   [ok] ldlat-loads
> > >   [ok] ldlat-stores

> > Can't this be:

> >   $ perf mem record -e list
> >   ldlat-loads:  Available on this machine
> >   ldlat-stores: Available on this machine
 
> hum IMO that would be quite long list if DATALA events will
> get in, so I wanted something short and punchy ;-)
 
> how about plaing it into back as you suggested
> with just 'available'
> 
> 	$ perf mem record -e list
> 	ldlat-loads:  available
> 	ldlat-stores: available

Fair enough, this is more in line with, say, 'perf test' output, i.e.:

  $ perf foo
  something: status/result

- Arnaldo