Hi David,

On Wed,  8 May 2013 22:31:39 -0600, David Ahern wrote:
> Handles initializations typically done as part of processing the file
> header and HEADER_TRACING_DATA event.
>
[SNIP]
> +int perf_evlist__trace_init(struct perf_evlist *evlist,
> +                         struct perf_session *session)
> +{
> +     struct tracing_data *tdata;
> +     char temp_file[] = "/tmp/perf-XXXXXXXX";
> +     int fd;
> +
> +     fd = mkstemp(temp_file);
> +     if (fd < 0) {
> +             pr_err("mkstemp failed\n");
> +             return -1;
> +     }
> +     unlink(temp_file);
> +
> +     tdata = tracing_data_get(&evlist->entries, fd, false);
> +     if (!tdata)
> +             return -1;
> +
> +     lseek(fd, 0, SEEK_SET);
> +     (void) trace_report(fd, &session->pevent, false);
> +     tracing_data_put(tdata);

I guess you need to close the fd here.

> +
> +     return perf_evlist__prepare_tracepoint_events(evlist, session->pevent);
> +}

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

Reply via email to