Re: [PATCH 12/15] perf tools: allow non-matching sample types

[Stephane Eranian]

On Tue, Jun 25, 2013 at 5:56 PM, David Ahern <dsahern@xxxxxxxxx> wrote:
> On 6/24/13 7:16 AM, Adrian Hunter wrote:
>>
>> Sample types need not be identical to determine
>> the sample id from the event.  Only the position
>> of the sample id needs to be the same.
>>
>> Compatible sample types are ones in which the bits
>> defined by PERF_COMPAT_MASK are the same.
>> 'perf_evlist__config()' forces sample types to be
>> compatible on that basis.
>
>
> Something is still missing to support different sample_types for events.
> Consider the case (S/W event + tracepoint):
>   perf record -e cs -c1 -e sched:sched_switch -a -- sleep 1
>   perf script
>
> This patch addresses the sample_type mismatch error message, but I get no
> event samples in the output. Besides the usual header I get:
>
>   No trace sample to read. Did you call 'perf record -R'?
>
> It's a 16-cpu box with 5 VMs running. I know there should be at least a few
> samples in 1 second.
>
> Stephane: are you looking at allowing sample_types per event?
>
Yes, this is what I need. I have a kernel patch to do this. I don't
know how to update perf to handle it correctly. So maybe you can
help. My patch is useful to drastically reduce the size of the perf.data
file in case we use the branch-stack with lots of events which is
what our Gooda tool would like to do.
--
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/