Hi Jiri,
On Tue, 15 Oct 2013 16:27:32 +0200, Jiri Olsa wrote:
> This patch is adding 'struct perf_data_file' object as
> a placeholder for all attributes regarding perf.data
> file handling. Changing perf_session__new to take it
> as an argument.
>
> The rest of the functionality will be added later to keep
> this change simple enough, because all the places using
> perf_session are changed now.
All three look good.
Btw, are you planning to support multiple per-cpu file record? As you
know I suggested perf.data.dir approach in my perf-ftrace patchset (I'll
resend a new version soonish) something like below. What do you think?
perf.data.dir/
|-- perf-cpu0.data
|-- perf-cpu1.data
|-- perf-cpu2.data
`-- perf-cpu3.data
Maybe we could split sample data and other data (e.g. COMM, MMAP or some
user data?) to another file(s).
Thanks,
Namhyung
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
