From mboxrd@z Thu Jan 1 00:00:00 1970 From: Arnaldo Carvalho de Melo Subject: Re: [PATCH 001/001] perf: let trace write to stderr by default Date: Wed, 5 Aug 2015 16:50:06 -0300 Message-ID: <20150805195006.GB32553@kernel.org> References: <3305230.aOMPB1fPqd@milian-kdab2> <3494762.Lck5LtIdUB@milian-kdab2> Mime-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: Received: from casper.infradead.org ([85.118.1.10]:38952 "EHLO casper.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752166AbbHETun (ORCPT ); Wed, 5 Aug 2015 15:50:43 -0400 Content-Disposition: inline In-Reply-To: <3494762.Lck5LtIdUB@milian-kdab2> Sender: linux-perf-users-owner@vger.kernel.org List-ID: To: Milian Wolff Cc: linux-perf-users Em Tue, Aug 04, 2015 at 07:14:22PM +0200, Milian Wolff escreveu: > On Tuesday 04 August 2015 13:50:14 Arnaldo Carvalho de Melo wrote: > > Le 4 ao=FBt 2015 1:46 PM, "Milian Wolff" a =E9cri= t : > > > See attached trivial patch which brings trace a bit closer to str= ace. > > >=20 > > > I've never published a patch to the Kernel yet, I hope it's OK as= -is. I > >=20 > > can > >=20 > > > amend it at will if you have any requests to the style > >=20 > > Start by not using attachments :-) >=20 > :) Second step, read Documentation/email-clients.txt, you are using Kmail and it is mangling the patches, I'll apply the patch by hand this time. - Arnaldo =20 > >From 188476d65bab17ad99f5bdf65294160ef2d5f49b Mon Sep 17 00:00:00 20= 01 > From: Milian Wolff > Date: Tue, 4 Aug 2015 18:30:52 +0200 > Subject: [PATCH] perf: let trace write to stderr by default >=20 > Without this patch, it is cumbersome to read the trace output > but ignoring the normal, potentially verbose, output of the debuggee. > One common example is doing something like the following: >=20 > perf trace -s find /tmp > /dev/null >=20 > Without this patch, the trace summary will be lost. Now, it will > still be printed at the end. This behavior is also applied by strace. > --- > tools/perf/builtin-trace.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) >=20 > diff --git a/tools/perf/builtin-trace.c b/tools/perf/builtin-trace.c > index 39ad4d0..d183d88 100644 > --- a/tools/perf/builtin-trace.c > +++ b/tools/perf/builtin-trace.c > @@ -2778,7 +2778,7 @@ int cmd_trace(int argc, const char **argv, cons= t char=20 > *prefix __maybe_unused) > .mmap_pages =3D UINT_MAX, > .proc_map_timeout =3D 500, > }, > - .output =3D stdout, > + .output =3D stderr, > .show_comm =3D true, > .trace_syscalls =3D true, > }; > --=20 > 2.5.0 >=20 > --=20 > Milian Wolff > mail@milianw.de > http://milianw.de