From: Bhavesh Davda <bhavesh@vmware.com>
To: frank rowand <frank.rowand@am.sony.com>
Cc: linux-rt-users@vger.kernel.org, John Kacur <jkacur@redhat.com>,
Frank Rowand <Frank_Rowand@sonyusa.com>
Subject: Re: [PATCH RT-TESTS] cyclictest: histogram overflow instance tracking
Date: Tue, 23 Oct 2012 11:59:19 -0700 (PDT) [thread overview]
Message-ID: <801391694.11621598.1351018758999.JavaMail.root@vmware.com> (raw)
In-Reply-To: <5085F5E5.3090204@am.sony.com>
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@am.sony.com>
To: "Frank Rowand" <Frank_Rowand@sonyusa.com>
Cc: "Bhavesh Davda" <bhavesh@vmware.com>, linux-rt-users@vger.kernel.org, "John Kacur" <jkacur@redhat.com>
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@vmware.com>
>>
>> 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@vmware.com>
>
>
> 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
next prev parent reply other threads:[~2012-10-23 18:59 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-10-16 17:02 [PATCH RT-TESTS] cyclictest: histogram overflow instance tracking Bhavesh Davda
2012-10-23 1:16 ` Frank Rowand
2012-10-23 1:41 ` Frank Rowand
2012-10-23 18:59 ` Bhavesh Davda [this message]
2012-10-23 19:25 ` John Kacur
2012-10-23 19:55 ` Frank Rowand
2012-10-23 19:59 ` Sven-Thorsten Dietrich
2012-10-25 21:36 ` Frank Rowand
2012-10-25 22:05 ` Frank Rowand
2012-11-13 23:41 ` Frank Rowand
2012-11-13 23:41 ` Frank Rowand
2012-11-13 23:48 ` Bhavesh Davda
2012-11-13 23:53 ` John Kacur
2012-11-14 0:02 ` Frank Rowand
2012-11-14 0:03 ` Bhavesh Davda
2012-11-14 0:04 ` Bhavesh Davda
2012-11-14 0:09 ` John Kacur
2012-11-14 0:17 ` Bhavesh Davda
2012-11-14 0:19 ` John Kacur
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=801391694.11621598.1351018758999.JavaMail.root@vmware.com \
--to=bhavesh@vmware.com \
--cc=Frank_Rowand@sonyusa.com \
--cc=frank.rowand@am.sony.com \
--cc=jkacur@redhat.com \
--cc=linux-rt-users@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).