Hi Frank, Thanks for the review. Unfortunately my git-fu is super weak, and I don't know how to re-spin the patches by simply editing the source file in my git tree and fixing these warnings and errors that checkpatch.pl is identifying. Can you either (a) help me with step-by-step instructions on how to do so in git (perhaps RTFM with a pointer to one), or (b) help fix up my 2 patches yourself in preparation for a review and a commit? Thanks - Bhavesh ----- Original Message ----- From: "Frank Rowand" <frank.rowand@xxxxxxxxxxx> To: "Frank Rowand" <Frank_Rowand@xxxxxxxxxxx> Cc: "Bhavesh Davda" <bhavesh@xxxxxxxxxx>, linux-rt-users@xxxxxxxxxxxxxxx, "John Kacur" <jkacur@xxxxxxxxxx> Sent: Monday, October 22, 2012 6:41:57 PM Subject: Re: [PATCH RT-TESTS] cyclictest: histogram overflow instance tracking Re: [PATCH RT-TESTS] cyclictest: histogram overflow instance tracking On 10/22/12 18:16, Frank Rowand wrote: > On 10/16/12 10:02, Bhavesh Davda wrote: >> From: Bhavesh Davda <bhavesh@xxxxxxxxxx> >> >> Add feature to cyclictest histogram mode to track cycle counts every time a >> sample overflows the histogram limit. This should help identify if there is a >> timing pattern to jitters in cyclictest runs. >> >> Example output (with -h 10): >> ... >> Histogram Overflows: 00001 00007 00000 00009 00004 00007 00000 00001 >> Histogram Overflow at cycle number: >> Thread 0: 09964 >> Thread 1: 00000 00004 00006 00008 00010 09962 11594 >> Thread 2: >> Thread 3: 01169 04698 06782 09033 10299 11561 21517 28734 29532 >> Thread 4: 11574 11580 11583 11586 >> Thread 5: 00020 09448 13954 14954 18954 20587 24973 >> Thread 6: >> Thread 7: 18950 >> ... >> >> Signed-off-by: Bhavesh Davda <bhavesh@xxxxxxxxxx> > > > After the patches survive checkpatch, I'll review and test them. < snip > > /home/frowand/tmp/hist_overflow_xxx_1 has style problems, please review. > > If any of these errors are false positives, please report > them to the maintainer, see CHECKPATCH in MAINTAINERS. Oooooops, the following message was about the series file I would use to apply the patches with quilt. Please ignore it. > ERROR: Does not appear to be a unified-diff format patch > > total: 1 errors, 0 warnings, 0 lines checked -Frank -- To unsubscribe from this list: send the line "unsubscribe linux-rt-users" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
