On Fri, May 11, 2018 at 9:59 AM, Abdul Haleem
[off-list ref] wrote:

Greeting's

Today's next kernel on powerpc machine has boot warnings with commit

a794d8d : crypto: ccree - enable support for hardware keys

Adding the crypto list and maintainer as it came in via the crypto tree.

Warning disappears when above commit is reverted.

Machine Type: Power8 PowerVM LPAR
kernel : 4.17.0-rc4-next-20180509
config: attached.
test: kexec boot

trace logs:
-----------
Initialise system trusted keyrings
workingset: timestamp_bits=38 max_order=18 bucket_order=0
zbud: loaded
pstore: using deflate compression
WARNING: CPU: 6 PID: 109 at crypto/testmgr.c:3720 alg_test.part.6+0xd4/0x460
Modules linked in:
CPU: 6 PID: 109 Comm: cryptomgr_test Not tainted 4.17.0-rc4-next-20180509-autotest-00002-g0244ae7 #2
NIP:  c0000000004a60f4 LR: c0000000004a60e4 CTR: c0000000004a0200
REGS: c000000289acb9d0 TRAP: 0700   Not tainted  (4.17.0-rc4-next-20180509-autotest-00002-g0244ae7)
MSR:  8000000000029033 <SF,EE,ME,IR,DR,RI,LE>  CR: 24000484  XER: 20000000
CFAR: c0000000009b9110 SOFTE: 0
GPR00: c0000000004a0260 c000000289acbc50 c00000000115df00 0000000000000001
GPR04: c000000000c13e64 000000000000000d 0000000000000400 0000000000000000
GPR08: 0000000000000000 0000000000000001 000000000000006b 0000000000000000
GPR12: 0000000000000000 c00000000eca7800 c000000000128918 c0000002812a3600
GPR16: 0000000000000000 0000000000000000 0000000000000000 0000000000000000
GPR20: 0000000000000000 c000000000c13560 c000000288c29880 0000000000000400
GPR24: c000000288c29800 000000000000000d c000000000c135a8 c000000000a1db38
GPR28: c000000000c13128 c000000000c13e60 c000000000a1c788 0000000000000001
NIP [c0000000004a60f4] alg_test.part.6+0xd4/0x460
LR [c0000000004a60e4] alg_test.part.6+0xc4/0x460
Call Trace:
[c000000289acbc50] [c000000289acbcc0] 0xc000000289acbcc0 (unreliable)
[c000000289acbd90] [c0000000004a0260] cryptomgr_test+0x60/0x80
[c000000289acbdc0] [c000000000128a68] kthread+0x158/0x1a0
[c000000289acbe30] [c00000000000b628] ret_from_kernel_thread+0x5c/0xb4
Instruction dump:
3b5a56a8 48000024 60000000 eb9effc8 ebbe0000 7f83e378 7fa4eb78 48513001
60000000 7c7f1b78 7d3f00d0 79290fe0 <0b090000> 2f9f0000 419d013c 7fe90034
---[ end trace f8ddd7633e720997 ]---
testmgr: alg_test_descs entries in wrong order: 'ecb(paes)' before 'ecb(khazad)'

Oouch! this is is my bad. I've changed the cipher name (from haes to
paes) based on review feedback
but forgot that the cipher test names are alphabetically ordered.

I wonder why I didn't see this myself as I'm 100% sure booted this fix
on 2 different platforms.

Maybe because Khazad was not enabled in my test setup and I guess
there are no other cipher starting in I, L,M,N,O ?

Fix coming up shortly. Sorry about this.

Gilad



-- 
Gilad Ben-Yossef
Chief Coffee Drinker

"If you take a class in large-scale robotics, can you end up in a
situation where the homework eats your dog?"
 -- Jean-Baptiste Queru