linux-kernel - Re: [PATCH 0/3] MIPS: Fix build issues from the introduction of `need-compiler'

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:   Wed, 19 Jul 2023 16:39:39 +0100 (BST)
From:   "Maciej W. Rozycki" <macro@...am.me.uk>
To:     Huacai Chen <chenhuacai@...nel.org>
cc:     Thomas Bogendoerfer <tsbogend@...ha.franken.de>,
        Jan-Benedict Glaw <jbglaw@...-owl.de>,
        Guillaume Tucker <guillaume.tucker@...labora.com>,
        Huacai Chen <chenhuacai@...ngson.cn>,
        linux-mips@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH 0/3] MIPS: Fix build issues from the introduction of
 `need-compiler'

On Tue, 18 Jul 2023, Huacai Chen wrote:

> Even if patch-2 resolves the problem, I don't think patch-3 is
> necessary because the original patch makes code simpler and more
> compact.

 Please don't top-post on a public mailing list.

 If you're referring to this part:

+ifdef CONFIG_CPU_LOONGSON64
+cflags-$(CONFIG_CPU_LOONGSON64)	+= -Wa,--trap
+cflags-$(CONFIG_CC_IS_GCC) += -march=loongson3a
+cflags-$(CONFIG_CC_IS_CLANG) += -march=mips64r2
+endif

then it can be done with a separate clean-up.  Otherwise it'll have been 
lost in the noise.

 Firstly:

cflags-$(CONFIG_CPU_LOONGSON64)	+= -Wa,--trap

doesn't have to be wrapped in `ifdef CONFIG_CPU_LOONGSON64'.

 Secondly:

cflags-$(CONFIG_CC_IS_GCC) += -march=loongson3a
cflags-$(CONFIG_CC_IS_CLANG) += -march=mips64r2

document compiler peculiarities.  Does Clang support, or intend to, 
`-march=loongson3a'?  If so, what version can we expect this stuff in?  
GCC has had it since 4.6 or Y2010, which is pretty long ago.

 Last but not least there are formatting anomalies there, which may have 
to be dealt with in a separate change.

  Maciej