linux-kernel - Re: [PATCH-next v3] arm32: enable HAVE_LD_DEAD_CODE_DATA

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]

Date: Fri, 15 Mar 2024 12:51:43 +0100
From: "Arnd Bergmann" <arnd@...db.de>
To: "Yuntao Liu" <liuyuntao12@...wei.com>, "Ard Biesheuvel" <ardb@...nel.org>
Cc: linux-arm-kernel@...ts.infradead.org, linux-kernel@...r.kernel.org,
 "Geert Uytterhoeven" <geert@...ux-m68k.org>,
 "Russell King" <linux@...linux.org.uk>, "Andrew Davis" <afd@...com>,
 "Andrew Morton" <akpm@...ux-foundation.org>,
 "Kirill A. Shutemov" <kirill.shutemov@...ux.intel.com>,
 "Geert Uytterhoeven" <geert+renesas@...der.be>,
 "Jonathan Corbet" <corbet@....net>, "Mike Rapoport" <rppt@...nel.org>,
 "Rob Herring" <robh@...nel.org>, "Thomas Gleixner" <tglx@...utronix.de>,
 "Linus Walleij" <linus.walleij@...aro.org>,
 "Fangrui Song" <maskray@...gle.com>
Subject: Re: [PATCH-next v3] arm32: enable HAVE_LD_DEAD_CODE_DATA_ELIMINATION

On Fri, Mar 15, 2024, at 12:48, liuyuntao (F) wrote:
> On 2024/3/15 18:46, Ard Biesheuvel wrote:
>> @@ -28,6 +28,7 @@
>>                                     -DEFI_HAVE_MEMCHR -DEFI_HAVE_STRRCHR \
>>                                     -DEFI_HAVE_STRCMP -fno-builtin -fpic \
>>                                     $(call cc-option,-mno-single-pic-base)
>> +cflags-$(CONFIG_ARM)           := $(filter-out -fdata-sections, $(CFLAGS-y))
>>   cflags-$(CONFIG_RISCV)         += -fpic -DNO_ALTERNATIVE -mno-relax
>>   cflags-$(CONFIG_LOONGARCH)     += -fpie
>
> Another error:
> cannot initialize array of ‘short unsigned int’ from a string literal 
> with type array of ‘unsigned int’
>   17 | static const efi_char16_t shim_MokSBState_name[] = L"MokSBStateRT";

The line sets cflags-y to an empty string.

It probably should have been

cflags-$(CONFIG_ARM)           := $(filter-out -fdata-sections, $(cflags-y))

(small cflags instead of CFLAGS).

     Arnd