linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: linux@arm.linux.org.uk (Russell King - ARM Linux)
To: linux-arm-kernel@lists.infradead.org
Subject: [GIT PULL] i.MX clock fixes for v3.8
Date: Tue, 5 Feb 2013 13:27:59 +0000	[thread overview]
Message-ID: <20130205132759.GA17833@n2100.arm.linux.org.uk> (raw)
In-Reply-To: <201302051312.11434.arnd@arndb.de>

On Tue, Feb 05, 2013 at 01:12:11PM +0000, Arnd Bergmann wrote:
> On Tuesday 05 February 2013, Sascha Hauer wrote:
> > >        So think twice - or thrice - before sending me patches or a pull
> > >        request for -rc7. You need to have some seriously good reasons for
> > >        doing so, and you need to state those reasons very clearly. And I
> > >        don't just mean for the pull request in general, I mean for every
> > >        single patch in it.
> > > ...
> > >        In other words, "It fixes a bug" just isn't good enough. The bug needs
> > >        to be something that actually matters.
> > 
> > At least 'ARM: i.MX25: clk: parent per5_clk to AHB clock' is a
> > regression that causes nasty oopses. Admittedly, it wasn't introduced in
> > the last merge window and the board requiring the fix needs out of tree
> > patches.
> > 
> > I'm fine with these going into v3.9 and waiting for the first stable
> > patch to get the fix for the above.
> 
> My rules is usually: if it's important enough to have it backported
> into a stable kernel, then it's also important enough to get sent
> at any time during the bug fix phase.
> 
> The patch you  mentioned certainly fits that category. The other one
> might as well, but the patch description does not actually say what
> the impact of the bug is, so it is hard to tell.

Look at what Linus is saying.  Too much stuff went into -rc6.  He doesn't
want that happening for -rc7.  He wants -rc7 to be really tiny, like
almost _no_ changes.  Look at the description about what he wants to
see: he doesn't want to see user visible regressions.  He wants to see
_big_ user visible regressions only.

Also look at what he's saying wrt describing _every_ _single_ _patch_ in
your pull request with a justification why it's there.  Can you do this
for every patch you have planned to push?  Do you have sufficient
information from those sending you these pull requests for fixes to v3.8
to do that?  If not, you probably shouldn't be pulling them with a view
to sending them to Linus (unless you wish to be flamed.)

I certainly can't with the three patches I had queued for fixes before
-rc6.  They may be fixing problems which cause a few platforms not to
boot or oops at boot, but I can't justify them against what Linus has
said.  So I'm going to wait for the next merge window and put them in
with a stable CC.  There's more patches like that post-rc6: there's one
which fixes an obscure problem with 'make' which occurs with 2G:2G VM
split.  But I'm not pushing that one either because it's not a _big_
enough problem (it's only had _one_ user report it.)  One user is not
sufficient justification.

  reply	other threads:[~2013-02-05 13:27 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-02-05  9:58 [GIT PULL] i.MX clock fixes for v3.8 Sascha Hauer
2013-02-05 11:08 ` Russell King - ARM Linux
2013-02-05 11:25   ` Sascha Hauer
2013-02-05 13:12     ` Arnd Bergmann
2013-02-05 13:27       ` Russell King - ARM Linux [this message]
2013-02-05 18:44         ` Olof Johansson
2013-02-05 19:11           ` Arnd Bergmann
2013-02-05 19:15         ` Olof Johansson
2013-02-12  8:02 ` Sascha Hauer
2013-02-19 21:46   ` Arnd Bergmann

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=20130205132759.GA17833@n2100.arm.linux.org.uk \
    --to=linux@arm.linux.org.uk \
    --cc=linux-arm-kernel@lists.infradead.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).