From: Sascha Hauer <s.hauer@pengutronix.de>
To: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>
Cc: barebox@lists.infradead.org
Subject: Re: [PATCH 3/7] crypto: sha1: switch to linux implementation
Date: Wed, 25 Mar 2015 10:06:12 +0100 [thread overview]
Message-ID: <20150325090612.GB9742@pengutronix.de> (raw)
In-Reply-To: <1427202519-15224-3-git-send-email-plagnioj@jcrosoft.com>
On Tue, Mar 24, 2015 at 02:08:35PM +0100, Jean-Christophe PLAGNIOL-VILLARD wrote:
> current implementation
>
> $ ls -al build/versatilpb/arch/arm/pbl/zbarebox.bin
> -rw-r--r-- 1 root root 211095 Mar 24 13:21 build/versatilpb/arch/arm/pbl/zbarebox.bin
>
> linux generic implementation
>
> $ ls -al build/versatilpb/arch/arm/pbl/zbarebox.bin
> -rw-r--r-- 1 root root 210829 Mar 24 13:21 build/versatilpb/arch/arm/pbl/zbarebox.bin
>
> on a compressed lzo barebox we will 266 bytes
>
> Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>
> ---
> crypto/sha1.c | 650 ++++++++++++++++++++++++---------------------------
> include/crypto/sha.h | 95 ++++++++
> include/digest.h | 5 +
> 3 files changed, 411 insertions(+), 339 deletions(-)
> rewrite crypto/sha1.c (92%)
> create mode 100644 include/crypto/sha.h
>
> diff --git a/crypto/sha1.c b/crypto/sha1.c
> dissimilarity index 92%
> index a2ca191..17f6385 100644
> --- a/crypto/sha1.c
> +++ b/crypto/sha1.c
> @@ -1,339 +1,311 @@
> -/*
> - * Heiko Schocher, DENX Software Engineering, hs@denx.de.
> - * based on:
> - * FIPS-180-1 compliant SHA-1 implementation
> - *
> - * Copyright (C) 2003-2006 Christophe Devine
> - *
> - * This library is free software; you can redistribute it and/or
> - * modify it under the terms of the GNU Lesser General Public
> - * License, version 2.1 as published by the Free Software Foundation.
> - *
> - * This library is distributed in the hope that it will be useful,
> - * but WITHOUT ANY WARRANTY; without even the implied warranty of
> - * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
> - * Lesser General Public License for more details.
> - */
> -/*
> - * The SHA-1 standard was published by NIST in 1993.
> - *
> - * http://www.itl.nist.gov/fipspubs/fip180-1.htm
> - */
> -
> -#include <common.h>
> -#include <digest.h>
> -#include <init.h>
> -#include <linux/string.h>
> -#include <asm/byteorder.h>
> -
> -#include <crypto/internal.h>
> -
> -#define SHA1_SUM_POS -0x20
> -#define SHA1_SUM_LEN 20
> -
> -typedef struct
> -{
> - uint32_t total[2]; /*!< number of bytes processed */
> - uint32_t state[5]; /*!< intermediate digest state */
> - uint8_t buffer[64]; /*!< data block being processed */
> -}
> -sha1_context;
> -
> -/*
> - * 32-bit integer manipulation macros (big endian)
> - */
> -#define GET_UINT32_BE(n,b,i) (n) = be32_to_cpu(((uint32_t*)(b))[i / 4])
> -#define PUT_UINT32_BE(n,b,i) ((uint32_t*)(b))[i / 4] = cpu_to_be32(n)
> -
> -/*
> - * SHA-1 context setup
> - */
> -static void sha1_starts (sha1_context *ctx)
> -{
> - ctx->total[0] = 0;
> - ctx->total[1] = 0;
> -
> - ctx->state[0] = 0x67452301;
> - ctx->state[1] = 0xEFCDAB89;
> - ctx->state[2] = 0x98BADCFE;
> - ctx->state[3] = 0x10325476;
> - ctx->state[4] = 0xC3D2E1F0;
> -}
> -
> -static void sha1_process (sha1_context *ctx, uint8_t data[64])
> -{
> - uint32_t temp, W[16], A, B, C, D, E;
> -
> - GET_UINT32_BE (W[0], data, 0);
> - GET_UINT32_BE (W[1], data, 4);
> - GET_UINT32_BE (W[2], data, 8);
> - GET_UINT32_BE (W[3], data, 12);
> - GET_UINT32_BE (W[4], data, 16);
> - GET_UINT32_BE (W[5], data, 20);
> - GET_UINT32_BE (W[6], data, 24);
> - GET_UINT32_BE (W[7], data, 28);
> - GET_UINT32_BE (W[8], data, 32);
> - GET_UINT32_BE (W[9], data, 36);
> - GET_UINT32_BE (W[10], data, 40);
> - GET_UINT32_BE (W[11], data, 44);
> - GET_UINT32_BE (W[12], data, 48);
> - GET_UINT32_BE (W[13], data, 52);
> - GET_UINT32_BE (W[14], data, 56);
> - GET_UINT32_BE (W[15], data, 60);
> -
> -#define S(x,n) ((x << n) | ((x & 0xFFFFFFFF) >> (32 - n)))
> -
> -#define R(t) ( \
> - temp = W[(t - 3) & 0x0F] ^ W[(t - 8) & 0x0F] ^ \
> - W[(t - 14) & 0x0F] ^ W[ t & 0x0F], \
> - ( W[t & 0x0F] = S(temp,1) ) \
> -)
> -
> -#define P(a,b,c,d,e,x) { \
> - e += S(a,5) + F(b,c,d) + K + x; b = S(b,30); \
> -}
> -
> - A = ctx->state[0];
> - B = ctx->state[1];
> - C = ctx->state[2];
> - D = ctx->state[3];
> - E = ctx->state[4];
> -
> -#define F(x,y,z) (z ^ (x & (y ^ z)))
> -#define K 0x5A827999
> -
> - P (A, B, C, D, E, W[0]);
> - P (E, A, B, C, D, W[1]);
> - P (D, E, A, B, C, W[2]);
> - P (C, D, E, A, B, W[3]);
> - P (B, C, D, E, A, W[4]);
> - P (A, B, C, D, E, W[5]);
> - P (E, A, B, C, D, W[6]);
> - P (D, E, A, B, C, W[7]);
> - P (C, D, E, A, B, W[8]);
> - P (B, C, D, E, A, W[9]);
> - P (A, B, C, D, E, W[10]);
> - P (E, A, B, C, D, W[11]);
> - P (D, E, A, B, C, W[12]);
> - P (C, D, E, A, B, W[13]);
> - P (B, C, D, E, A, W[14]);
> - P (A, B, C, D, E, W[15]);
> - P (E, A, B, C, D, R (16));
> - P (D, E, A, B, C, R (17));
> - P (C, D, E, A, B, R (18));
> - P (B, C, D, E, A, R (19));
> -
> -#undef K
> -#undef F
> -
> -#define F(x,y,z) (x ^ y ^ z)
> -#define K 0x6ED9EBA1
> -
> - P (A, B, C, D, E, R (20));
> - P (E, A, B, C, D, R (21));
> - P (D, E, A, B, C, R (22));
> - P (C, D, E, A, B, R (23));
> - P (B, C, D, E, A, R (24));
> - P (A, B, C, D, E, R (25));
> - P (E, A, B, C, D, R (26));
> - P (D, E, A, B, C, R (27));
> - P (C, D, E, A, B, R (28));
> - P (B, C, D, E, A, R (29));
> - P (A, B, C, D, E, R (30));
> - P (E, A, B, C, D, R (31));
> - P (D, E, A, B, C, R (32));
> - P (C, D, E, A, B, R (33));
> - P (B, C, D, E, A, R (34));
> - P (A, B, C, D, E, R (35));
> - P (E, A, B, C, D, R (36));
> - P (D, E, A, B, C, R (37));
> - P (C, D, E, A, B, R (38));
> - P (B, C, D, E, A, R (39));
> -
> -#undef K
> -#undef F
> -
> -#define F(x,y,z) ((x & y) | (z & (x | y)))
> -#define K 0x8F1BBCDC
> -
> - P (A, B, C, D, E, R (40));
> - P (E, A, B, C, D, R (41));
> - P (D, E, A, B, C, R (42));
> - P (C, D, E, A, B, R (43));
> - P (B, C, D, E, A, R (44));
> - P (A, B, C, D, E, R (45));
> - P (E, A, B, C, D, R (46));
> - P (D, E, A, B, C, R (47));
> - P (C, D, E, A, B, R (48));
> - P (B, C, D, E, A, R (49));
> - P (A, B, C, D, E, R (50));
> - P (E, A, B, C, D, R (51));
> - P (D, E, A, B, C, R (52));
> - P (C, D, E, A, B, R (53));
> - P (B, C, D, E, A, R (54));
> - P (A, B, C, D, E, R (55));
> - P (E, A, B, C, D, R (56));
> - P (D, E, A, B, C, R (57));
> - P (C, D, E, A, B, R (58));
> - P (B, C, D, E, A, R (59));
> -
> -#undef K
> -#undef F
> -
> -#define F(x,y,z) (x ^ y ^ z)
> -#define K 0xCA62C1D6
> -
> - P (A, B, C, D, E, R (60));
> - P (E, A, B, C, D, R (61));
> - P (D, E, A, B, C, R (62));
> - P (C, D, E, A, B, R (63));
> - P (B, C, D, E, A, R (64));
> - P (A, B, C, D, E, R (65));
> - P (E, A, B, C, D, R (66));
> - P (D, E, A, B, C, R (67));
> - P (C, D, E, A, B, R (68));
> - P (B, C, D, E, A, R (69));
> - P (A, B, C, D, E, R (70));
> - P (E, A, B, C, D, R (71));
> - P (D, E, A, B, C, R (72));
> - P (C, D, E, A, B, R (73));
> - P (B, C, D, E, A, R (74));
> - P (A, B, C, D, E, R (75));
> - P (E, A, B, C, D, R (76));
> - P (D, E, A, B, C, R (77));
> - P (C, D, E, A, B, R (78));
> - P (B, C, D, E, A, R (79));
> -
> -#undef K
> -#undef F
> -
> - ctx->state[0] += A;
> - ctx->state[1] += B;
> - ctx->state[2] += C;
> - ctx->state[3] += D;
> - ctx->state[4] += E;
> -}
> -
> -/*
> - * SHA-1 process buffer
> - */
> -static void sha1_update (sha1_context *ctx, uint8_t *input, uint32_t ilen)
> -{
> - uint32_t fill, left;
> -
> - if (ilen <= 0)
> - return;
> -
> - left = ctx->total[0] & 0x3F;
> - fill = 64 - left;
> -
> - ctx->total[0] += ilen;
> - ctx->total[0] &= 0xFFFFFFFF;
> -
> - if (ctx->total[0] < ilen)
> - ctx->total[1]++;
> -
> - if (left && ilen >= fill) {
> - memcpy ((void *) (ctx->buffer + left), (void *) input, fill);
> - sha1_process (ctx, ctx->buffer);
> - input += fill;
> - ilen -= fill;
> - left = 0;
> - }
> -
> - while (ilen >= 64) {
> - sha1_process (ctx, input);
> - input += 64;
> - ilen -= 64;
> - }
> -
> - if (ilen > 0) {
> - memcpy ((void *) (ctx->buffer + left), (void *) input, ilen);
> - }
> -}
> -
> -static uint8_t sha1_padding[64] = {
> - 0x80, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
> - 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
> - 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
> - 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0
> -};
> -
> -/*
> - * SHA-1 final digest
> - */
> -static void sha1_finish (sha1_context *ctx, uint8_t output[20])
> -{
> - uint32_t last, padn;
> - uint32_t high, low;
> - uint8_t msglen[8];
> -
> - high = (ctx->total[0] >> 29)
> - | (ctx->total[1] << 3);
> - low = (ctx->total[0] << 3);
> -
> - PUT_UINT32_BE (high, msglen, 0);
> - PUT_UINT32_BE (low, msglen, 4);
> -
> - last = ctx->total[0] & 0x3F;
> - padn = (last < 56) ? (56 - last) : (120 - last);
> -
> - sha1_update (ctx, sha1_padding, padn);
> - sha1_update (ctx, msglen, 8);
> -
> - PUT_UINT32_BE (ctx->state[0], output, 0);
> - PUT_UINT32_BE (ctx->state[1], output, 4);
> - PUT_UINT32_BE (ctx->state[2], output, 8);
> - PUT_UINT32_BE (ctx->state[3], output, 12);
> - PUT_UINT32_BE (ctx->state[4], output, 16);
> -}
> -
> -static int digest_sha1_init(struct digest *d)
> -{
> - sha1_starts(d->ctx);
> -
> - return 0;
> -}
> -
> -static int digest_sha1_update(struct digest *d, const void *data,
> - unsigned long len)
This removes digest_sha1_update...
> +static int sha1_final(struct digest *desc, unsigned char *md)
> +{
> + struct sha1_state *sctx = digest_ctx(desc);
> + __be32 *dst = (__be32 *)md;
> + u32 i, index, padlen;
> + __be64 bits;
> + static const u8 padding[64] = { 0x80, };
> +
> + bits = cpu_to_be64(sctx->count << 3);
> +
> + /* Pad out to 56 mod 64 */
> + index = sctx->count & 0x3f;
> + padlen = (index < 56) ? (56 - index) : ((64+56) - index);
> + digest_sha1_update(desc, padding, padlen);
...but calls it here, so compilation fails with:
crypto/sha1.c: In function 'sha1_final':
crypto/sha1.c:270:2: error: implicit declaration of function 'digest_sha1_update' [-Werror=implicit-function-declaration]
Sascha
--
Pengutronix e.K. | |
Industrial Linux Solutions | http://www.pengutronix.de/ |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 |
Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox
next prev parent reply other threads:[~2015-03-25 9:06 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-03-24 13:00 [PATCH 0/7 v2] digest: allow multiple implementation of digest Jean-Christophe PLAGNIOL-VILLARD
2015-03-24 13:08 ` [PATCH 1/7] command: digest: only set the key when specified Jean-Christophe PLAGNIOL-VILLARD
2015-03-24 13:08 ` [PATCH 2/7] crypto: prepare to allow multiple digest driver Jean-Christophe PLAGNIOL-VILLARD
2015-03-24 13:08 ` [PATCH 3/7] crypto: sha1: switch to linux implementation Jean-Christophe PLAGNIOL-VILLARD
2015-03-25 9:06 ` Sascha Hauer [this message]
2015-03-24 13:08 ` [PATCH 4/7] crypto: sha256: " Jean-Christophe PLAGNIOL-VILLARD
2015-03-24 13:08 ` [PATCH 5/7] crypto: sha512: " Jean-Christophe PLAGNIOL-VILLARD
2015-03-24 13:08 ` [PATCH 6/7] arm: crypto: add sha1 assembly support Jean-Christophe PLAGNIOL-VILLARD
2015-03-24 13:08 ` [PATCH 7/7] arm: crypto: add sha256 " Jean-Christophe PLAGNIOL-VILLARD
2015-03-25 2:55 ` [PATCH 0/7 v2] digest: allow multiple implementation of digest Jean-Christophe PLAGNIOL-VILLARD
2015-03-25 10:26 ` Sascha Hauer
2015-03-25 11:44 ` Jean-Christophe PLAGNIOL-VILLARD
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20150325090612.GB9742@pengutronix.de \
--to=s.hauer@pengutronix.de \
--cc=barebox@lists.infradead.org \
--cc=plagnioj@jcrosoft.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox