Re: [GIT PULL] Please pull powerpc/linux.git powerpc-4.9-6 tag

From: Linus Torvalds
Date: Sat Nov 26 2016 - 14:29:44 EST


On Sat, Nov 26, 2016 at 12:11 AM, Michael Ellerman <mpe@xxxxxxxxxxxxxx> wrote:
>
> powerpc fixes for 4.9 #6
>
> Fixes marked for stable:
> - Set missing wakeup bit in LPCR on POWER9 (Benjamin Herrenschmidt)
> - Fix the early OPAL console wrappers (Oliver O'Halloran)
> - Fixup kernel read only mapping (Aneesh Kumar K.V)
>
> Fixes for code merged this cycle:
> - Fix missing CRCs, add more asm-prototypes.h declarations (Nicholas Piggin)

Pulled, but I wanted to talk about your merge "summary".

Your merge summaries seem to be entirely automatically generated,
which makes them less than great. I can see all that stuff in the git
tree already, just formatting it differently isn't all that useful.

For something like this late-rc pull when there are only a couple of
commits, the end result actually ends up looking almost like a summary
and all I did was remove the names that don't add to the description
(and are in the git commits).

For some of the bigger pull requests, the summary is almost anything
but, and the only real value-add is the grouping by subject area.

I really prefer a _summary_. Something that is human-legible. So that
when people read the merge commit log, they get an overview of what
changed, not a list of the details.

Again, when there are four commits, a "list of the details" pretty
much works. So the reason I react to _this_ pull request is mainly
that I have way more time to react to it during the late rc series
than I do earlier in the cycle..

Linus