• Sedat Dilek's avatar
    crypto: aesni - Fix build with LLVM_IAS=1 · 98d7ab74
    Sedat Dilek authored
    [ Upstream commit 3347c8a0 ]
    
    When building with LLVM_IAS=1 means using Clang's Integrated Assembly (IAS)
    from LLVM/Clang >= v10.0.1-rc1+ instead of GNU/as from GNU/binutils
    I see the following breakage in Debian/testing AMD64:
    
    <instantiation>:15:74: error: too many positional arguments
     PRECOMPUTE 8*3+8(%rsp), %xmm1, %xmm2, %xmm3, %xmm4, %xmm5, %xmm6, %xmm7,
                                                                             ^
     arch/x86/crypto/aesni-intel_asm.S:1598:2: note: while in macro instantiation
     GCM_INIT %r9, 8*3 +8(%rsp), 8*3 +16(%rsp), 8*3 +24(%rsp)
     ^
    <instantiation>:47:2: error: unknown use of instruction mnemonic without a size suffix
     GHASH_4_ENCRYPT_4_PARALLEL_dec %xmm9, %xmm10, %xmm11, %xmm12, %xmm13, %xmm14, %xmm0, %xmm1, %xmm2, %xmm3, %xmm4, %xmm5, %xmm6, %xmm7, %xmm8, enc
     ^
    arch/x86/crypto/aesni-intel_asm.S:1599:2: note: while in macro instantiation
     GCM_ENC_DEC dec
     ^
    <instantiation>:15:74: error: too many positional arguments
     PRECOMPUTE 8*3+8(%rsp), %xmm1, %xmm2, %xmm3, %xmm4, %xmm5, %xmm6, %xmm7,
                                                                             ^
    arch/x86/crypto/aesni-intel_asm.S:1686:2: note: while in macro instantiation
     GCM_INIT %r9, 8*3 +8(%rsp), 8*3 +16(%rsp), 8*3 +24(%rsp)
     ^
    <instantiation>:47:2: error: unknown use of instruction mnemonic without a size suffix
     GHASH_4_ENCRYPT_4_PARALLEL_enc %xmm9, %xmm10, %xmm11, %xmm12, %xmm13, %xmm14, %xmm0, %xmm1, %xmm2, %xmm3, %xmm4, %xmm5, %xmm6, %xmm7, %xmm8, enc
     ^
    arch/x86/crypto/aesni-intel_asm.S:1687:2: note: while in macro instantiation
     GCM_ENC_DEC enc
    
    Craig Topper suggested me in ClangBuiltLinux issue #1050:
    
    > I think the "too many positional arguments" is because the parser isn't able
    > to handle the trailing commas.
    >
    > The "unknown use of instruction mnemonic" is because the macro was named
    > GHASH_4_ENCRYPT_4_PARALLEL_DEC but its being instantiated with
    > GHASH_4_ENCRYPT_4_PARALLEL_dec I guess gas ignores case on the
    > macro instantiation, but llvm doesn't.
    
    First, I removed the trailing comma in the PRECOMPUTE line.
    
    Second, I substituted:
    1. GHASH_4_ENCRYPT_4_PARALLEL_DEC -> GHASH_4_ENCRYPT_4_PARALLEL_dec
    2. GHASH_4_ENCRYPT_4_PARALLEL_ENC -> GHASH_4_ENCRYPT_4_PARALLEL_enc
    
    With these changes I was able to build with LLVM_IAS=1 and boot on bare metal.
    
    I confirmed that this works with Linux-kernel v5.7.5 final.
    
    NOTE: This patch is on top of Linux v5.7 final.
    
    Thanks to Craig and especially Nick for double-checking and his comments.
    Suggested-by: default avatarCraig Topper <craig.topper@intel.com>
    Suggested-by: default avatarCraig Topper <craig.topper@gmail.com>
    Suggested-by: default avatarNick Desaulniers <ndesaulniers@google.com>
    Reviewed-by: default avatarNick Desaulniers <ndesaulniers@google.com>
    Cc: "ClangBuiltLinux" <clang-built-linux@googlegroups.com>
    Link: https://github.com/ClangBuiltLinux/linux/issues/1050
    Link: https://bugs.llvm.org/show_bug.cgi?id=24494Signed-off-by: default avatarSedat Dilek <sedat.dilek@gmail.com>
    Signed-off-by: default avatarHerbert Xu <herbert@gondor.apana.org.au>
    Signed-off-by: default avatarSasha Levin <sashal@kernel.org>
    98d7ab74
aesni-intel_asm.S 76.6 KB