[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <87pqb7cdbz.fsf@sejong.aot.lge.com>
Date: Tue, 17 Apr 2012 11:16:32 +0900
From: Namhyung Kim <namhyung.kim@....com>
To: Jiri Olsa <jolsa@...hat.com>
Cc: Peter Zijlstra <a.p.zijlstra@...llo.nl>, acme@...hat.com,
mingo@...e.hu, paulus@...ba.org, cjashfor@...ux.vnet.ibm.com,
fweisbec@...il.com, linux-kernel@...r.kernel.org,
tglx@...utronix.de, andi@...stfloor.org
Subject: Re: [RFCv2 0/8] perf tool: Add new event group management
Hi,
On Mon, 16 Apr 2012 17:37:39 +0200, Jiri Olsa wrote:
> On Mon, Apr 16, 2012 at 05:26:02PM +0200, Peter Zijlstra wrote:
>> On Mon, 2012-04-16 at 14:16 +0200, Jiri Olsa wrote:
>> > group: group_spec '{' events '}'
>> > group_spec: name ':' mod | empty
>>
>> How about something like:
>>
>> groups: groups ',' group | group
>> group: group_name '{' events '}' group_mod
>> group_name: name ':' | empty
>> group_mod: ':' group_mods | empty
>> group_mods: [l]
>>
>>
>
> so you'd like something like:
> 'group1:{cycles,faults}:1
>
> hm, do you want the ':' behind the name? It does not seem necessary,
> but might be readable though.. ok :)
> 'group1{cycles,faults}:1
>
+1 for the last form :)
Thanks,
Namhyung
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists