Re: [PATCH net-next v5 04/20] zinc: ChaCha20 x86_64 implementation
From: Thomas Gleixner <hidden>
Date: 2018-09-18 22:30:03
Also in:
linux-crypto, lkml
Jason, On Tue, 18 Sep 2018, Jason A. Donenfeld wrote:
This provides SSSE3, AVX-2, AVX-512F, and AVX-512VL implementations for ChaCha20. The AVX-512F implementation is disabled on Skylake, due to throttling, and the VL ymm implementation is used instead. These come from Andy Polyakov's implementation, with the following modifications from Samuel Neves:
...
Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com> Signed-off-by: Samuel Neves <redacted>
I'm a bit confused by this SOB chain. So above you write that it's from Andy Polakovs implementation and Samuel did the changes. But here it seems you are the main author. If Samuel just did some modifications then you want to use the Co-developed-by tag along with his SOB. Also I'd recommend to add a Originally-by or Based-on-code-from: Andy Polyakov tag. Both are not formal tags but widely in use for attributions.
quoted hunk ↗ jump to hunk
+++ b/lib/zinc/chacha20/chacha20-x86_64-glue.h@@ -0,0 +1,100 @@ +/* SPDX-License-Identifier: MIT
Please put that into a separate one liner comment. Also this should be 'GPL-2.0[+] or MIT' I think.
quoted hunk ↗ jump to hunk
+++ b/lib/zinc/chacha20/chacha20-x86_64.S@@ -0,0 +1,2632 @@ +/* SPDX-License-Identifier: BSD-3-Clause
See above. That applies to all the others as well. Thanks, tglx