Re: [PATCHv4 0/9] perf tool: parser generator for events parsing

From: Jiri Olsa
Date: Wed Feb 15 2012 - 04:05:32 EST


On Tue, Feb 14, 2012 at 09:57:10PM +0100, Peter Zijlstra wrote:
> On Tue, 2012-02-14 at 21:20 +0100, Peter Zijlstra wrote:
> > OK, this one works. the attributes thing wants NULL termination.
> > Thanks
> > for mentioning that Jiri.
>
> OK, it mostly works, except:
>
> perf record -e "cpu/config=0x108000c0/upp" ./loop_1b_locks
>
> doesn't seem to work...
>
> One thing that might be nice is not having to do =1 for single bit
> fields, so that we can write: inv,edge instead of inv=1,edge=1.

ok, would you think there's any use for following behaviour:
whenever field is written without the assignment, all the field defined
bits are set.. regardless if it's sengle ot multiple bit field..

looks like this would ease the implementation a little,
not mentioning the above behaviour could be usefull ;)

jirka
--
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/