Re: [PATCH] crypto: testmgr: add test vectors for skein

[Eric Biggers]

On Wed, Jun 20, 2018 at 07:57:14PM +0900, Juan Manuel Torres Palma wrote:
> Adds test vectors specified in skein v1.3
> specification. The main code is still in staging
> directory, but this is required to eventually be
> moved to the crypto directory.

Also, can you describe the users of Skein in the kernel?  If there are no users,
there's no need to move it out of staging, or even have it in the kernel at all
anymore.  I say that as someone who has had to volunteer to fix critical bugs
found by fuzzing in crypto algorithms for which it's unclear why they are in the
kernel at all, as there are no apparent users.

By the way, syzbot [1] doesn't have CONFIG_CRYPTO_SKEIN enabled, so the Skein
algorithms aren't being fuzzed currently.

[1] https://github.com/google/syzkaller/blob/master/docs/syzbot.md

- Eric