On Fri, Feb 28, 2014 at 10:57:48AM -0800, Andi Kleen wrote: > Don Zickus <[email protected]> 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 [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
