Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753250AbbDXCHX (ORCPT ); Thu, 23 Apr 2015 22:07:23 -0400 Received: from lgeamrelo02.lge.com ([156.147.1.126]:56366 "EHLO lgeamrelo02.lge.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753168AbbDXCHV (ORCPT ); Thu, 23 Apr 2015 22:07:21 -0400 X-Original-SENDERIP: 10.177.220.203 X-Original-MAILFROM: namhyung@kernel.org Date: Fri, 24 Apr 2015 11:02:18 +0900 From: Namhyung Kim To: Arnaldo Carvalho de Melo Cc: Ingo Molnar , linux-kernel@vger.kernel.org, Adrian Hunter , Bobby Powers , Borislav Petkov , David Ahern , Dirk Gouders , Don Zickus , Frederic Weisbecker , Jiri Olsa , Joonsoo Kim , linux-kbuild@vger.kernel.org, Michael Petlan , Michal Marek , Minchan Kim , Paul Mackerras , Peter Zijlstra , Stephane Eranian , Will Deacon , Arnaldo Carvalho de Melo , Javi Merino , Steven Rostedt Subject: Re: [GIT PULL 0/4] perf/urgent fixes Message-ID: <20150424020218.GF1905@sejong> References: <1429826590-32193-1-git-send-email-acme@kernel.org> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <1429826590-32193-1-git-send-email-acme@kernel.org> 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: 4350 Lines: 122 Hi Arnaldo, I've set up some docker containers for build test, and found a couple of failures.. It seems David's kmem build fix ("perf kmem: Fix compiles on RHEL6/OL6") which is in your perf/core branch also needs to be in perf/urgent. Sorry about the kmem breakages.. And I also found this.. >From 581ae7f48c89377755391c3f95637a1d48eefc73 Mon Sep 17 00:00:00 2001 From: Namhyung Kim Date: Fri, 24 Apr 2015 10:45:16 +0900 Subject: [PATCH] tools lib traceevent: Fix build failure on 32-bit arch In my i386 build, it failed like this: CC event-parse.o event-parse.c: In function 'print_str_arg': event-parse.c:3868:5: warning: format '%lu' expects argument of type 'long unsigned int', but argument 3 has type 'uint64_t' [-Wformat] Cc: Javi Merino Signed-off-by: Namhyung Kim --- tools/lib/traceevent/event-parse.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/lib/traceevent/event-parse.c b/tools/lib/traceevent/event-parse.c index 12a7e2a40c89..aa21bd55bd8a 100644 --- a/tools/lib/traceevent/event-parse.c +++ b/tools/lib/traceevent/event-parse.c @@ -3865,7 +3865,7 @@ static void print_str_arg(struct trace_seq *s, void *data, int size, } else if (el_size == 4) { trace_seq_printf(s, "%u", *(uint32_t *)num); } else if (el_size == 8) { - trace_seq_printf(s, "%lu", *(uint64_t *)num); + trace_seq_printf(s, "%"PRIu64, *(uint64_t *)num); } else { trace_seq_printf(s, "BAD SIZE:%d 0x%x", el_size, *(uint8_t *)num); -- 2.3.4 Thanks, Namhyung On Thu, Apr 23, 2015 at 07:03:06PM -0300, Arnaldo Carvalho de Melo wrote: > Hi Ingo, > > Please consider pulling, > > - Arnaldo > > The following changes since commit 0140e6141e4f1d4b15fb469e6912b0e71b7d1cc2: > > perf/x86/intel/uncore: Move PCI IDs for IMC to uncore driver (2015-04-22 08:29:19 +0200) > > are available in the git repository at: > > git://git.kernel.org/pub/scm/linux/kernel/git/acme/linux.git tags/perf-urgent-for-mingo > > for you to fetch changes up to de28c15daf60e9625bece22f13a091fac8d05f1d: > > tools lib api: Undefine _FORTIFY_SOURCE before setting it (2015-04-23 17:08:23 -0300) > > ---------------------------------------------------------------- > perf/urgent fixes: > > User visible: > > - Enable events when doing system wide 'trace' and starting a > workload, e.g: > > # trace -a sleep 1 > > now it matches the pattern in 'record' and will show envents > instead of sitting doing nothing while waiting for the started > workload to finish (Arnaldo Carvalho de Melo) > > - Disable and drain events when forked 'trace' workload ends > making sure we notice the end of the workload instead of > trying to keep up with the seemingly neverending flux of > system wide events (Arnaldo Carvalho de Melo) > > Infrastructure: > > - Fix the build on 32-bit ARM by consistently use PRIu64 for printing u64 > values in 'perf kmem' (Will Deacon) > > - Undefine _FORTIFY_SOURCE before setting it in tools/perf/api, fixing the build on > Hardened Gentoo systems (Bobby Powers) > > Signed-off-by: Arnaldo Carvalho de Melo > > ---------------------------------------------------------------- > Arnaldo Carvalho de Melo (2): > perf trace: Enable events when doing system wide tracing and starting a workload > perf trace: Disable events and drain events when forked workload ends > > Bobby Powers (1): > tools lib api: Undefine _FORTIFY_SOURCE before setting it > > Will Deacon (1): > perf kmem: Consistently use PRIu64 for printing u64 values > > tools/lib/api/Makefile | 2 +- > tools/perf/builtin-kmem.c | 4 ++-- > tools/perf/builtin-trace.c | 10 ++++++++-- > 3 files changed, 11 insertions(+), 5 deletions(-) > -- > 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/ -- 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/