From mboxrd@z Thu Jan 1 00:00:00 1970 From: Martin Schwidefsky Subject: Re: [S390] ftrace: update system call tracer support Date: Tue, 18 Aug 2009 12:22:39 +0200 Message-ID: <20090818122239.03ca8bcc@skybase> References: <1250016545-6601-1-git-send-email-fweisbec@gmail.com> <20090812091133.GA21655@elte.hu> <20090818004654.GA4402@linux-sh.org> <20090818073211.GA26670@elte.hu> <20090818085110.GA2074@elte.hu> <20090818105927.30f6a140@skybase> <20090818100526.GA14374@elte.hu> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Return-path: Received: from mtagate6.de.ibm.com ([195.212.29.155]:62048 "EHLO mtagate6.de.ibm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750880AbZHRKXX (ORCPT ); Tue, 18 Aug 2009 06:23:23 -0400 In-Reply-To: <20090818100526.GA14374@elte.hu> Sender: linux-next-owner@vger.kernel.org List-ID: To: Ingo Molnar Cc: Paul Mundt , Stephen Rothwell , Jason Baron , Frederic Weisbecker , LKML , Lai Jiangshan , Steven Rostedt , Peter Zijlstra , Mathieu Desnoyers , Jiaying Zhang , Martin Bligh , Li Zefan , Masami Hiramatsu , Wu Zhangjin , linux-next@vger.kernel.org, Heiko Carstens On Tue, 18 Aug 2009 12:05:26 +0200 Ingo Molnar wrote: > > * Martin Schwidefsky wrote: > > > On Tue, 18 Aug 2009 10:51:10 +0200 > > Ingo Molnar wrote: > > > > > > > > * Ingo Molnar wrote: > > > > > > > * Paul Mundt wrote: > > > > > > > > > [ Adding to Cc everyone that now has a broken tree thanks to this .. ] > > > > > > > > > > On Wed, Aug 12, 2009 at 11:11:33AM +0200, Ingo Molnar wrote: > > > > > > * Frederic Weisbecker wrote: > > > > > > > This pull request integrate one cleanup/fix for ftrace and an > > > > > > > update for syscall tracing: the migration from old-style tracer to > > > > > > > individual tracepoints/trace_events and the support for perf > > > > > > > counter. > > > > > > > > > > > > > > I've tested it with success either with ftrace (every syscall > > > > > > > tracepoints enabled at the same time without problems) and with > > > > > > > perfcounter. > > > > > > > > > > > > > > May be one drawback: it creates so much trace events that the > > > > > > > ftrace selftests can take some time :-) > > > > > > > > > > > > Pulled, thanks a lot! > > > > > > > > > > And this has now subsequently broken every single SH and S390 > > > > > configuration, [...] > > > > > > > > I test SH cross-builds regularly. I just checked the SH defconfig > > > > and it builds just fine here: > > > > > > > > $ make -j32 CROSS_COMPILE=sh3-linux- ARCH=sh vmlinux > > > > > > The s390 build indeed broke. (This got masked by the s390 toolchain > > > i'm using not having been able to build Linus's tree - i fixed > > > that.) > > > > > > Could you try the fix below? It does the trick here. > > > > > > Martin, Heiko - does the fix look good to you? regs->gprs[2] seems > > > to be the register used for both the syscall number (enter > > > callback) and for the return code (exit callback). > > > > Correct, for do_syscall_trace_{enter,exit} the code in entry.S > > stores the system call number in regs->gprs[2]. The fix is fine. > > Thanks Ingo. > > Thanks, i've added your Acked-by to the commit. I suspect you dont > want to pull tracing infrastructure changes into the S390 tree, so > keeping this fix in the tracing tree would be the best option? Indeed, the patch that introduced the api change is in the tracing tree so it makes sense to put the s390 adaptions there too. -- blue skies, Martin. "Reality continues to ruin my life." - Calvin.