Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753908AbcCRFn6 (ORCPT ); Fri, 18 Mar 2016 01:43:58 -0400 Received: from mail-vk0-f53.google.com ([209.85.213.53]:35244 "EHLO mail-vk0-f53.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753892AbcCRFnr (ORCPT ); Fri, 18 Mar 2016 01:43:47 -0400 MIME-Version: 1.0 In-Reply-To: <1458279643.9556.52.camel@perches.com> References: <20160314235035.GA5799@localhost.localdomain> <1457999823.11972.143.camel@perches.com> <20160315112458.7248d665@canb.auug.org.au> <20160315130339.688689bd@canb.auug.org.au> <20160315151618.GQ29218@thunk.org> <1458279643.9556.52.camel@perches.com> Date: Thu, 17 Mar 2016 23:43:46 -0600 Message-ID: Subject: Re: [GIT PULL v4.6] MDB Linux Kernel Debugger x86/x86_64 From: Jeffrey Merkey To: Joe Perches Cc: "Theodore Ts'o" , Stephen Rothwell , linux-kernel@vger.kernel.org, torvalds@linux-foundation.org, linux-kbuild Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2149 Lines: 48 On 3/17/16, Joe Perches wrote: > On Thu, 2016-03-17 at 23:31 -0600, Jeffrey Merkey wrote: >> On 3/16/16, Jeffrey Merkey wrote: >> > On 3/15/16, Theodore Ts'o wrote: >> > > On Tue, Mar 15, 2016 at 01:03:39PM +1100, Stephen Rothwell wrote: >> > > > >> > > > We don't generally PGP (GPG) sign commits in the kernel tree (so >> > > > "-S" >> > > > is not required), just tags. However we always require that anyone >> > > > who >> > > > handles a patch adds a Signed-off-by line to the final commit. See >> > > > Documentation/SubmittingPatches Section 11. >> > > In general all commits should have a Signed-Off-By line added. Once >> > > the git branch gets merged, it's hard to tell what is the final >> > > commit >> > > and what isn't, and in general different commits will have different >> > > people needing to vouch for the origins for the contents of that >> > > commit. > [] >> > I will repost this pull request with a signed-off by line for Linus to >> > consider. I am not certain who all should be included in the review >> > and I have to be careful how many email recipients I copy. I will >> > post to the folks Joe Perches copied. > [] >> Well, Joe updated checkpatch.pl so now there are some more areas of >> cleanup I need to do on the code to remove unsigned casts and replace >> them with unsigned int. I will resubmit at the next merge window for >> 4.7. Have not heard a word from Linus about this, Hey Linus, please >> don't lion-us by throwing us to the lions. :-) > > I rather doubt Linus is going to pull this code during > this merge window. > > I think you should try to get this into -next immediately > after this current merge window closes in a 10 or so days. > > I think you don't have to do random cleanups just to > satisfy checkpatch. Get the code in -next and then > cleanup stuff as you see fit. There will then be > individual commit changelogs for the mostly whitespace > changes that checkpatch might point out. > > OK, then I will setup a pull request for Stephen to pull against the -next tree. He can pull it at his convenience. Jeff