linux-kernel - Re: [GIT PULL] PCI changes for v4.9

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Fri, 14 Oct 2016 08:25:48 -0500
From:   Bjorn Helgaas <helgaas@...nel.org>
To:     Linus Torvalds <torvalds@...ux-foundation.org>
Cc:     "linux-pci@...r.kernel.org" <linux-pci@...r.kernel.org>,
        Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
        Thomas Petazzoni <thomas.petazzoni@...e-electrons.com>,
        Dan Carpenter <dan.carpenter@...cle.com>,
        Kishon Vijay Abraham I <kishon@...com>
Subject: Re: [GIT PULL] PCI changes for v4.9

On Thu, Oct 13, 2016 at 05:21:40PM -0700, Linus Torvalds wrote:
> On Thu, Oct 13, 2016 at 4:48 PM, Linus Torvalds
> <torvalds@...ux-foundation.org> wrote:
> >
> > Ugh:
> >
> > Your "summary" is actually less legible than the shortlog, and looks
> > entirely auto-generated.
> >
> > That's against the whole point of having a summary for a pull request.
> >
> > Please tell me what changed, don't auto-generate pointless unreadable crud. Ok?
> 
> I wrote some kind of summary based on actually (hopefully) intelligent
> culling of relevant information instead of just bunching up together
> automated data.
> 
> It may not be complete, but it's the kind of "what does this merge
> actually _merge_" information that is useful. Please do something like
> this in the future rather than the automated illegible mush:
> 
>  "Summary:
> 
>    - use local struct device pointers in many host bridge drivers for
>      clarity
> 
>    - remove unused platform data
> 
>    - use generic DesignWare accessors
> 
>    - misc cleanups: remove redundant structure entries and re-order
>      structure members to put comon generic fields first etc"

That's accurate, thank you, and sorry for not providing it myself.

I guess I've always been a little confused on what to put in the email
vs. what should be in the tag message itself.  I'm glad you pointed
this out because it's just dawning on me that the tag message does not
become part of your tree when you pull the tag, while the summary from
the email pull request normally does.  I've been focusing on the tag
message, not the email summary, which was completely backwards.

Bjorn