From: Masami Hiramatsu <masami.hiramatsu.pt@hitachi.com>
To: Steven Rostedt <rostedt@goodmis.org>
Cc: Shuah Khan <shuah.kh@samsung.com>,
Tom Zanussi <tom.zanussi@linux.intel.com>,
Oleg Nesterov <oleg@redhat.com>,
Namhyung Kim <namhyung@kernel.org>,
Fengguang Wu <fengguang.wu@intel.com>,
Ingo Molnar <mingo@kernel.org>,
Shuah Khan <shuahkh@osg.samsung.com>,
Linux Kernel Mailing List <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH v5 1/4] ftracetest: Initial commit for ftracetest
Date: Wed, 24 Sep 2014 10:16:45 +0900 [thread overview]
Message-ID: <54221B7D.3010104@hitachi.com> (raw)
In-Reply-To: <20140923093610.4f404274@gandalf.local.home>
(2014/09/23 22:36), Steven Rostedt wrote:
> For future reference:
>
> On Mon, 22 Sep 2014 23:42:50 +0000
> Masami Hiramatsu <masami.hiramatsu.pt@hitachi.com> wrote:
>
>> ftracetest is a collection of testcase shell-scripts for ftrace.
>> To avoid regressions of ftrace, these testcases check correct
>> ftrace behaviors. If someone would like to add any features on
>> ftrace, the patch series should have at least one testcase for
>> checking the new behavior.
>>
>> Changes in v5:
>> - Fix a TODO item, since this already integrated to kselftest.
>> - Print the test name at the first line.
>>
>> Changes in v4:
>> - Move this under selftests. :)
>> - Add a copyright and note of GPLv2.
>> - Fix documents acconding to Steven's comments.
>> - Fix a small bug pointed by Namhyng.
>>
>> Changes in v3:
>> - Use "." instead of "source".
>> - Don't use -e option for echo since dash doesn't support it.
>>
>> Changes in v2:
>> - Remove unneeded 'function' keyword.
>> - Add abspath and find_testcases.
>> - Make OPT_TEST_CASES a local var.
>> - Ensure given testcase ended with .tc.
>> - Accept a directory option which has multiple testcases.
>> - Change [PASSED]/[FAILED] to [PASS]/[FAIL]
>> - Change the basic shell to sh (dash).
>
> Place the version changes after the '---'. They are great for patch
> series, but we prefer not to have them in the git logs. The "Link:" tag
> should allow people to find the version history of patches if they are
> interested. By placing the version updates after the '---', git will
> ignore them when pulling them in.
Ah, I see :)
For the next series, I'll do that.
>
>>
>> Signed-off-by: Masami Hiramatsu <masami.hiramatsu.pt@hitachi.com>
>> Acked-by: Shuah Khan <shuahkh@osg.samsung.com>
>> ---
>
> Also, please have the Signed-off-by be the last tag, otherwise it
> screws up my scripts ;-)
OK.
Thank you!
--
Masami HIRAMATSU
Software Platform Research Dept. Linux Technology Research Center
Hitachi, Ltd., Yokohama Research Laboratory
E-mail: masami.hiramatsu.pt@hitachi.com
next prev parent reply other threads:[~2014-09-24 1:16 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-09-22 23:42 [PATCH v5 0/4] ftrace: Add a ftrace test collection Masami Hiramatsu
2014-09-22 23:42 ` [PATCH v5 1/4] ftracetest: Initial commit for ftracetest Masami Hiramatsu
2014-09-23 13:36 ` Steven Rostedt
2014-09-24 1:16 ` Masami Hiramatsu [this message]
2014-09-22 23:42 ` [PATCH v5 2/4] ftracetest: Add ftrace basic testcases Masami Hiramatsu
2014-09-22 23:42 ` [PATCH v5 3/4] ftracetest: Add kprobe " Masami Hiramatsu
2014-09-22 23:42 ` [PATCH v5 4/4] ftracetest: Add POSIX.3 standard and XFAIL result codes Masami Hiramatsu
2014-09-23 0:32 ` [PATCH v5 0/4] ftrace: Add a ftrace test collection Namhyung Kim
2014-09-23 1:03 ` Steven Rostedt
2014-09-23 1:17 ` Namhyung Kim
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=54221B7D.3010104@hitachi.com \
--to=masami.hiramatsu.pt@hitachi.com \
--cc=fengguang.wu@intel.com \
--cc=linux-kernel@vger.kernel.org \
--cc=mingo@kernel.org \
--cc=namhyung@kernel.org \
--cc=oleg@redhat.com \
--cc=rostedt@goodmis.org \
--cc=shuah.kh@samsung.com \
--cc=shuahkh@osg.samsung.com \
--cc=tom.zanussi@linux.intel.com \
/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
Powered by JetHome