Commit 58d7d18b authored by Herbert Xu's avatar Herbert Xu

crypto: sha512 - Use binary and instead of modulus

The previous patch used the modulus operator over a power of 2
unnecessarily which may produce suboptimal binary code.  This
patch changes changes them to binary ands instead.
Signed-off-by: default avatarHerbert Xu <herbert@gondor.apana.org.au>
parent 51fc6dc8
...@@ -78,7 +78,7 @@ static inline void LOAD_OP(int I, u64 *W, const u8 *input) ...@@ -78,7 +78,7 @@ static inline void LOAD_OP(int I, u64 *W, const u8 *input)
static inline void BLEND_OP(int I, u64 *W) static inline void BLEND_OP(int I, u64 *W)
{ {
W[I % 16] += s1(W[(I-2) % 16]) + W[(I-7) % 16] + s0(W[(I-15) % 16]); W[I & 15] += s1(W[(I-2) & 15]) + W[(I-7) & 15] + s0(W[(I-15) & 15]);
} }
static void static void
...@@ -105,7 +105,7 @@ sha512_transform(u64 *state, const u8 *input) ...@@ -105,7 +105,7 @@ sha512_transform(u64 *state, const u8 *input)
#define SHA512_16_79(i, a, b, c, d, e, f, g, h) \ #define SHA512_16_79(i, a, b, c, d, e, f, g, h) \
BLEND_OP(i, W); \ BLEND_OP(i, W); \
t1 = h + e1(e) + Ch(e, f, g) + sha512_K[i] + W[(i)%16]; \ t1 = h + e1(e) + Ch(e, f, g) + sha512_K[i] + W[(i)&15]; \
t2 = e0(a) + Maj(a, b, c); \ t2 = e0(a) + Maj(a, b, c); \
d += t1; \ d += t1; \
h = t1 + t2 h = t1 + t2
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment