Re: [GIT PULL] Block pull request for- 4.11-rc1

From: Linus Torvalds
Date: Tue Feb 21 2017 - 14:14:09 EST

On Sun, Feb 19, 2017 at 4:10 PM, Jens Axboe <axboe@xxxxxxxxx> wrote:
> Please pull! Either this pre-merged branch:
> git:// for-4.11/linus-merge-signed
> or
> git:// for-4.11/block-signed
> git:// for-4.11/next-signed

So normally I'd merge them separately, but since you didn't actually
give me explanations for what the two branches were (ie "block-signed
does X, next-signed does Y") I didn't feel like I could write a sane
merge message for the two branches - so I took the pre-merged one.

Which does bring me to my next issue: *your* merge messages suck too.
They don't actually talk about what you are merging and why.

A merge is a commit, and needs to have a message, unless it's really
really obvious (and they seldom are - merges are generally a lot less
obvious than most non-merge commits). So just saying

Merge branch 'for-4.11/block' into for-4.11/linus-merge

Signed-off-by: Jens Axboe <axboe@xxxxxx>

is simply not an acceptable merge message. What are you merging, and why?

Please. We've been very good at having good commit messages in the
kernel. Merges need good commit messages too!