Date: Thu, 9 Apr 2015 11:46:54 +0200
From: Jiri Olsa <jolsa@redhat.com>
To: He Kuang <hekuang@huawei.com>
Cc: bigeasy@linutronix.de, jolsa@kernel.org, acme@kernel.org,
        a.p.zijlstra@chello.nl, mingo@redhat.com, wangnan0@huawei.com,
        linux-kernel@vger.kernel.org,
        Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Subject: Re: [PATCH 1/2] perf data: Show error message when ctf setup failed
Message-ID: <20150409094654.GA1321@krava.brq.redhat.com>
References: <1428468560-27401-1-git-send-email-hekuang@huawei.com>
 <20150408174510.GA20250@krava.redhat.com>
 <55263608.2070301@huawei.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <55263608.2070301@huawei.com>
User-Agent: Mutt/1.5.23 (2014-03-12)
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1319
Lines: 37

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:
> >   http://marc.info/?l=linux-kernel&m=142736197610573&w=2
> >
> >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 ^^^

jirka
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/