linux-kernel - Re: [PATCH] crypto: marvell/cesa - Avoid empty transfer descriptor

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: <aCcyXkeBvHQYvf2d@Red>
Date: Fri, 16 May 2025 14:41:02 +0200
From: Corentin Labbe <clabbe.montjoie@...il.com>
To: Herbert Xu <herbert@...dor.apana.org.au>
Cc: Klaus Kudielka <klaus.kudielka@...il.com>,
	Eric Biggers <ebiggers@...nel.org>, regressions@...ts.linux.dev,
	linux-kernel@...r.kernel.org,
	Linux Crypto Mailing List <linux-crypto@...r.kernel.org>,
	Boris Brezillon <bbrezillon@...nel.org>,
	EBALARD Arnaud <Arnaud.Ebalard@....gouv.fr>,
	Romain Perier <romain.perier@...il.com>
Subject: Re: [PATCH] crypto: marvell/cesa - Avoid empty transfer descriptor

Le Fri, May 16, 2025 at 07:25:49AM +0800, Herbert Xu a écrit :
> On Thu, May 15, 2025 at 08:45:39PM +0200, Klaus Kudielka wrote:
> >
> > ...and the failing marvell-cesa self-tests seem to have magically disappeared.
> > I now had five successful reboot / modprobe marvell-cesa in a row.
> 
> It's always unfortunate when a printk patch makes the problem
> go away :)
> 
> Correntin, can you still reproduce the failures with the latest
> cryptodev tree?

Yes I have still errors:
[   12.798454] marvell-cesa f1090000.crypto: CESA device successfully registered
[   13.282357] alg: ahash: mv-sha256 test failed (wrong result) on test vector 1, cfg="random: inplace_one_sglist use_final src_divs=[<flush>42.5%@...54, 57.95%@...74]"
[   13.286675] alg: ahash: mv-sha1 test failed (wrong result) on test vector "random: psize=47 ksize=0", cfg="random: inplace_one_sglist use_final src_divs=[<reimport,nosimd>8.86%@...58, 91.14%@...4] iv_offset=91 key_offset=58"
[   13.297245] alg: self-tests for sha256 using mv-sha256 failed (rc=-22)
Setting prompt string to ['-+\\[ end trace \\w* \\]-+[^\\n]*\\r', '/ #', '~ #', 'sh-5.1#', 'Login timed out', 'Login incorrect']
[   13.317153] ------------[ cut here ]------------
[   13.317157] alg: self-tests for sha1 using mv-sha1 failed (rc=-22)
[   13.323696] WARNING: CPU: 1 PID: 149 at crypto/testmgr.c:5808 alg_test+0x42c/0x654
[   13.328333] ------------[ cut here ]------------
[   13.334524] alg: self-tests for sha256 using mv-sha256 failed (rc=-22)
[   13.342146] WARNING: CPU: 0 PID: 148 at crypto/testmgr.c:5808 alg_test+0x42c/0x654
[   13.346745] Modules linked in: md5 marvell_cesa
[   13.353288] alg: self-tests for sha1 using mv-sha1 failed (rc=-22)
[   13.360875]  libdes sfp
[   13.365414] Modules linked in:
[   13.371609]  mdio_i2c
[   13.374059]  md5
[   13.377121] CPU: 1 UID: 0 PID: 149 Comm: cryptomgr_test Not tainted 6.15.0-rc5-g1bafd82d9a40 #105 NONE 
[   13.377130] Hardware name: Marvell Armada 380/385 (Device Tree)
[   13.377133] Call trace: 
[   13.377139]  unwind_backtrace from show_stack+0x10/0x14
[   13.377153]  show_stack from dump_stack_lvl+0x54/0x68
[   13.377164]  dump_stack_lvl from __warn+0x80/0x124
[   13.377177]  __warn from warn_slowpath_fmt+0x124/0x18c
[   13.377190]  warn_slowpath_fmt from alg_test+0x42c/0x654
[   13.377201]  alg_test from cryptomgr_test+0x18/0x38
[   13.377208]  cryptomgr_test from kthread+0x108/0x234
[   13.377221]  kthread from ret_from_fork+0x14/0x28
[   13.377231] Exception stack(0xf0b11fb0 to 0xf0b11ff8)
[   13.377236] 1fa0:                                     00000000 00000000 00000000 00000000
[   13.377241] 1fc0: 00000000 00000000 00000000 00000000 00000000 00000000 00000000 00000000
[   13.377246] 1fe0: 00000000 00000000 00000000 00000000 00000013 00000000
[   13.377249] ---[ end trace 0000000000000000 ]---