console noise after commit c1e9b3b0eea

2016-04-19 Thread Sowmini Varadhan
Hi Anatoly, after commit c1e9b3b0eea1 ("hwrng: n2 - Attach on T5/M5, T7/M7 SPARC CPUs") I get a *lot* of console noise on my T5-2, of the form: n2rng f028f21c: Selftest failed on unit 0 n2rng f028f21c: Test buffer slot 0 [0x] n2rng f028f21c: Test buffer slot 1

[PATCH 1/2] crypto: talitos - fix crash in talitos_cra_init()

2016-04-19 Thread Horia Geantă
From: Jonas Eymann Conversion of talitos driver to the new AEAD interface hasn't been properly tested. AEAD algorithms crash in talitos_cra_init as follows: [...] [1.141095] talitos ffe3.crypto: hwrng [1.145381] Unable to handle kernel paging request for data at

[PATCH 2/2] crypto: talitos - fix AEAD tcrypt tests

2016-04-19 Thread Horia Geantă
After conversion to new AEAD interface, tcrypt tests fail as follows: [...] [1.145414] alg: aead: Test 1 failed on encryption for authenc-hmac-sha1-cbc-aes-talitos [1.153564] : 53 69 6e 67 6c 65 20 62 6c 6f 63 6b 20 6d 73 67 [1.160041] 0010: 00 00 00 00 00 00 00 00 00 00

[PATCH 0/2] crypto: talitos - fix conversion to new AEAD interface

2016-04-19 Thread Horia Geantă
Jonas Eymann reports a crash and some failures of AED algorithms with kernel 4.4.6, all easily reproducible with tcrypt. These have to do with the onversion to the new AEAD interface: aeb4c132f33d crypto: talitos - Convert to new AEAD interface Problems occur for 4.3+ kernels, sending patches to

Re: [PATCH] crypto: marvell/cesa: Improving code readability

2016-04-19 Thread Boris Brezillon
On Tue, 19 Apr 2016 17:09:20 +0200 Romain Perier wrote: > When looking for available engines, the variable "engine" is > assigned to ">engines[i]" at the beginning of the for loop. Replacing > next occurences of ">engines[i]" by "engine" and in order to improve

[PATCH] crypto: marvell/cesa: Improving code readability

2016-04-19 Thread Romain Perier
When looking for available engines, the variable "engine" is assigned to ">engines[i]" at the beginning of the for loop. Replacing next occurences of ">engines[i]" by "engine" and in order to improve readability. Signed-off-by: Romain Perier ---

[PATCH 2/2] crypto: s5p-sss - Remove useless hash interrupt handler

2016-04-19 Thread Krzysztof Kozlowski
Beside regular feed control interrupt, the driver requires also hash interrupt for older SoCs (samsung,s5pv210-secss). However after requesting it, the interrupt handler isn't doing anything with it, not even clearing the hash interrupt bit. Driver does not provide hash functions so it is safe to

[PATCH 1/2] crypto: s5p-sss - Fix use after free of copied input buffer in error path

2016-04-19 Thread Krzysztof Kozlowski
The driver makes copies of memory (input or output scatterlists) if they are not aligned. In s5p_aes_crypt_start() error path (on unsuccessful initialization of output scatterlist), if input scatterlist was not aligned, the driver first freed copied input memory and then unmapped it from the

Re: [PATCH v5 6/6] crypto: AF_ALG - add support for key_id

2016-04-19 Thread Philip Li
On Fri, Apr 15, 2016 at 02:32:47PM -0700, Tadeusz Struk wrote: > On 04/15/2016 02:32 PM, kbuild test robot wrote: > > Hi Tadeusz, > > > > [auto build test ERROR on cryptodev/master] > > [also build test ERROR on v4.6-rc3 next-20160415] > > [if your patch is applied to the wrong git tree, please