On Wed, 23 Dec 2020 at 09:12, Eric Biggers <[email protected]> wrote: > > From: Eric Biggers <[email protected]> > > 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 <[email protected]>
Acked-by: Ard Biesheuvel <[email protected]> > --- > 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 >
