From: Loup Vaillant <loup@loup-vaillant.fr>
Date: Thu, 11 Jan 2018 17:35:07 +0000 (+0100)
Subject: Optimised chacha20 loading & unloading
X-Git-Url: https://git.codecow.com/?a=commitdiff_plain;h=d1be682;p=Monocypher.git

Optimised chacha20 loading & unloading

Testing inside the loop caused a noticeable slow down.  Now the whole
thing looks kinda like a copy-pasta job, but the alternative was a less
readable data flow anyway.
---

diff --git a/src/monocypher.c b/src/monocypher.c
index a982ce5..261dab8 100644
--- a/src/monocypher.c
+++ b/src/monocypher.c
@@ -214,51 +214,67 @@ void crypto_chacha20_encrypt(crypto_chacha_ctx *ctx,
                              const u8          *plain_text,
                              size_t             text_size)
 {
-    // Align ourselves with a block
-    while (ctx->pool_idx % 64 != 0 && text_size > 0) {
-        u8 stream = chacha20_pool_byte(ctx);
-        u8 plain  = 0;
-        if (plain_text != 0) {
-            plain = *plain_text;
+    if (plain_text != 0) {
+        // Align ourselves with a block
+        while (ctx->pool_idx % 64 != 0 && text_size > 0) {
+            *cipher_text = chacha20_pool_byte(ctx) ^ *plain_text;
             plain_text++;
+            cipher_text++;
+            text_size--;
         }
-        *cipher_text = stream ^ plain;
-        text_size--;
-        cipher_text++;
-    }
-    // Main processing by 64 byte chunks
-    size_t nb_blocks = text_size / 64;
-    size_t remainder = text_size % 64;
-    FOR (i, 0, nb_blocks) {
-        chacha20_refill_pool(ctx);
-        FOR (j, 0, 16) {
-            u32 txt;
-            if (plain_text) {
-                txt = load32_le(plain_text);
+        // Main processing by 64 byte chunks
+        size_t nb_blocks = text_size / 64;
+        size_t remainder = text_size % 64;
+        FOR (i, 0, nb_blocks) {
+            chacha20_refill_pool(ctx);
+            FOR (j, 0, 16) {
+                store32_le(cipher_text + j * 4,
+                           ctx->pool[j] ^ load32_le(plain_text));
                 plain_text += 4;
-            } else {
-                txt = 0;
             }
-            store32_le(cipher_text + j * 4, ctx->pool[j] ^ txt);
+            cipher_text += 64;
+        }
+        if (nb_blocks > 0) {
+            ctx->pool_idx = 64;
+        }
+        // Remaining input, byte by byte
+        FOR (i, 0, remainder) {
+            if (ctx->pool_idx == 64) {
+                chacha20_refill_pool(ctx);
+            }
+            *cipher_text = chacha20_pool_byte(ctx) ^ *plain_text;
+            plain_text++;
+            cipher_text++;
         }
-        cipher_text += 64;
-    }
-    if (nb_blocks > 0) {
-        ctx->pool_idx = 64;
     }
-    // Remaining input, byte by byte
-    FOR (i, 0, remainder) {
-        if (ctx->pool_idx == 64) {
+    else {
+        // Align ourselves with a block
+        while (ctx->pool_idx % 64 != 0 && text_size > 0) {
+            *cipher_text = chacha20_pool_byte(ctx);
+            cipher_text++;
+            text_size--;
+        }
+        // Main processing by 64 byte chunks
+        size_t nb_blocks = text_size / 64;
+        size_t remainder = text_size % 64;
+        FOR (i, 0, nb_blocks) {
             chacha20_refill_pool(ctx);
+            FOR (j, 0, 16) {
+                store32_le(cipher_text + j * 4, ctx->pool[j]);
+            }
+            cipher_text += 64;
         }
-        u8 stream = chacha20_pool_byte(ctx);
-        u8 plain  = 0;
-        if (plain_text != 0) {
-            plain = *plain_text;
-            plain_text++;
+        if (nb_blocks > 0) {
+            ctx->pool_idx = 64;
+        }
+        // Remaining input, byte by byte
+        FOR (i, 0, remainder) {
+            if (ctx->pool_idx == 64) {
+                chacha20_refill_pool(ctx);
+            }
+            *cipher_text = chacha20_pool_byte(ctx);
+            cipher_text++;
         }
-        *cipher_text = stream ^ plain;
-        cipher_text++;
     }
 }