Messages in this thread | | | Date | Mon, 20 Apr 2015 17:41:29 -0300 | From | Arnaldo Carvalho de Melo <> | Subject | Re: [PATCH 2/7] perf data: Switch to multiple cpu stream files |
| |
Em Mon, Apr 20, 2015 at 10:19:44PM +0200, Jiri Olsa escreveu: > On Mon, Apr 20, 2015 at 04:58:54PM -0300, Arnaldo Carvalho de Melo wrote: > > Em Sat, Apr 18, 2015 at 05:50:15PM +0200, Jiri Olsa escreveu: > > > From: Sebastian Andrzej Siewior <bigeasy@linutronix.de> > > > > > > Currently we store the data into single data strea/file. The cpu > > > if data is stored within the event sample. The lttng puts the CPU > > > number that belongs to the event into the packet context instead > > > into the event. > > > > > > This patch makes sure that the trace produce by perf does look the > > > same way. We now use one stream per-CPU. Having it all in one stream > > > increased the total size of the resulting file. The test went from > > > 416KiB (with perf_cpu event member) to 24MiB due to the required > > > (and pointless) flush. With the per-cpu streams the total size went > > > up to 588KiB. > > > > > > Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de> > > > Link: http://lkml.kernel.org/n/tip-fco07vxi6yx4m9et2aimm4u0@git.kernel.org > > > Signed-off-by: Jiri Olsa <jolsa@kernel.org> > > > > With latest babeltrace.git: > > > > [acme@zoo babeltrace]$ git log --oneline | head -1 > > 48d711a204f6 Fix: Missing stdint.h in stream-class.h > > hum, this is rather old one (Mar 10th), current is: > > [jolsa@krava babeltrace]$ git show --oneline | head -1 > d8f190b24d52 Add tests for bt_ctf_field_string_append_len()
I did a git remove update...
Lemme double check:
[acme@zoo babeltrace]$ git remote update Fetching origin [acme@zoo babeltrace]$ git remote -v origin git://git.efficios.com/babeltrace.git (fetch) origin git://git.efficios.com/babeltrace.git (push) [acme@zoo babeltrace]$ git log --oneline | head -1 48d711a204f6 Fix: Missing stdint.h in stream-class.h [acme@zoo babeltrace]$
What is wrong here? Is it in a different branch?
/me checks again that... Yeah, I probably created my local branch from some other branch, now that I did:
[acme@zoo babeltrace]$ git checkout -b master origin/master Branch master set up to track remote branch master from origin. Switched to a new branch 'master' [acme@zoo babeltrace]$ git log --oneline | head -1 d8f190b24d52 Add tests for bt_ctf_field_string_append_len() [acme@zoo babeltrace]$
will try again those patches...
- ARnaldo
| |