On Wed, Jul 29, 2015 at 05:21:40PM -0700, Andi Kleen wrote: > From: Andi Kleen <[email protected]> > > Now support CSV output for metrics. With the new output callbacks > this is relatively straight forward by creating new callbacks. > > The new line callback needs to know the number of fields to skip them > correctly > > To avoid reordering the existing CSV fields, I had to move > the noise printing into the lower level print metrics call back, > so that noise can be printed before metrics. > > This actually cleans up the callers because it avoids a lot > of duplicated code.
nice, please separate the cleanup from the actual addition of the metrics CSV fields could you please also start some documentation in perf stat man? I dont think we have anything yet.. and it's getting bigger ;-) thanks, jirka -- 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/
