Re: [PATCH 3/3] perf script: Align event name properly

From: Jiri Olsa
Date: Sun Dec 27 2015 - 06:59:16 EST


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@xxxxxxxxx>

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@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/