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

Corentin Labbe
2025-05-16T12:41:07Z

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%@+4054, 57.95%@+4074]"
[ 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%@+4058, 91.14%@+164] 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 ]---