Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754541AbbDTUlg (ORCPT ); Mon, 20 Apr 2015 16:41:36 -0400 Received: from mail.kernel.org ([198.145.29.136]:38331 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754458AbbDTUle (ORCPT ); Mon, 20 Apr 2015 16:41:34 -0400 Date: Mon, 20 Apr 2015 17:41:29 -0300 From: Arnaldo Carvalho de Melo To: Jiri Olsa Cc: Jiri Olsa , David Ahern , Frederic Weisbecker , He Kuang , Ingo Molnar , Jeremie Galarneau , Namhyung Kim , Paul Mackerras , Peter Zijlstra , Sebastian Andrzej Siewior , Tom Zanussi , Wang Nan , lkml Subject: Re: [PATCH 2/7] perf data: Switch to multiple cpu stream files Message-ID: <20150420204129.GP11111@kernel.org> References: <1429372220-6406-1-git-send-email-jolsa@kernel.org> <1429372220-6406-3-git-send-email-jolsa@kernel.org> <20150420195854.GJ11111@kernel.org> <20150420201944.GD10644@krava.redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20150420201944.GD10644@krava.redhat.com> X-Url: http://acmel.wordpress.com User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2562 Lines: 64 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 > > > > > > 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 > > > Link: http://lkml.kernel.org/n/tip-fco07vxi6yx4m9et2aimm4u0@git.kernel.org > > > Signed-off-by: Jiri Olsa > > > > 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 -- 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/