On Mon, Feb 27, 2023 at 11:39 AM Herbert Xu <herbert@xxxxxxxxxxxxxxxxxxx> wrote: > v3 moves the exported state into its own structure so that it > doesn't contain unnecessary data. It also subsumes the "empty > message" patch, as only the stm32_hash_copy_hash hunk was really > needed and it's easiest if we put that here. Thanks Herbert, now the driver probes fine, but unfortunately the tests fail: [ 2.963725] alg: extra crypto tests enabled. This is intended for developer use only. [ 17.106532] Key type encrypted registered [ 23.804729] stm32-hash a03c2000.hash: allocated hmac(sha256) fallback [ 23.812529] alg: ahash: stm32-hmac-sha256 test failed (wrong result) on test vector 0, cfg="init+update+final aligned buffer" [ 23.823902] alg: self-tests for hmac(sha256) using stm32-hmac-sha256 failed (rc=-22) [ 23.823924] ------------[ cut here ]------------ [ 23.836312] WARNING: CPU: 0 PID: 72 at crypto/testmgr.c:5858 alg_test.part.0+0x4d0/0x4dc [ 23.844449] alg: self-tests for hmac(sha256) using stm32-hmac-sha256 failed (rc=-22) [ 23.844460] Modules linked in: [ 23.855261] CPU: 0 PID: 72 Comm: cryptomgr_test Not tainted 6.2.0-12019-g4bc85a2f8f63 #65 [ 23.863440] Hardware name: ST-Ericsson Ux5x0 platform (Device Tree Support) [ 23.870408] unwind_backtrace from show_stack+0x10/0x14 [ 23.875653] show_stack from dump_stack_lvl+0x40/0x4c [ 23.880727] dump_stack_lvl from __warn+0x94/0xc0 [ 23.885450] __warn from warn_slowpath_fmt+0x118/0x164 [ 23.890603] warn_slowpath_fmt from alg_test.part.0+0x4d0/0x4dc [ 23.896540] alg_test.part.0 from cryptomgr_test+0x18/0x38 [ 23.902042] cryptomgr_test from kthread+0xc0/0xc4 [ 23.906847] kthread from ret_from_fork+0x14/0x2c [ 23.911557] Exception stack(0xf0f39fb0 to 0xf0f39ff8) [ 23.916609] 9fa0: 00000000 00000000 00000000 00000000 [ 23.924783] 9fc0: 00000000 00000000 00000000 00000000 00000000 00000000 00000000 00000000 [ 23.932956] 9fe0: 00000000 00000000 00000000 00000000 00000013 00000000 [ 23.939604] ---[ end trace 0000000000000000 ]--- All tests fail on vector 0 or 1. I am trying to debug! Yours, Linus Walleij
