[lkml]   [2015]   [Apr]   [9]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: [PATCH 1/2] perf data: Show error message when ctf setup failed
On 2015-04-09 05:46 AM, Jiri Olsa wrote:
> On Thu, Apr 09, 2015 at 04:19:20PM +0800, He Kuang wrote:
>> Hi, jirka
>> On 2015/4/9 1:45, Jiri Olsa wrote:
>>> On Wed, Apr 08, 2015 at 12:49:19PM +0800, He Kuang wrote:
>>>> Show message when errors occurred during ctf conversion setup.
>>>> Before this patch:
>>>> $ ./perf data convert --to-ctf=ctf
>>>> $ echo $?
>>>> 255
>>>> After this patch:
>>>> $ ./perf data convert --to-ctf=ctf
>>>> Error during CTF convert setup.
>>> so I have like 5 more patches from the original CTF set
>>> which I'm holding until all works with tracecompass:
>>> Is it working for you? How do you test resulted CTF data?
>>> anyway the patch looks ok, just small nit below
>> I tested by using babeltrace binary and it works.
>> After receiving your reply, I test on the latest tracecompass. A
>> folder named 'ctf' is showed instead of the expected file
>> 'ctf-data', this folder only contains the raw metadata and
>> perf-stream files but not analysed.
> CC-ing Alexandre from tracecompass devel ^^^


I just came back from vacation, sorry for not replying earlier!

I managed to compile perf with CTF support, but by using Babeltrace's
commit 5584a48. It fails to compile against current master, because of
private headers getting exposed. I reported that to the BT maintainers.

Then it seems there's another bug with Trace Compass's current master,
trace validation cannot fail, and any file will get imported with no
errors. We will look into this.
But the root of the problem was that the converted CTF trace was not
being recognized as valid. This is because some events define "stream_id
= 0;", and others don't specify a stream_id at all. It seems quite
random, see the full metadata here:

Is there a reason why some events specify a stream_id and some don't?

We could patch Trace Compass to accept it, since Babeltrace does. But
it's not very clear according to the spec, I'll check with the CTF guys
if it should be considered valid or not.


> jirka

 \ /
  Last update: 2015-04-09 17:21    [W:0.085 / U:1.984 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site