[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <20151227115908.GC12145@krava.brq.redhat.com>
Date: Sun, 27 Dec 2015 12:59:08 +0100
From: Jiri Olsa <jolsa@...hat.com>
To: David Ahern <dsahern@...il.com>
Cc: Jiri Olsa <jolsa@...nel.org>,
Arnaldo Carvalho de Melo <acme@...nel.org>,
lkml <linux-kernel@...r.kernel.org>,
Ingo Molnar <mingo@...nel.org>,
Namhyung Kim <namhyung@...nel.org>,
Peter Zijlstra <a.p.zijlstra@...llo.nl>
Subject: Re: [PATCH 3/3] perf script: Align event name properly
On Thu, Nov 26, 2015 at 07:25:17AM -0700, David Ahern wrote:
> On 11/26/15 6:55 AM, Jiri Olsa wrote:
> >Adding code to align event names, so we get aligned output
> >in case of multiple events with different names.
> >
> >Before:
> > $ perf script
> > :13757 13757 163918.230829: cpu/mem-snp-none/P: ffff88085f20d010
> > :13757 13757 163918.230832: cpu/mem-loads,ldlat=30/P: 7f5a5f719f00
> > :13757 13757 163918.230835: cpu/mem-loads,ldlat=30/P: 7f5a5f719f00
> > :13758 13758 163918.230838: cpu/mem-snp-none/P: ffff88085f4ad810
> > :13758 13758 163918.154093: cpu/mem-stores/P: ffff88085bb53f28
> > :13757 13757 163918.155264: cpu/mem-snp-hitm/P: 601080
> > ...
> >
>
> What's up with the ":13757" for the process name?
>
> Patches LGTM
>
> Acked-by: David Ahern <dsahern@...il.com>
this one was missed, I pushed it on the top of my perf/fixes
branch with David's ack
thanks,
jirka
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists