linux-kernel - Re: [PATCH] powerpc/math

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, 2 Sep 2022 08:43:49 +0000
From:   Christophe Leroy <christophe.leroy@...roup.eu>
To:     Segher Boessenkool <segher@...nel.crashing.org>
CC:     Nathan Chancellor <nathan@...nel.org>,
        Michael Ellerman <mpe@...erman.id.au>,
        kernel test robot <lkp@...el.com>,
        Arnd Bergmann <arnd@...db.de>, Tom Rix <trix@...hat.com>,
        "llvm@...ts.linux.dev" <llvm@...ts.linux.dev>,
        Nick Desaulniers <ndesaulniers@...gle.com>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        Nicholas Piggin <npiggin@...il.com>,
        "patches@...ts.linux.dev" <patches@...ts.linux.dev>,
        "linuxppc-dev@...ts.ozlabs.org" <linuxppc-dev@...ts.ozlabs.org>
Subject: Re: [PATCH] powerpc/math_emu/efp: Include module.h



Le 01/09/2022 à 21:47, Segher Boessenkool a écrit :
> On Thu, Sep 01, 2022 at 05:41:33AM +0000, Christophe Leroy wrote:
>> I think it would be worth a GCC bug report.
> 
> We need a stand-alone testcase for this.  When you have created one, at
> least 98% of the time you discover the bug is in user code after all.
> 
> Which is a very good thing, it means the problem can be fixed simpler,
> cheaper, and a lot faster :-)

Easy to reproduce with a .c file that has a single line:

non_existing_macro(xxx);


Apparently that's due to the -w option in arch/powerpc/math_emu/Makefile:

	ccflags-y = -w

Was introduced by commit d2b194ed8208 ("powerpc/math-emu: Use kernel 
generic math-emu code")

If I understand correctly it means 'ignore all warnings'. Then it seems 
CLANG doesn't honor that request.

Christophe