From: Ken Moffat <zarniwhoop@ntlworld.com>
To: Joe Perches <joe@perches.com>
Cc: josh@joshtriplett.org, Dave Chinner <david@fromorbit.com>,
Steven Rostedt <rostedt@goodmis.org>,
paulmck@linux.vnet.ibm.com, linux-kernel@vger.kernel.org,
mingo@kernel.org, laijs@cn.fujitsu.com, dipankar@in.ibm.com,
akpm@linux-foundation.org, mathieu.desnoyers@efficios.com,
niv@us.ibm.com, tglx@linutronix.de, peterz@infradead.org,
dhowells@redhat.com, edumazet@google.com, dvhart@linux.intel.com,
fweisbec@gmail.com, oleg@redhat.com, sbw@mit.edu
Subject: Re: [PATCH RFC 1/2] MAINTAINERS: Add "R:" designated-reviewers tag
Date: Wed, 4 Jun 2014 00:48:16 +0100 [thread overview]
Message-ID: <20140603234816.GB26144@milliways> (raw)
In-Reply-To: <1401754325.7323.50.camel@joe-AO725>
On Mon, Jun 02, 2014 at 05:12:05PM -0700, Joe Perches wrote:
>
> "Tested-by:" tags would be more helpful if the test
> cases that were used were somehow sent along with the
> signature.
>
To me, that seems either a perverse, or else a bureaucratic,
interpretation of what should go where.
Tested-by is usually used for a fix of some problem, often a
regression. A good commit message will explain the problem. I have
recently offered this tag in two cases - in the first case it did
not boot without the fix, in the second it did not wake up from
suspend. In each case, only one of my boxes was affected. Do you
think I should have insisted that some of my lspci -V information
was appended to the commit (they both affected the radeon code) if
my tag was going to be added ?
This is _often_ not like userspace programs where you can write a
testsuite to exercise the corner cases. Kernel problems can be
tied up with intricate details of the hardware, or equally they
might happen only for certain usage, and for those it might not be
at all obvious what is "special" about the affected usage.
ĸen
--
Nanny Ogg usually went to bed early. After all, she was an old lady.
Sometimes she went to bed as early as 6 a.m.
next prev parent reply other threads:[~2014-06-03 23:48 UTC|newest]
Thread overview: 64+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-06-02 17:00 [PATCH RFC 1/2] MAINTAINERS: Add "R:" designated-reviewers tag Paul E. McKenney
2014-06-02 17:00 ` [PATCH RFC 2/2] rcu: Add Josh Triplett as designated reviewer Paul E. McKenney
2014-06-02 20:35 ` Andrew Morton
2014-06-02 20:36 ` Joe Perches
2014-06-02 20:38 ` Randy Dunlap
2014-06-03 0:02 ` josh
2014-06-03 1:07 ` Randy Dunlap
2014-06-03 1:51 ` Josh Triplett
2014-06-03 3:11 ` Joe Perches
2014-06-03 5:10 ` Josh Triplett
2014-06-03 5:21 ` Joe Perches
2014-06-03 17:21 ` Randy Dunlap
2014-06-02 17:22 ` [PATCH RFC 1/2] MAINTAINERS: Add "R:" designated-reviewers tag Joe Perches
2014-06-02 17:29 ` Steven Rostedt
2014-06-02 17:34 ` Joe Perches
2014-06-02 17:48 ` josh
2014-06-02 17:59 ` Joe Perches
2014-06-02 18:12 ` Josh Boyer
2014-06-02 18:15 ` Joe Perches
2014-06-02 18:16 ` Paul E. McKenney
2014-06-02 18:44 ` Joe Perches
2014-06-02 18:50 ` Steven Rostedt
2014-06-02 18:55 ` josh
2014-06-02 19:05 ` Joe Perches
2014-06-02 19:09 ` josh
2014-06-02 19:17 ` Joe Perches
2014-06-02 23:19 ` Dave Chinner
2014-06-02 23:24 ` Andrew Morton
2014-06-03 0:35 ` Steven Rostedt
2014-06-02 23:59 ` josh
2014-06-03 0:12 ` Joe Perches
2014-06-03 23:48 ` Ken Moffat [this message]
2014-06-04 0:03 ` Steven Rostedt
2014-06-04 0:33 ` Joe Perches
2014-06-03 1:11 ` Dave Chinner
2014-06-03 1:30 ` Steven Rostedt
2014-06-03 7:16 ` Dave Chinner
2014-06-03 13:24 ` Mathieu Desnoyers
2014-06-03 15:54 ` Paul E. McKenney
2014-06-03 17:43 ` Steven Rostedt
2014-06-03 18:05 ` Randy Dunlap
2014-06-03 20:52 ` Theodore Ts'o
2014-06-03 21:46 ` Steven Rostedt
2014-06-03 22:08 ` josh
2014-06-05 4:01 ` Dave Chinner
2014-06-05 21:14 ` Frank Rowand
2014-06-02 19:26 ` Paul E. McKenney
2014-06-02 20:41 ` Dipankar Sarma
2014-06-02 19:07 ` Paul E. McKenney
2014-06-02 18:56 ` josh
2014-06-02 19:08 ` Paul E. McKenney
2014-06-02 19:11 ` josh
2014-06-02 19:27 ` Paul E. McKenney
2014-06-02 19:36 ` Joe Perches
2014-06-02 19:40 ` Randy Dunlap
2014-06-02 20:29 ` josh
2014-06-02 19:50 ` Paul E. McKenney
2014-06-02 19:55 ` Joe Perches
2014-06-02 20:07 ` Paul E. McKenney
2014-06-02 20:25 ` Mathieu Desnoyers
2014-06-03 15:37 ` Paul E. McKenney
2014-06-03 16:16 ` Steven Rostedt
2014-06-03 16:25 ` Paul E. McKenney
2014-06-04 1:35 ` Lai Jiangshan
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=20140603234816.GB26144@milliways \
--to=zarniwhoop@ntlworld.com \
--cc=akpm@linux-foundation.org \
--cc=david@fromorbit.com \
--cc=dhowells@redhat.com \
--cc=dipankar@in.ibm.com \
--cc=dvhart@linux.intel.com \
--cc=edumazet@google.com \
--cc=fweisbec@gmail.com \
--cc=joe@perches.com \
--cc=josh@joshtriplett.org \
--cc=laijs@cn.fujitsu.com \
--cc=linux-kernel@vger.kernel.org \
--cc=mathieu.desnoyers@efficios.com \
--cc=mingo@kernel.org \
--cc=niv@us.ibm.com \
--cc=oleg@redhat.com \
--cc=paulmck@linux.vnet.ibm.com \
--cc=peterz@infradead.org \
--cc=rostedt@goodmis.org \
--cc=sbw@mit.edu \
--cc=tglx@linutronix.de \
/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