[PATCH v3 07/14] crypto: blake2s - add comment for blake2s_state fields
Ard Biesheuvel
ardb at kernel.org
Wed Dec 23 04:07:03 EST 2020
On Wed, 23 Dec 2020 at 09:12, Eric Biggers <ebiggers at kernel.org> wrote:
>
> From: Eric Biggers <ebiggers at google.com>
>
> The first three fields of 'struct blake2s_state' are used in assembly
> code, which isn't immediately obvious, so add a comment to this effect.
>
> Signed-off-by: Eric Biggers <ebiggers at google.com>
Acked-by: Ard Biesheuvel <ardb at kernel.org>
> ---
> include/crypto/blake2s.h | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/include/crypto/blake2s.h b/include/crypto/blake2s.h
> index 734ed22b7a6aa..f1c8330a61a91 100644
> --- a/include/crypto/blake2s.h
> +++ b/include/crypto/blake2s.h
> @@ -24,6 +24,7 @@ enum blake2s_lengths {
> };
>
> struct blake2s_state {
> + /* 'h', 't', and 'f' are used in assembly code, so keep them as-is. */
> u32 h[8];
> u32 t[2];
> u32 f[2];
> --
> 2.29.2
>
More information about the linux-arm-kernel
mailing list