linux-kernel - Re: [PATCH v2] MIPS: Makefile: Replace -pg with CC_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] [day] [month] [year] [list]

Message-ID: <167b7021-d471-89f0-4446-b1d022bdf6c9@gmail.com>
Date:   Mon, 19 Apr 2021 12:18:21 +0300
From:   Sergei Shtylyov <sergei.shtylyov@...il.com>
To:     zhaoxiao <zhaoxiao@...ontech.com>, tsbogend@...ha.franken.de,
        jiaxun.yang@...goat.com, chenhuacai@...nel.org,
        viro@...iv.linux.org.uk, morbo@...gle.com
Cc:     paul@...pouillou.net, alobakin@...me, suxingxing@...ngson.cn,
        tangyouling@...ngson.cn, yangtiezhu@...ngson.cn,
        ralf@...ux-mips.org, nathan@...nel.org, masahiroy@...nel.org,
        keescook@...omium.org, linux-mips@...r.kernel.org,
        linux-kernel@...r.kernel.org
Subject: Re: [PATCH v2] MIPS: Makefile: Replace -pg with CC_FLAGS_FTRACE

Hello!

On 19.04.2021 8:51, zhaoxiao wrote:

> In preparation for mips supporting ftrace built on other compiler
> options, let's have the mips Makefiles remove the $(CC_FLAGS_FTRACE)
> flags, whatever these may be, rather than assuming '-pg'.

    But your patch is adding 'em,  not removing?

> Signed-off-by: zhaoxiao <zhaoxiao@...ontech.com>
> ---
> v2:add this same change be appropriate to all of the below Makefile.
>   arch/mips/boot/compressed/Makefile | 2 +-
>   arch/mips/kernel/Makefile          | 8 ++++----
>   arch/mips/vdso/Makefile            | 4 ++--
>   3 files changed, 7 insertions(+), 7 deletions(-)
> 
> diff --git a/arch/mips/boot/compressed/Makefile b/arch/mips/boot/compressed/Makefile
> index f93f72bcba97..e4b7839293e1 100644
> --- a/arch/mips/boot/compressed/Makefile
> +++ b/arch/mips/boot/compressed/Makefile
> @@ -18,7 +18,7 @@ include $(srctree)/arch/mips/Kbuild.platforms
>   BOOT_HEAP_SIZE := 0x400000
>   
>   # Disable Function Tracer
> -KBUILD_CFLAGS := $(filter-out -pg, $(KBUILD_CFLAGS))
> +KBUILD_CFLAGS := $(filter-out $(CC_FLAGS_FTRACE), $(KBUILD_CFLAGS))
>   
>   KBUILD_CFLAGS := $(filter-out -fstack-protector, $(KBUILD_CFLAGS))
>   
> diff --git a/arch/mips/kernel/Makefile b/arch/mips/kernel/Makefile
> index b4a57f1de772..814b3da30501 100644
> --- a/arch/mips/kernel/Makefile
> +++ b/arch/mips/kernel/Makefile
> @@ -17,10 +17,10 @@ obj-y		+= cpu-probe.o
>   endif
>   
>   ifdef CONFIG_FUNCTION_TRACER
> -CFLAGS_REMOVE_ftrace.o = -pg
> -CFLAGS_REMOVE_early_printk.o = -pg
> -CFLAGS_REMOVE_perf_event.o = -pg
> -CFLAGS_REMOVE_perf_event_mipsxx.o = -pg
> +CFLAGS_REMOVE_ftrace.o = $(CC_FLAGS_FTRACE)
> +CFLAGS_REMOVE_early_printk.o =  $(CC_FLAGS_FTRACE)
> +CFLAGS_REMOVE_perf_event.o = $(CC_FLAGS_FTRACE)
> +CFLAGS_REMOVE_perf_event_mipsxx.o = $(CC_FLAGS_FTRACE)
>   endif
>   
>   obj-$(CONFIG_CEVT_BCM1480)	+= cevt-bcm1480.o
> diff --git a/arch/mips/vdso/Makefile b/arch/mips/vdso/Makefile
> index 2131d3fd7333..1b2ea34c3d3b 100644
> --- a/arch/mips/vdso/Makefile
> +++ b/arch/mips/vdso/Makefile
> @@ -46,7 +46,7 @@ CFLAGS_vgettimeofday-o32.o = -include $(srctree)/$(src)/config-n32-o32-env.c -in
>   CFLAGS_vgettimeofday-n32.o = -include $(srctree)/$(src)/config-n32-o32-env.c -include $(c-gettimeofday-y)
>   endif
>   
> -CFLAGS_REMOVE_vgettimeofday.o = -pg
> +CFLAGS_REMOVE_vgettimeofday.o = $(CC_FLAGS_FTRACE)
>   
>   ifdef CONFIG_MIPS_DISABLE_VDSO
>     ifndef CONFIG_MIPS_LD_CAN_LINK_VDSO
> @@ -60,7 +60,7 @@ ldflags-y := -Bsymbolic --no-undefined -soname=linux-vdso.so.1 \
>   	$(filter -E%,$(KBUILD_CFLAGS)) -nostdlib -shared \
>   	-G 0 --eh-frame-hdr --hash-style=sysv --build-id=sha1 -T
>   
> -CFLAGS_REMOVE_vdso.o = -pg
> +CFLAGS_REMOVE_vdso.o = $(CC_FLAGS_FTRACE)
>   
>   GCOV_PROFILE := n
>   UBSAN_SANITIZE := n

MBR, Sergei