Re: [PATCH 2/2] perf data: Fix ctf_writer setupenv failure

From: Jiri Olsa
Date: Wed Apr 08 2015 - 14:00:10 EST


On Wed, Apr 08, 2015 at 12:49:20PM +0800, He Kuang wrote:
> Due to babeltrace commit:
> 7f800dc7c2a1 ("ir: make trace environment use bt_object")
>
> The trace->frozen flag is set in bt_ctf_trace_create_stream(), this flag
> is checked before adding environment field to trace, and causes
> ctf_writer__setup_env() failed. Fix this by setting all environment
> fields before bt_ctf_trace_create_stream().
>
> Before this patch:
> $ perf data convert --to-ctf=ctf
> Error during CTF convert setup.

have you tested with the latest babeltrace sources?
this reminds me the bug they fixed recently, CCing Jeremie

thanks,
jirka

>
> After this patch:
> $ perf data convert --to-ctf=ctf
> [ perf data convert: Converted 'perf.data' into CTF data 'ctf' ]
> [ perf data convert: Converted and wrote 0.023 MB (596 samples) ]
>
> Signed-off-by: He Kuang <hekuang@xxxxxxxxxx>
> ---
> tools/perf/util/data-convert-bt.c | 30 ++++++++++++++++++++----------
> 1 file changed, 20 insertions(+), 10 deletions(-)
>
> diff --git a/tools/perf/util/data-convert-bt.c b/tools/perf/util/data-convert-bt.c
> index a5b89b9..718dc8a 100644
> --- a/tools/perf/util/data-convert-bt.c
> +++ b/tools/perf/util/data-convert-bt.c
> @@ -604,6 +604,22 @@ static int setup_events(struct ctf_writer *cw, struct perf_session *session)
> return 0;
> }
>
> +static int ctf_writer__setup_stream(struct ctf_writer *cw)
> +{
> + struct bt_ctf_stream *stream;
> +
> + /* CTF stream instance */
> + stream = bt_ctf_writer_create_stream(cw->writer, cw->stream_class);
> + if (!stream) {
> + pr("Failed to create CTF stream.\n");
> + return -1;
> + }
> +
> + cw->stream = stream;
> +
> + return 0;
> +}
> +
> static int ctf_writer__setup_env(struct ctf_writer *cw,
> struct perf_session *session)
> {
> @@ -725,7 +741,6 @@ static int ctf_writer__init(struct ctf_writer *cw, const char *path)
> {
> struct bt_ctf_writer *writer;
> struct bt_ctf_stream_class *stream_class;
> - struct bt_ctf_stream *stream;
> struct bt_ctf_clock *clock;
>
> /* CTF writer */
> @@ -767,15 +782,6 @@ static int ctf_writer__init(struct ctf_writer *cw, const char *path)
> if (ctf_writer__init_data(cw))
> goto err_cleanup;
>
> - /* CTF stream instance */
> - stream = bt_ctf_writer_create_stream(writer, stream_class);
> - if (!stream) {
> - pr("Failed to create CTF stream.\n");
> - goto err_cleanup;
> - }
> -
> - cw->stream = stream;
> -
> /* CTF clock writer setup */
> if (bt_ctf_writer_add_clock(writer, clock)) {
> pr("Failed to assign CTF clock to writer.\n");
> @@ -830,6 +836,10 @@ int bt_convert__perf2ctf(const char *input, const char *path, bool force)
> if (ctf_writer__setup_env(cw, session))
> goto free_session;
>
> + /* CTF writer trace stream setup */
> + if (ctf_writer__setup_stream(cw))
> + goto free_session;
> +
> /* CTF events setup */
> if (setup_events(cw, session))
> goto free_session;
> --
> 2.3.3.220.g9ab698f
>
--
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/