Re: [PATCH 1/3] crypto: sm4 - create SM4 library based on sm4 generic code

From: Tianjia Zhang
Date: Tue Mar 01 2022 - 06:50:24 EST


Hi Jason,

On 3/1/22 6:34 PM, Jason A. Donenfeld wrote:
lib/crypto/Kconfig | 3 +
lib/crypto/Makefile | 3 +
lib/crypto/sm4.c | 184 +++++++++++++++++++++++++++++++++++++++++++

If this is only used by the crypto API, it does not belong in
lib/crypto. I understand you want fallback generic code for the SIMD
implementation, but we've generally done that in crypto/ when the use
case is only the crypto API. Can you move this to the right place?

This is not only used by the crypto API, but also used for SIMD
acceleration under the x86 and arm architectures, mainly for processing
the remaining blocks after SIMD acceleration. In general, the
performance of SIMD processing a single block is not as good as that of
general software implementations.

Kind regards,
Tianjia