Re: [GIT PULL v4.6] MDB Linux Kernel Debugger x86/x86_64

[Jeffrey Merkey]

On 3/17/16, Joe Perches <joe@xxxxxxxxxxx> wrote:
> On Thu, 2016-03-17 at 23:31 -0600, Jeffrey Merkey wrote:
>> On 3/16/16, Jeffrey Merkey <jeffmerkey@xxxxxxxxx> wrote:
>> > On 3/15/16, Theodore Ts'o <tytso@xxxxxxx> 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