linux-kernel - Re: [PATCH net-next v6 03/23] zinc: ChaCha20 generic C implementation and selftest

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20181002031542.dbxu44gvqrughlgw@gondor.apana.org.au>
Date:   Tue, 2 Oct 2018 11:15:42 +0800
From:   Herbert Xu <herbert@...dor.apana.org.au>
To:     "Jason A. Donenfeld" <Jason@...c4.com>
Cc:     linux-kernel@...r.kernel.org, netdev@...r.kernel.org,
        linux-crypto@...r.kernel.org, davem@...emloft.net,
        gregkh@...uxfoundation.org, Jason@...c4.com, sneves@....uc.pt,
        luto@...nel.org, jeanphilippe.aumasson@...il.com
Subject: Re: [PATCH net-next v6 03/23] zinc: ChaCha20 generic C
 implementation and selftest

Jason A. Donenfeld <Jason@...c4.com> wrote:
> This implements the ChaCha20 permutation as a single C statement, by way
> of the comma operator, which the compiler is able to simplify
> terrifically.
> 
> Information: https://cr.yp.to/chacha.html
> 
> Signed-off-by: Jason A. Donenfeld <Jason@...c4.com>
> Cc: Samuel Neves <sneves@....uc.pt>
> Cc: Andy Lutomirski <luto@...nel.org>
> Cc: Greg KH <gregkh@...uxfoundation.org>
> Cc: Jean-Philippe Aumasson <jeanphilippe.aumasson@...il.com>

I don't have any problems with this interface.

However, I'm very unhappy about the fact that this adds another
copy of the chacha20 implementation to the kernel.  Either this
is better than the one that we already have or it isn't.

So if it's better please replace the existing chacha20 C
implementation with this or otherwise just adapt the existing
implementation to this interface.

If we're replacing the existing one then you can either replace
it directly within this patch or in a subsequent patch but please
do it in the same series because otherwise we could wind up with
two implementations for years to come.

Thanks,
-- 
Email: Herbert Xu <herbert@...dor.apana.org.au>
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt