Commit 45933112 authored by Uros Bizjak's avatar Uros Bizjak Committed by Borislav Petkov (AMD)
Browse files

crypto: X86 - Remove CONFIG_AS_VAES 



Current minimum required version of binutils is 2.30, which supports VAES
instruction mnemonics.

Remove check for assembler support of VAES instructions and all relevant macros
for conditional compilation.

No functional change intended.

Signed-off-by: default avatarUros Bizjak <ubizjak@gmail.com>
Signed-off-by: default avatarBorislav Petkov (AMD) <bp@alien8.de>
Acked-by: default avatarHerbert Xu <herbert@gondor.apana.org.au>
Link: https://lore.kernel.org/20250819085855.333380-2-ubizjak@gmail.com
parent a35da573

arch/x86/Kconfig.assembler

+0 −5
Original line number Diff line number Diff line
@@ -6,11 +6,6 @@ config AS_AVX512 
	help

	  Supported by binutils >= 2.25 and LLVM integrated assembler



config AS_VAES

	def_bool $(as-instr,vaesenc %ymm0$(comma)%ymm1$(comma)%ymm2)

	help

	  Supported by binutils >= 2.30 and LLVM integrated assembler



config AS_VPCLMULQDQ

	def_bool $(as-instr,vpclmulqdq \$0x10$(comma)%ymm0$(comma)%ymm1$(comma)%ymm2)

	help

arch/x86/crypto/Makefile

+1 −1
Original line number Diff line number Diff line
@@ -47,7 +47,7 @@ aesni-intel-y := aesni-intel_asm.o aesni-intel_glue.o 
aesni-intel-$(CONFIG_64BIT) += aes-ctr-avx-x86_64.o \

			       aes-gcm-aesni-x86_64.o \

			       aes-xts-avx-x86_64.o

ifeq ($(CONFIG_AS_VAES)$(CONFIG_AS_VPCLMULQDQ),yy)

ifeq ($(CONFIG_AS_VPCLMULQDQ),y)

aesni-intel-$(CONFIG_64BIT) += aes-gcm-avx10-x86_64.o

endif

arch/x86/crypto/aes-ctr-avx-x86_64.S

+2 −2
Original line number Diff line number Diff line
@@ -552,7 +552,7 @@ SYM_TYPED_FUNC_START(aes_xctr_crypt_aesni_avx) 
	_aes_ctr_crypt	1

SYM_FUNC_END(aes_xctr_crypt_aesni_avx)



#if defined(CONFIG_AS_VAES) && defined(CONFIG_AS_VPCLMULQDQ)

#if defined(CONFIG_AS_VPCLMULQDQ)

.set	VL, 32

.set	USE_AVX512, 0

SYM_TYPED_FUNC_START(aes_ctr64_crypt_vaes_avx2)
@@ -570,4 +570,4 @@ SYM_FUNC_END(aes_ctr64_crypt_vaes_avx512) 
SYM_TYPED_FUNC_START(aes_xctr_crypt_vaes_avx512)

	_aes_ctr_crypt	1

SYM_FUNC_END(aes_xctr_crypt_vaes_avx512)

#endif // CONFIG_AS_VAES && CONFIG_AS_VPCLMULQDQ
 
#endif // CONFIG_AS_VPCLMULQDQ

arch/x86/crypto/aes-xts-avx-x86_64.S

+2 −2
Original line number Diff line number Diff line
@@ -886,7 +886,7 @@ SYM_TYPED_FUNC_START(aes_xts_decrypt_aesni_avx) 
	_aes_xts_crypt	0

SYM_FUNC_END(aes_xts_decrypt_aesni_avx)



#if defined(CONFIG_AS_VAES) && defined(CONFIG_AS_VPCLMULQDQ)

#if defined(CONFIG_AS_VPCLMULQDQ)

.set	VL, 32

.set	USE_AVX512, 0

SYM_TYPED_FUNC_START(aes_xts_encrypt_vaes_avx2)
@@ -904,4 +904,4 @@ SYM_FUNC_END(aes_xts_encrypt_vaes_avx512) 
SYM_TYPED_FUNC_START(aes_xts_decrypt_vaes_avx512)

	_aes_xts_crypt	0

SYM_FUNC_END(aes_xts_decrypt_vaes_avx512)

#endif /* CONFIG_AS_VAES && CONFIG_AS_VPCLMULQDQ */
 
#endif /* CONFIG_AS_VPCLMULQDQ */

arch/x86/crypto/aesni-intel_glue.c

+7 −7
Original line number Diff line number Diff line
@@ -828,7 +828,7 @@ static struct skcipher_alg skcipher_algs_##suffix[] = {{ \ 
}}



DEFINE_AVX_SKCIPHER_ALGS(aesni_avx, "aesni-avx", 500);

#if defined(CONFIG_AS_VAES) && defined(CONFIG_AS_VPCLMULQDQ)

#if defined(CONFIG_AS_VPCLMULQDQ)

DEFINE_AVX_SKCIPHER_ALGS(vaes_avx2, "vaes-avx2", 600);

DEFINE_AVX_SKCIPHER_ALGS(vaes_avx512, "vaes-avx512", 800);

#endif
@@ -912,7 +912,7 @@ struct aes_gcm_key_avx10 { 
#define FLAG_RFC4106	BIT(0)

#define FLAG_ENC	BIT(1)

#define FLAG_AVX	BIT(2)

#if defined(CONFIG_AS_VAES) && defined(CONFIG_AS_VPCLMULQDQ)

#if defined(CONFIG_AS_VPCLMULQDQ)

#  define FLAG_AVX10_256	BIT(3)

#  define FLAG_AVX10_512	BIT(4)

#else
@@ -1519,7 +1519,7 @@ DEFINE_GCM_ALGS(aesni_avx, FLAG_AVX, 
		"generic-gcm-aesni-avx", "rfc4106-gcm-aesni-avx",

		AES_GCM_KEY_AESNI_SIZE, 500);



#if defined(CONFIG_AS_VAES) && defined(CONFIG_AS_VPCLMULQDQ)

#if defined(CONFIG_AS_VPCLMULQDQ)

/* aes_gcm_algs_vaes_avx10_256 */

DEFINE_GCM_ALGS(vaes_avx10_256, FLAG_AVX10_256,

		"generic-gcm-vaes-avx10_256", "rfc4106-gcm-vaes-avx10_256",
@@ -1529,7 +1529,7 @@ DEFINE_GCM_ALGS(vaes_avx10_256, FLAG_AVX10_256, 
DEFINE_GCM_ALGS(vaes_avx10_512, FLAG_AVX10_512,

		"generic-gcm-vaes-avx10_512", "rfc4106-gcm-vaes-avx10_512",

		AES_GCM_KEY_AVX10_SIZE, 800);

#endif /* CONFIG_AS_VAES && CONFIG_AS_VPCLMULQDQ */

#endif /* CONFIG_AS_VPCLMULQDQ */



static int __init register_avx_algs(void)

{
@@ -1551,7 +1551,7 @@ static int __init register_avx_algs(void) 
	 * Similarly, the assembler support was added at about the same time.

	 * For simplicity, just always check for VAES and VPCLMULQDQ together.

	 */

#if defined(CONFIG_AS_VAES) && defined(CONFIG_AS_VPCLMULQDQ)

#if defined(CONFIG_AS_VPCLMULQDQ)

	if (!boot_cpu_has(X86_FEATURE_AVX2) ||

	    !boot_cpu_has(X86_FEATURE_VAES) ||

	    !boot_cpu_has(X86_FEATURE_VPCLMULQDQ) ||
@@ -1592,7 +1592,7 @@ static int __init register_avx_algs(void) 
				    ARRAY_SIZE(aes_gcm_algs_vaes_avx10_512));

	if (err)

		return err;

#endif /* CONFIG_AS_VAES && CONFIG_AS_VPCLMULQDQ */

#endif /* CONFIG_AS_VPCLMULQDQ */

	return 0;

}
@@ -1607,7 +1607,7 @@ static void unregister_avx_algs(void) 
{

	unregister_skciphers(skcipher_algs_aesni_avx);

	unregister_aeads(aes_gcm_algs_aesni_avx);

#if defined(CONFIG_AS_VAES) && defined(CONFIG_AS_VPCLMULQDQ)

#if defined(CONFIG_AS_VPCLMULQDQ)

	unregister_skciphers(skcipher_algs_vaes_avx2);

	unregister_skciphers(skcipher_algs_vaes_avx512);

	unregister_aeads(aes_gcm_algs_vaes_avx10_256);