On Thu, 1 Aug 2019 at 15:28, Heiko Carstens <heiko.carstens@xxxxxxxxxx> wrote: > > On Wed, Jul 31, 2019 at 01:44:54PM +0200, Heiko Carstens wrote: > > On Wed, Jul 31, 2019 at 09:32:16PM +1000, Herbert Xu wrote: > > > On Wed, Jul 31, 2019 at 01:15:20PM +0200, Heiko Carstens wrote: > > > > > > > > However that doesn't fix the simd.h header file breakage with the > > > > second patch :) > > > > > > That fix should be there now too. > > > > Yes, works now. Thank you! > > Still not... with linux-next as of today I get this (s390 defconfig): > > ERROR: "crypto_aegis128_decrypt_chunk_simd" [crypto/aegis128.ko] undefined! > ERROR: "crypto_aegis128_update_simd" [crypto/aegis128.ko] undefined! > ERROR: "crypto_aegis128_encrypt_chunk_simd" [crypto/aegis128.ko] undefined! > scripts/Makefile.modpost:105: recipe for target 'modules-modpost' failed > Hello Heiko, Apologies for the breakage. The first two fixes addressed obvious shortcomings in my code, but with this issue, I'm a bit puzzled tbh. The calls to these missing functions should be optimized away, since have_simd never gets assigned if CONFIG_CRYPTO_AEGIS128_SIMD is not defined, but for some reason, this isn't working. Which version of GCC are you using? Also, could you please try whether the patch below fixes the problem? Thanks https://lore.kernel.org/linux-crypto/20190729074434.21064-1-ard.biesheuvel@xxxxxxxxxx/
