Re: [RFCv2 0/8] perf tool: Add new event group management

From: Namhyung Kim
Date: Mon Apr 16 2012 - 22:17:30 EST


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@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/