Re: Patch "tools/objtool/Makefile: don't assume sync-check.sh is executable" has been added to the 4.14-stable tree

[Greg KH <gregkh@linuxfoundation.org>]

On Wed, Jan 17, 2018 at 03:07:14PM -0800, Andrew Morton wrote:
> On Wed, 17 Jan 2018 11:02:12 +0100 <gregkh@linuxfoundation.org> wrote:
> 
> > 
> > This is a note to let you know that I've just added the patch titled
> > 
> >     tools/objtool/Makefile: don't assume sync-check.sh is executable
> > 
> > to the 4.14-stable tree which can be found at:
> >     http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary
> > 
> > The filename of the patch is:
> >      tools-objtool-makefile-don-t-assume-sync-check.sh-is-executable.patch
> > and it can be found in the queue-4.14 subdirectory.
> > 
> > If you, or anyone else, feels it should not be added to the stable tree,
> > please let <stable@vger.kernel.org> know about it.
> > 
> > 
> > >From 0f908ccbeca99ddf0ad60afa710e72aded4a5ea7 Mon Sep 17 00:00:00 2001
> > From: Andrew Morton <akpm@linux-foundation.org>
> > Date: Fri, 12 Jan 2018 16:53:17 -0800
> > Subject: tools/objtool/Makefile: don't assume sync-check.sh is executable
> > 
> > From: Andrew Morton <akpm@linux-foundation.org>
> > 
> > commit 0f908ccbeca99ddf0ad60afa710e72aded4a5ea7 upstream.
> > 
> > patch(1) loses the x bit.  So if a user follows our patching
> > instructions in Documentation/admin-guide/README.rst, their kernel will
> > not compile.
> > 
> > Fixes: 3bd51c5a371de ("objtool: Move kernel headers/code sync check to a script")
> > Reported-by: Nicolas Bock <nicolasbock@gentoo.org>
> > Reported-by Joakim Tjernlund <Joakim.Tjernlund@infinera.com>
> > Cc: Ingo Molnar <mingo@kernel.org>
> > Cc: Josh Poimboeuf <jpoimboe@redhat.com>
> > Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
> > Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
> > Cc: Holger Hoffst�tte <holger@applied-asynchrony.com>
> > Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
> 
> I'm curious.  3bd51c5a371de wasn't marked for any -stable so why is its
> fix being merged?

Because that commit is in the 4.14.10 release as part of the whole PTI
mess.  Distros like Gentoo that package the kernel as a patch on top of
a clean tree break when trying to build the kernel without this commit.

thanks,

greg k-h