All of lore.kernel.org
 help / color / mirror / Atom feed
From: Greg KH <gregkh@linuxfoundation.org>
To: Nathan Chancellor <nathan@kernel.org>
Cc: anders.roxell@linaro.org, natechancellor@gmail.com,
	tsbogend@alpha.franken.de, stable@vger.kernel.org
Subject: Re: FAILED: patch "[PATCH] MIPS: VDSO: Use CLANG_FLAGS instead of filtering out" failed to apply to 5.4-stable tree
Date: Thu, 4 Mar 2021 14:51:42 +0100	[thread overview]
Message-ID: <YEDl7qlbeWjGfoL+@kroah.com> (raw)
In-Reply-To: <20210301162116.hx5vjaeldfvgtieq@24bbad8f3778>

On Mon, Mar 01, 2021 at 09:21:16AM -0700, Nathan Chancellor wrote:
> On Mon, Mar 01, 2021 at 10:58:07AM +0100, gregkh@linuxfoundation.org wrote:
> > 
> > The patch below does not apply to the 5.4-stable tree.
> > If someone wants it applied there, or to any other stable or longterm
> > tree, then please email the backport, including the original git commit
> > id to <stable@vger.kernel.org>.
> > 
> > thanks,
> > 
> > greg k-h
> > 
> > ------------------ original commit in Linus's tree ------------------
> > 
> > From 76d7fff22be3e4185ee5f9da2eecbd8188e76b2c Mon Sep 17 00:00:00 2001
> > From: Nathan Chancellor <nathan@kernel.org>
> > Date: Fri, 15 Jan 2021 12:26:22 -0700
> > Subject: [PATCH] MIPS: VDSO: Use CLANG_FLAGS instead of filtering out
> >  '--target='
> > 
> > Commit ee67855ecd9d ("MIPS: vdso: Allow clang's --target flag in VDSO
> > cflags") allowed the '--target=' flag from the main Makefile to filter
> > through to the vDSO. However, it did not bring any of the other clang
> > specific flags for controlling the integrated assembler and the GNU
> > tools locations (--prefix=, --gcc-toolchain=, and -no-integrated-as).
> > Without these, we will get a warning (visible with tinyconfig):
> > 
> > arch/mips/vdso/elf.S:14:1: warning: DWARF2 only supports one section per
> > compilation unit
> > .pushsection .note.Linux, "a",@note ; .balign 4 ; .long 2f - 1f ; .long
> > 4484f - 3f ; .long 0 ; 1:.asciz "Linux" ; 2:.balign 4 ; 3:
> > ^
> > arch/mips/vdso/elf.S:34:2: warning: DWARF2 only supports one section per
> > compilation unit
> >  .section .mips_abiflags, "a"
> >  ^
> > 
> > All of these flags are bundled up under CLANG_FLAGS in the main Makefile
> > and exported so that they can be added to Makefiles that set their own
> > CFLAGS. Use this value instead of filtering out '--target=' so there is
> > no warning and all of the tools are properly used.
> > 
> > Cc: stable@vger.kernel.org
> > Fixes: ee67855ecd9d ("MIPS: vdso: Allow clang's --target flag in VDSO cflags")
> > Link: https://github.com/ClangBuiltLinux/linux/issues/1256
> > Reported-by: Anders Roxell <anders.roxell@linaro.org>
> > Signed-off-by: Nathan Chancellor <natechancellor@gmail.com>
> > Tested-by: Anders Roxell <anders.roxell@linaro.org>
> > Signed-off-by: Thomas Bogendoerfer <tsbogend@alpha.franken.de>
> > 
> > diff --git a/arch/mips/vdso/Makefile b/arch/mips/vdso/Makefile
> > index 5810cc12bc1d..2131d3fd7333 100644
> > --- a/arch/mips/vdso/Makefile
> > +++ b/arch/mips/vdso/Makefile
> > @@ -16,16 +16,13 @@ ccflags-vdso := \
> >  	$(filter -march=%,$(KBUILD_CFLAGS)) \
> >  	$(filter -m%-float,$(KBUILD_CFLAGS)) \
> >  	$(filter -mno-loongson-%,$(KBUILD_CFLAGS)) \
> > +	$(CLANG_FLAGS) \
> >  	-D__VDSO__
> >  
> >  ifndef CONFIG_64BIT
> >  ccflags-vdso += -DBUILD_VDSO32
> >  endif
> >  
> > -ifdef CONFIG_CC_IS_CLANG
> > -ccflags-vdso += $(filter --target=%,$(KBUILD_CFLAGS))
> > -endif
> > -
> >  #
> >  # The -fno-jump-tables flag only prevents the compiler from generating
> >  # jump tables but does not prevent the compiler from emitting absolute
> > 
> 
> Attached are the 4.19 and 5.4 backports.

Both now queued up, thanks.

greg k-h

      reply	other threads:[~2021-03-04 13:53 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-03-01  9:58 FAILED: patch "[PATCH] MIPS: VDSO: Use CLANG_FLAGS instead of filtering out" failed to apply to 5.4-stable tree gregkh
2021-03-01 16:21 ` Nathan Chancellor
2021-03-04 13:51   ` Greg KH [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=YEDl7qlbeWjGfoL+@kroah.com \
    --to=gregkh@linuxfoundation.org \
    --cc=anders.roxell@linaro.org \
    --cc=natechancellor@gmail.com \
    --cc=nathan@kernel.org \
    --cc=stable@vger.kernel.org \
    --cc=tsbogend@alpha.franken.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.