linux-kernel - Re: [PATCH 1/3] arm/kbuild: replace {C,LD}FLAGS_MODULE with KBUILD_{C,LD}FLAGS

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <CAK7LNARj0Lp-pNdcmfTgadZrO6rzazXmdG2xqeL0Pt1pbRWcWw@mail.gmail.com>
Date:   Tue, 31 Oct 2017 00:41:57 +0900
From:   Masahiro Yamada <yamada.masahiro@...ionext.com>
To:     Cao jin <caoj.fnst@...fujitsu.com>
Cc:     Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
        Russell King <linux@...linux.org.uk>,
        linux-arm-kernel <linux-arm-kernel@...ts.infradead.org>
Subject: Re: [PATCH 1/3] arm/kbuild: replace {C,LD}FLAGS_MODULE with KBUILD_{C,LD}FLAGS_MODULE

2017-10-09 16:11 GMT+09:00 Cao jin <caoj.fnst@...fujitsu.com>:
> As kbuild document & commit 6588169d51 says: KBUILD_{C,LD}FLAGS_MODULE are
> used to add arch-specific options for $(CC) and $(LD). From commandline,
> {C,LD}FLAGS_MODULE shall be used.
> Doesn't have any functional change, but just follow kbuild rules.
>
> Signed-off-by: Cao jin <caoj.fnst@...fujitsu.com>
> CC: Russell King <linux@...linux.org.uk>
> CC: linux-arm-kernel@...ts.infradead.org

Applied to linux-kbuild.  Thanks!


> ---
>  arch/arm/Makefile | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/arch/arm/Makefile b/arch/arm/Makefile
> index 47d3a1a..a945b1f 100644
> --- a/arch/arm/Makefile
> +++ b/arch/arm/Makefile
> @@ -16,11 +16,11 @@ LDFLAGS             :=
>  LDFLAGS_vmlinux        :=-p --no-undefined -X --pic-veneer
>  ifeq ($(CONFIG_CPU_ENDIAN_BE8),y)
>  LDFLAGS_vmlinux        += --be8
> -LDFLAGS_MODULE += --be8
> +KBUILD_LDFLAGS_MODULE  += --be8
>  endif
>
>  ifeq ($(CONFIG_ARM_MODULE_PLTS),y)
> -LDFLAGS_MODULE += -T $(srctree)/arch/arm/kernel/module.lds
> +KBUILD_LDFLAGS_MODULE  += -T $(srctree)/arch/arm/kernel/module.lds
>  endif
>
>  GZFLAGS                :=-9
> @@ -120,7 +120,7 @@ CFLAGS_ISA  :=-mthumb $(AFLAGS_AUTOIT) $(AFLAGS_NOWARN)
>  AFLAGS_ISA     :=$(CFLAGS_ISA) -Wa$(comma)-mthumb
>  # Work around buggy relocation from gas if requested:
>  ifeq ($(CONFIG_THUMB2_AVOID_R_ARM_THM_JUMP11),y)
> -CFLAGS_MODULE  +=-fno-optimize-sibling-calls
> +KBUILD_CFLAGS_MODULE   +=-fno-optimize-sibling-calls
>  endif
>  else
>  CFLAGS_ISA     :=$(call cc-option,-marm,)
> --
> 2.1.0
>
>
>



-- 
Best Regards
Masahiro Yamada