public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Tilman Schmidt <tilman@imap.cc>
To: "Luis R. Rodriguez" <mcgrof@gmail.com>
Cc: stable-review@kernel.org, linux-kernel@vger.kernel.org,
	Greg KH <greg@kroah.com>, Chris Wright <chrisw@sous-sol.org>
Subject: Re: Would it help to encourage users to read/test stable-review patches
Date: Sun, 30 Aug 2009 19:18:19 +0200	[thread overview]
Message-ID: <4A9AB45B.6060500@imap.cc> (raw)
In-Reply-To: <43e72e890908271122l6b08aaa5lc66de78301c7c88d@mail.gmail.com>

[-- Attachment #1: Type: text/plain, Size: 1232 bytes --]

Luis R. Rodriguez schrieb:
> We get development review eyes on stable patches during submission
> into a subsystem, and perhaps 1-2 user tests then. Typically patches
> are pretty easy to review for stable submission but I'm wondering
> about cases where things are not so clear [...]
> Do we want more user exposure to stable-review mailing list or is just
> a few developer eyes enough?

Having been bitten quite recently by a -stable mismerge, I am of two
minds about this. On one hand, it would have been nice to catch that
problem during stable review instead of when it hit an actual
distribution. (Ubuntu, as it happened.) On the other hand, I'm not
sure user exposure would have helped all that much in this case.

Generally speaking, the way I understand stable rules, those cases
where things are not so clear shouldn't make it into the review in
the first place. The problem is with the cases where things appear
to be quite clear but aren't. Those are the ones the review cycle
must catch.

Jm2c
T.

-- 
Tilman Schmidt                    E-Mail: tilman@imap.cc
Bonn, Germany
Diese Nachricht besteht zu 100% aus wiederverwerteten Bits.
Ungeöffnet mindestens haltbar bis: (siehe Rückseite)


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 254 bytes --]

      parent reply	other threads:[~2009-08-30 17:18 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-08-27 18:22 Would it help to encourage users to read/test stable-review patches Luis R. Rodriguez
2009-08-27 18:29 ` Chris Wright
2009-08-27 19:09   ` Luis R. Rodriguez
2009-08-28  8:23     ` Stefan Richter
2009-08-30 17:18 ` Tilman Schmidt [this message]

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=4A9AB45B.6060500@imap.cc \
    --to=tilman@imap.cc \
    --cc=chrisw@sous-sol.org \
    --cc=greg@kroah.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mcgrof@gmail.com \
    --cc=stable-review@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