Commit 2f93238b authored by Thomas Gleixner's avatar Thomas Gleixner Committed by Peter Zijlstra
Browse files

crypto: x86/sm[34]: Remove redundant alignments 



SYM_FUNC_START*() and friends already imply alignment, remove custom
alignment hacks to make code consistent. This prepares for future
function call ABI changes.

Also, with having pushed the function alignment to 16 bytes, this
custom alignment is completely superfluous.

( this code couldn't seem to make up it's mind about what alignment it
  actually wanted, randomly mixing 8 and 16 bytes )

Signed-off-by: default avatarThomas Gleixner <tglx@linutronix.de>
Signed-off-by: default avatarPeter Zijlstra (Intel) <peterz@infradead.org>
Link: https://lore.kernel.org/r/20220915111144.868540856@infradead.org
parent 3ba56d0b

arch/x86/crypto/sm3-avx-asm_64.S

+0 −1
Original line number Diff line number Diff line
@@ -327,7 +327,6 @@ 
 * void sm3_transform_avx(struct sm3_state *state,

 *                        const u8 *data, int nblocks);

 */

.align 16

SYM_FUNC_START(sm3_transform_avx)

	/* input:

	 *	%rdi: ctx, CTX

arch/x86/crypto/sm4-aesni-avx-asm_64.S

+0 −7
Original line number Diff line number Diff line
@@ -139,13 +139,11 @@ 




.text

.align 16



/*

 * void sm4_aesni_avx_crypt4(const u32 *rk, u8 *dst,

 *                           const u8 *src, int nblocks)

 */

.align 8

SYM_FUNC_START(sm4_aesni_avx_crypt4)

	/* input:

	 *	%rdi: round key array, CTX
@@ -249,7 +247,6 @@ SYM_FUNC_START(sm4_aesni_avx_crypt4) 
	RET;

SYM_FUNC_END(sm4_aesni_avx_crypt4)



.align 8

SYM_FUNC_START_LOCAL(__sm4_crypt_blk8)

	/* input:

	 *	%rdi: round key array, CTX
@@ -363,7 +360,6 @@ SYM_FUNC_END(__sm4_crypt_blk8) 
 * void sm4_aesni_avx_crypt8(const u32 *rk, u8 *dst,

 *                           const u8 *src, int nblocks)

 */

.align 8

SYM_FUNC_START(sm4_aesni_avx_crypt8)

	/* input:

	 *	%rdi: round key array, CTX
@@ -419,7 +415,6 @@ SYM_FUNC_END(sm4_aesni_avx_crypt8) 
 * void sm4_aesni_avx_ctr_enc_blk8(const u32 *rk, u8 *dst,

 *                                 const u8 *src, u8 *iv)

 */

.align 8

SYM_FUNC_START(sm4_aesni_avx_ctr_enc_blk8)

	/* input:

	 *	%rdi: round key array, CTX
@@ -494,7 +489,6 @@ SYM_FUNC_END(sm4_aesni_avx_ctr_enc_blk8) 
 * void sm4_aesni_avx_cbc_dec_blk8(const u32 *rk, u8 *dst,

 *                                 const u8 *src, u8 *iv)

 */

.align 8

SYM_FUNC_START(sm4_aesni_avx_cbc_dec_blk8)

	/* input:

	 *	%rdi: round key array, CTX
@@ -544,7 +538,6 @@ SYM_FUNC_END(sm4_aesni_avx_cbc_dec_blk8) 
 * void sm4_aesni_avx_cfb_dec_blk8(const u32 *rk, u8 *dst,

 *                                 const u8 *src, u8 *iv)

 */

.align 8

SYM_FUNC_START(sm4_aesni_avx_cfb_dec_blk8)

	/* input:

	 *	%rdi: round key array, CTX

arch/x86/crypto/sm4-aesni-avx2-asm_64.S

+0 −6
Original line number Diff line number Diff line
@@ -153,9 +153,6 @@ 
	.long 0xdeadbeef, 0xdeadbeef, 0xdeadbeef



.text

.align 16



.align 8

SYM_FUNC_START_LOCAL(__sm4_crypt_blk16)

	/* input:

	 *	%rdi: round key array, CTX
@@ -281,7 +278,6 @@ SYM_FUNC_END(__sm4_crypt_blk16) 
 * void sm4_aesni_avx2_ctr_enc_blk16(const u32 *rk, u8 *dst,

 *                                   const u8 *src, u8 *iv)

 */

.align 8

SYM_FUNC_START(sm4_aesni_avx2_ctr_enc_blk16)

	/* input:

	 *	%rdi: round key array, CTX
@@ -394,7 +390,6 @@ SYM_FUNC_END(sm4_aesni_avx2_ctr_enc_blk16) 
 * void sm4_aesni_avx2_cbc_dec_blk16(const u32 *rk, u8 *dst,

 *                                   const u8 *src, u8 *iv)

 */

.align 8

SYM_FUNC_START(sm4_aesni_avx2_cbc_dec_blk16)

	/* input:

	 *	%rdi: round key array, CTX
@@ -448,7 +443,6 @@ SYM_FUNC_END(sm4_aesni_avx2_cbc_dec_blk16) 
 * void sm4_aesni_avx2_cfb_dec_blk16(const u32 *rk, u8 *dst,

 *                                   const u8 *src, u8 *iv)

 */

.align 8

SYM_FUNC_START(sm4_aesni_avx2_cfb_dec_blk16)

	/* input:

	 *	%rdi: round key array, CTX