On Tue, 2010-06-22 at 14:32 -0700, Arun Sharma wrote: > On Thu, May 27, 2010 at 10:54 AM, Arun Sharma <[email protected]> wrote: > > > > On Tue, May 18, 2010 at 10:49 AM, Lucas Meneghel Rodrigues > > <[email protected]> wrote: > > > > Hi Lucas, > > > > > > > > Hi Arun, thanks for your work on those profilers, it's high time we > > > start exploring perf for profiling purposes. I verified that the cpistat > > > utility that you've contributed on this patch follows different coding > > > style rules than the rest of autotest, maybe because the script already > > > exists on its own right on another project/repository. Is that the case? > > > I pointed out the places where the code does not comply with the defined > > > autotest coding style: > > > > Thanks for looking into this. The patch to perf that adds --sort cpu > > option isn't upstream yet. Once that's committed, I'll fixup the style > > issues and resend. GPLv2 is fine. > > Incremental patch to address your review comments attached.
Could you send a full patch against the latest trunk please? > -Arun _______________________________________________ Autotest mailing list [email protected] http://test.kernel.org/cgi-bin/mailman/listinfo/autotest
