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 S965773AbbJ1NDb (ORCPT <rfc822;w@1wt.eu>);
	Wed, 28 Oct 2015 09:03:31 -0400
Received: from mx1.redhat.com ([209.132.183.28]:49370 "EHLO mx1.redhat.com"
	rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP
	id S965420AbbJ1NDa (ORCPT <rfc822;linux-kernel@vger.kernel.org>);
	Wed, 28 Oct 2015 09:03:30 -0400
Date: Wed, 28 Oct 2015 14:03:24 +0100
From: Jiri Olsa <jolsa@redhat.com>
To: Arnaldo Carvalho de Melo <acme@kernel.org>
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>,
        "Liang, Kan" <kan.liang@intel.com>
Subject: Re: [PATCH 31/52] perf stat record: Synthesize stat record data
Message-ID: <20151028130324.GA5216@krava.brq.redhat.com>
References: <1445784728-21732-1-git-send-email-jolsa@kernel.org>
 <1445784728-21732-32-git-send-email-jolsa@kernel.org>
 <20151027144244.GE9405@kernel.org>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20151027144244.GE9405@kernel.org>
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

On Tue, Oct 27, 2015 at 11:42:44AM -0300, Arnaldo Carvalho de Melo wrote:
> Em Sun, Oct 25, 2015 at 03:51:47PM +0100, Jiri Olsa escreveu:
> > Synthesizing needed stat record data for report/script:
> >   - cpu/thread maps
> >   - stat config
> 
> After this it gets a bit better, but then I expected that to specify an
> event I would be able to use:
> 
> [root@zoo linux]# perf stat record -e cycles usleep 1
>   Error: unknown switch `e'
> 
>  Usage: perf record [<options>] [<command>]
>     or: perf record [<options>] -- <command> [<options>]
> 
>     -o, --output <file>   output file name
> 
> 
> But I need to do it as:
> 
> [root@zoo linux]# perf stat -e cycles record usleep 1
> 
>  Performance counter stats for 'usleep 1':
> 
>             948417      cycles                                                      
> 
>        0.000749965 seconds time elapsed
> 
> [root@zoo linux]# 
> 
> --------------------------------------------------
> 
> This is confusing...

yep, the thing is that we need to have all the options supported
by perf stat to work under perf stat record.

you basically let user use perf stat with 'record' that just says
'store the data I'm monitoring'

I'll try to find some way to move the options under perf stat record
without too many changes.. but not sure

jirka