Re: [PATCH 00/19 V2] perf, c2c: Add new tool to analyze cacheline contention on NUMA systems

From: Don Zickus
Date: Fri Feb 28 2014 - 14:43:01 EST


On Fri, Feb 28, 2014 at 10:57:48AM -0800, Andi Kleen wrote:
> Don Zickus <dzickus@xxxxxxxxxx> writes:
> >
> > A handful of patches include re-enabling MMAP2 support and some fixes
> > to perf itself.
>
> I would suggest to pursue the lone kernel patch separately. Hopefully
> that can be merged soon, once the remainin problems with that are
> addressed.

Good point.

>
> >
> > Comemnts, feedback, anything else welcomed.
>
> As a high level comment, can you add support for CSV mode?
> (-x, in other tools)

It's there. :-)

perf c2c -r -x, report

will spit out raw records in CSV format.

>
> I assume most people would data mine the output in some form,
> and that's much easier with a more machine oriented format.
>
> Also you should probably have the standard perf conventions
> for output fds (--log-fd, --output, default to stderr). otherwise
> the output may often be lost.

Ok, I'll look into what that entails.

Thanks!

Cheers,
Don
--
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/