Re: [PATCH 01/12] perf_events: add generic taken branch samplingsupport (v2)

From: Stephane Eranian
Date: Tue Dec 06 2011 - 14:42:16 EST


On Mon, Dec 5, 2011 at 1:06 PM, Peter Zijlstra <peterz@xxxxxxxxxxxxx> wrote:
> On Fri, 2011-10-14 at 14:37 +0200, Stephane Eranian wrote:
>> @@ -455,6 +483,8 @@ enum perf_event_type {
>> Â Â Â Â Â*
>> Â Â Â Â Â* Â Â Â{ u32 Â Â Â Â Â Â Â Â Â size;
>>     Â*    Âchar         Âdata[size];}&& PERF_SAMPLE_RAW
>> + Â Â Â Â*
>> + Â Â Â Â* Â Â Â{ u64 from, to, flags } lbr[nr];} && PERF_SAMPLE_BRANCH_STACK
>
> Â Â Â Â Â Â Â Â{ u64 Â nr;
> Â Â Â Â Â Â Â Â Â{ u64 from, to, flags } brstack[nr]; } && PERF_SAMPLE_BRANCH_STACK
>
> Perhaps? It looks like you lost a line somewhere, even you curly braces
> are unmatched in the correct way.

Yes, that the right way of describing the layout. Will fix that.
--
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/