| Message ID | 5de7d155.1c69fb81.c06f8.3583@mx.google.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | ardb/for-kernelci bisection: boot on rk3288-rock2-square | expand |
On 04/12/2019 15:31, kernelci.org bot wrote: > * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * > * This automated bisection report was sent to you on the basis * > * that you may be involved with the breaking commit it has * > * found. No manual investigation has been done to verify it, * > * and the root cause of the problem may be somewhere else. * > * * > * If you do send a fix, please include this trailer: * > * Reported-by: "kernelci.org bot" <bot@kernelci.org> * > * * > * Hope this helps! * > * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * > > ardb/for-kernelci bisection: boot on rk3288-rock2-square > > Summary: > Start: 16839329da69 enable extra tests by default > Details: https://kernelci.org/boot/id/5de79104990bc03e5a960f0b > Plain log: https://storage.kernelci.org//ardb/for-kernelci/v5.4-9340-g16839329da69/arm/multi_v7_defconfig+CONFIG_EFI=y+CONFIG_ARM_LPAE=y/gcc-8/lab-collabora/boot-rk3288-rock2-square.txt > HTML log: https://storage.kernelci.org//ardb/for-kernelci/v5.4-9340-g16839329da69/arm/multi_v7_defconfig+CONFIG_EFI=y+CONFIG_ARM_LPAE=y/gcc-8/lab-collabora/boot-rk3288-rock2-square.html > Result: 16839329da69 enable extra tests by default > > Checks: > revert: PASS > verify: PASS > > Parameters: > Tree: ardb > URL: git://git.kernel.org/pub/scm/linux/kernel/git/ardb/linux.git > Branch: for-kernelci > Target: rk3288-rock2-square > CPU arch: arm > Lab: lab-collabora > Compiler: gcc-8 > Config: multi_v7_defconfig+CONFIG_EFI=y+CONFIG_ARM_LPAE=y > Test suite: boot > > Breaking commit found: > > ------------------------------------------------------------------------------- > commit 16839329da69263e7360f3819bae01bcf4b220ec > Author: Ard Biesheuvel <ardb@kernel.org> > Date: Tue Dec 3 12:29:31 2019 +0000 > > enable extra tests by default > > diff --git a/crypto/Kconfig b/crypto/Kconfig > index 5575d48473bd..36af840aa820 100644 > --- a/crypto/Kconfig > +++ b/crypto/Kconfig > @@ -140,7 +140,6 @@ if CRYPTO_MANAGER2 > > config CRYPTO_MANAGER_DISABLE_TESTS > bool "Disable run-time self tests" > - default y > help > Disable run-time self tests that normally take place at > algorithm registration. > @@ -148,6 +147,7 @@ config CRYPTO_MANAGER_DISABLE_TESTS > config CRYPTO_MANAGER_EXTRA_TESTS > bool "Enable extra run-time crypto self tests" > depends on DEBUG_KERNEL && !CRYPTO_MANAGER_DISABLE_TESTS > + default y > help > Enable extra run-time self tests of registered crypto algorithms, > including randomized fuzz tests. > diff --git a/crypto/testmgr.c b/crypto/testmgr.c > index 88f33c0efb23..5df87bcf6c4d 100644 > --- a/crypto/testmgr.c > +++ b/crypto/testmgr.c > @@ -40,7 +40,7 @@ static bool notests; > module_param(notests, bool, 0644); > MODULE_PARM_DESC(notests, "disable crypto self-tests"); > > -static bool panic_on_fail; > +static bool panic_on_fail = true; > module_param(panic_on_fail, bool, 0444); > > #ifdef CONFIG_CRYPTO_MANAGER_EXTRA_TESTS > ------------------------------------------------------------------------------- Seems legit, from the log: <3>[ 18.186181] rk3288-crypto ff8a0000.cypto-controller: [rk_load_data:123] pcopy err <3>[ 18.199432] alg: skcipher: ecb-aes-rk encryption failed on test vector \"random: len=0 klen=32\"; expected_error=0, actual_error=-22, cfg=\"random: inplace use_finup nosimd src_divs=[100.0%@+2054] key_offset=16\" <0>[ 18.220458] Kernel panic - not syncing: alg: self-tests for ecb-aes-rk (ecb(aes)) failed in panic_on_fail mode! Let me know if you need any help with testing a fix on this platform or anything. Also, as you probably only want this to be enabled in KernelCI and not merged upstream, we could have a config fragment to enable the config with your branch and maybe even others. Guillaume > Git bisection log: > > ------------------------------------------------------------------------------- > git bisect start > # good: [b94ae8ad9fe79da61231999f347f79645b909bda] Merge tag 'seccomp-v5.5-rc1' of git://git.kernel.org/pub/scm/linux/kernel/git/kees/linux > git bisect good b94ae8ad9fe79da61231999f347f79645b909bda > # bad: [16839329da69263e7360f3819bae01bcf4b220ec] enable extra tests by default > git bisect bad 16839329da69263e7360f3819bae01bcf4b220ec > # good: [25cbf24a7eec7c3dee4113b2e98b572e128009b7] crypto: aead - move crypto_aead_maxauthsize() to <crypto/aead.h> > git bisect good 25cbf24a7eec7c3dee4113b2e98b572e128009b7 > # good: [7b19c7a82950ed034645fa92adce29cd6163ed3e] crypto: testmgr - check skcipher min_keysize > git bisect good 7b19c7a82950ed034645fa92adce29cd6163ed3e > # good: [062752a354aaf03b46b86cba5fdaa2fd5c932860] crypto: testmgr - create struct aead_extra_tests_ctx > git bisect good 062752a354aaf03b46b86cba5fdaa2fd5c932860 > # good: [2cd56a00fff8584e342164c65e6b55da61f79c4a] crypto: testmgr - generate inauthentic AEAD test vectors > git bisect good 2cd56a00fff8584e342164c65e6b55da61f79c4a > # first bad commit: [16839329da69263e7360f3819bae01bcf4b220ec] enable extra tests by default > ------------------------------------------------------------------------------- >
(+ Eric) On Wed, 4 Dec 2019 at 17:17, Guillaume Tucker <guillaume.tucker@collabora.com> wrote: > > On 04/12/2019 15:31, kernelci.org bot wrote: > > * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * > > * This automated bisection report was sent to you on the basis * > > * that you may be involved with the breaking commit it has * > > * found. No manual investigation has been done to verify it, * > > * and the root cause of the problem may be somewhere else. * > > * * > > * If you do send a fix, please include this trailer: * > > * Reported-by: "kernelci.org bot" <bot@kernelci.org> * > > * * > > * Hope this helps! * > > * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * > > > > ardb/for-kernelci bisection: boot on rk3288-rock2-square > > > > Summary: > > Start: 16839329da69 enable extra tests by default > > Details: https://kernelci.org/boot/id/5de79104990bc03e5a960f0b > > Plain log: https://storage.kernelci.org//ardb/for-kernelci/v5.4-9340-g16839329da69/arm/multi_v7_defconfig+CONFIG_EFI=y+CONFIG_ARM_LPAE=y/gcc-8/lab-collabora/boot-rk3288-rock2-square.txt > > HTML log: https://storage.kernelci.org//ardb/for-kernelci/v5.4-9340-g16839329da69/arm/multi_v7_defconfig+CONFIG_EFI=y+CONFIG_ARM_LPAE=y/gcc-8/lab-collabora/boot-rk3288-rock2-square.html > > Result: 16839329da69 enable extra tests by default > > > > Checks: > > revert: PASS > > verify: PASS > > > > Parameters: > > Tree: ardb > > URL: git://git.kernel.org/pub/scm/linux/kernel/git/ardb/linux.git > > Branch: for-kernelci > > Target: rk3288-rock2-square > > CPU arch: arm > > Lab: lab-collabora > > Compiler: gcc-8 > > Config: multi_v7_defconfig+CONFIG_EFI=y+CONFIG_ARM_LPAE=y > > Test suite: boot > > > > Breaking commit found: > > > > ------------------------------------------------------------------------------- > > commit 16839329da69263e7360f3819bae01bcf4b220ec > > Author: Ard Biesheuvel <ardb@kernel.org> > > Date: Tue Dec 3 12:29:31 2019 +0000 > > > > enable extra tests by default > > > > diff --git a/crypto/Kconfig b/crypto/Kconfig > > index 5575d48473bd..36af840aa820 100644 > > --- a/crypto/Kconfig > > +++ b/crypto/Kconfig > > @@ -140,7 +140,6 @@ if CRYPTO_MANAGER2 > > > > config CRYPTO_MANAGER_DISABLE_TESTS > > bool "Disable run-time self tests" > > - default y > > help > > Disable run-time self tests that normally take place at > > algorithm registration. > > @@ -148,6 +147,7 @@ config CRYPTO_MANAGER_DISABLE_TESTS > > config CRYPTO_MANAGER_EXTRA_TESTS > > bool "Enable extra run-time crypto self tests" > > depends on DEBUG_KERNEL && !CRYPTO_MANAGER_DISABLE_TESTS > > + default y > > help > > Enable extra run-time self tests of registered crypto algorithms, > > including randomized fuzz tests. > > diff --git a/crypto/testmgr.c b/crypto/testmgr.c > > index 88f33c0efb23..5df87bcf6c4d 100644 > > --- a/crypto/testmgr.c > > +++ b/crypto/testmgr.c > > @@ -40,7 +40,7 @@ static bool notests; > > module_param(notests, bool, 0644); > > MODULE_PARM_DESC(notests, "disable crypto self-tests"); > > > > -static bool panic_on_fail; > > +static bool panic_on_fail = true; > > module_param(panic_on_fail, bool, 0444); > > > > #ifdef CONFIG_CRYPTO_MANAGER_EXTRA_TESTS > > ------------------------------------------------------------------------------- > > > Seems legit, from the log: > > <3>[ 18.186181] rk3288-crypto ff8a0000.cypto-controller: [rk_load_data:123] pcopy err > <3>[ 18.199432] alg: skcipher: ecb-aes-rk encryption failed on test vector \"random: len=0 klen=32\"; expected_error=0, actual_error=-22, cfg=\"random: inplace use_finup nosimd src_divs=[100.0%@+2054] key_offset=16\" > <0>[ 18.220458] Kernel panic - not syncing: alg: self-tests for ecb-aes-rk (ecb(aes)) failed in panic_on_fail mode! > > Let me know if you need any help with testing a fix on this > platform or anything. > This is an expected failure. I pushed this to my branch to check if Eric's new AEAD testing code identifies any new problems on the hardware we have in kernelCI, but it only found stuff we already knew about. > Also, as you probably only want this to be enabled in KernelCI > and not merged upstream, we could have a config fragment to > enable the config with your branch and maybe even others. > It would be *very* helpful if we could add Herbert's cryptodev branch [0] to kernelCI with a kconfig fragment that turns off CRYPTO_MANAGER_DISABLE_TESTS and turns on CRYPTO_MANAGER_EXTRA_TESTS, and passes cryptomgr.panic_on_fail=1 on the kernel command line. That way, we'll have rolling coverage of just the crypto changes queued up in linux-next, but tested thoroughly on actual hardware, and without the need to carry patches like the above to trigger the tests explicitly. [0] https://git.kernel.org/pub/scm/linux/kernel/git/herbert/cryptodev-2.6.git/
diff --git a/crypto/Kconfig b/crypto/Kconfig index 5575d48473bd..36af840aa820 100644 --- a/crypto/Kconfig +++ b/crypto/Kconfig @@ -140,7 +140,6 @@ if CRYPTO_MANAGER2 config CRYPTO_MANAGER_DISABLE_TESTS bool "Disable run-time self tests" - default y help Disable run-time self tests that normally take place at algorithm registration. @@ -148,6 +147,7 @@ config CRYPTO_MANAGER_DISABLE_TESTS config CRYPTO_MANAGER_EXTRA_TESTS bool "Enable extra run-time crypto self tests" depends on DEBUG_KERNEL && !CRYPTO_MANAGER_DISABLE_TESTS + default y help Enable extra run-time self tests of registered crypto algorithms, including randomized fuzz tests. diff --git a/crypto/testmgr.c b/crypto/testmgr.c index 88f33c0efb23..5df87bcf6c4d 100644 --- a/crypto/testmgr.c +++ b/crypto/testmgr.c @@ -40,7 +40,7 @@ static bool notests; module_param(notests, bool, 0644); MODULE_PARM_DESC(notests, "disable crypto self-tests"); -static bool panic_on_fail; +static bool panic_on_fail = true; module_param(panic_on_fail, bool, 0444); #ifdef CONFIG_CRYPTO_MANAGER_EXTRA_TESTS
* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * This automated bisection report was sent to you on the basis * * that you may be involved with the breaking commit it has * * found. No manual investigation has been done to verify it, * * and the root cause of the problem may be somewhere else. * * * * If you do send a fix, please include this trailer: * * Reported-by: "kernelci.org bot" <bot@kernelci.org> * * * * Hope this helps! * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * ardb/for-kernelci bisection: boot on rk3288-rock2-square Summary: Start: 16839329da69 enable extra tests by default Details: https://kernelci.org/boot/id/5de79104990bc03e5a960f0b Plain log: https://storage.kernelci.org//ardb/for-kernelci/v5.4-9340-g16839329da69/arm/multi_v7_defconfig+CONFIG_EFI=y+CONFIG_ARM_LPAE=y/gcc-8/lab-collabora/boot-rk3288-rock2-square.txt HTML log: https://storage.kernelci.org//ardb/for-kernelci/v5.4-9340-g16839329da69/arm/multi_v7_defconfig+CONFIG_EFI=y+CONFIG_ARM_LPAE=y/gcc-8/lab-collabora/boot-rk3288-rock2-square.html Result: 16839329da69 enable extra tests by default Checks: revert: PASS verify: PASS Parameters: Tree: ardb URL: git://git.kernel.org/pub/scm/linux/kernel/git/ardb/linux.git Branch: for-kernelci Target: rk3288-rock2-square CPU arch: arm Lab: lab-collabora Compiler: gcc-8 Config: multi_v7_defconfig+CONFIG_EFI=y+CONFIG_ARM_LPAE=y Test suite: boot Breaking commit found: ------------------------------------------------------------------------------- commit 16839329da69263e7360f3819bae01bcf4b220ec Author: Ard Biesheuvel <ardb@kernel.org> Date: Tue Dec 3 12:29:31 2019 +0000 enable extra tests by default ------------------------------------------------------------------------------- Git bisection log: ------------------------------------------------------------------------------- git bisect start # good: [b94ae8ad9fe79da61231999f347f79645b909bda] Merge tag 'seccomp-v5.5-rc1' of git://git.kernel.org/pub/scm/linux/kernel/git/kees/linux git bisect good b94ae8ad9fe79da61231999f347f79645b909bda # bad: [16839329da69263e7360f3819bae01bcf4b220ec] enable extra tests by default git bisect bad 16839329da69263e7360f3819bae01bcf4b220ec # good: [25cbf24a7eec7c3dee4113b2e98b572e128009b7] crypto: aead - move crypto_aead_maxauthsize() to <crypto/aead.h> git bisect good 25cbf24a7eec7c3dee4113b2e98b572e128009b7 # good: [7b19c7a82950ed034645fa92adce29cd6163ed3e] crypto: testmgr - check skcipher min_keysize git bisect good 7b19c7a82950ed034645fa92adce29cd6163ed3e # good: [062752a354aaf03b46b86cba5fdaa2fd5c932860] crypto: testmgr - create struct aead_extra_tests_ctx git bisect good 062752a354aaf03b46b86cba5fdaa2fd5c932860 # good: [2cd56a00fff8584e342164c65e6b55da61f79c4a] crypto: testmgr - generate inauthentic AEAD test vectors git bisect good 2cd56a00fff8584e342164c65e6b55da61f79c4a # first bad commit: [16839329da69263e7360f3819bae01bcf4b220ec] enable extra tests by default -------------------------------------------------------------------------------