Re: [PATCH net-next v4 17/20] crypto: port Poly1305 to Zinc

From: kbuild test robot
Date: Sat Sep 15 2018 - 20:36:09 EST

Next message: Kees Cook: "[PATCH 17/18] LSM: Provide init debugging"
Previous message: Kees Cook: "[PATCH 02/18] LSM: Rename .security_initcall section to .lsm_info"
In reply to: Jason A. Donenfeld: "Re: [PATCH net-next v4 17/20] crypto: port Poly1305 to Zinc"
Next in thread: Jason A. Donenfeld: "[PATCH net-next v4 19/20] security/keys: rewrite big_key crypto to use Zinc"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

Hi Jason,

I love your patch! Yet something to improve:

[auto build test ERROR on net-next/master]

url: https://github.com/0day-ci/linux/commits/Jason-A-Donenfeld/WireGuard-Secure-Network-Tunnel/20180916-043623
config: arm64-defconfig
compiler: aarch64-linux-gnu-gcc (Debian 7.2.0-11) 7.2.0
reproduce:
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
GCC_VERSION=7.2.0 make.cross ARCH=arm64 defconfig
GCC_VERSION=7.2.0 make.cross ARCH=arm64

All errors (new ones prefixed by >>):

>> fs/sysfs/Kconfig:1:error: recursive dependency detected!
>> fs/sysfs/Kconfig:1: symbol SYSFS is selected by CONFIGFS_FS
>> fs/configfs/Kconfig:1: symbol CONFIGFS_FS is selected by ACPI_CONFIGFS
>> drivers/acpi/Kconfig:542: symbol ACPI_CONFIGFS depends on ACPI
drivers/acpi/Kconfig:9: symbol ACPI depends on ARCH_SUPPORTS_ACPI
drivers/acpi/Kconfig:6: symbol ARCH_SUPPORTS_ACPI is selected by EFI
arch/arm64/Kconfig:1253: symbol EFI depends on KERNEL_MODE_NEON
arch/arm64/Kconfig:262: symbol KERNEL_MODE_NEON is implied by ZINC_ARCH_ARM
lib/zinc/Kconfig:42: symbol ZINC_ARCH_ARM depends on ZINC
>> lib/zinc/Kconfig:1: symbol ZINC is selected by ZINC_POLY1305
>> lib/zinc/Kconfig:9: symbol ZINC_POLY1305 is selected by CRYPTO_POLY1305
>> crypto/Kconfig:656: symbol CRYPTO_POLY1305 depends on CRYPTO
>> crypto/Kconfig:16: symbol CRYPTO is selected by IP_SCTP
>> net/sctp/Kconfig:5: symbol IP_SCTP is selected by DLM
>> fs/dlm/Kconfig:1: symbol DLM depends on SYSFS
For a resolution refer to Documentation/kbuild/kconfig-language.txt
subsection "Kconfig recursive dependency limitations"

vim +1 lib/zinc/Kconfig

32bbe22e Jason A. Donenfeld 2018-09-14 @1 config ZINC
32bbe22e Jason A. Donenfeld 2018-09-14 2 tristate
32bbe22e Jason A. Donenfeld 2018-09-14 3
35f45248 Jason A. Donenfeld 2018-09-14 4 config ZINC_CHACHA20
35f45248 Jason A. Donenfeld 2018-09-14 5 bool
35f45248 Jason A. Donenfeld 2018-09-14 6 select ZINC
35f45248 Jason A. Donenfeld 2018-09-14 7 select CRYPTO_ALGAPI
35f45248 Jason A. Donenfeld 2018-09-14 8
0a36c146 Jason A. Donenfeld 2018-09-14 @9 config ZINC_POLY1305
0a36c146 Jason A. Donenfeld 2018-09-14 10 bool
0a36c146 Jason A. Donenfeld 2018-09-14 11 select ZINC
0a36c146 Jason A. Donenfeld 2018-09-14 12
1b5dbb86 Jason A. Donenfeld 2018-09-14 13 config ZINC_CHACHA20POLY1305
1b5dbb86 Jason A. Donenfeld 2018-09-14 14 bool
1b5dbb86 Jason A. Donenfeld 2018-09-14 15 select ZINC
1b5dbb86 Jason A. Donenfeld 2018-09-14 16 select ZINC_CHACHA20
1b5dbb86 Jason A. Donenfeld 2018-09-14 17 select ZINC_POLY1305
1b5dbb86 Jason A. Donenfeld 2018-09-14 18 select CRYPTO_BLKCIPHER
1b5dbb86 Jason A. Donenfeld 2018-09-14 19
a740374c Jason A. Donenfeld 2018-09-14 20 config ZINC_BLAKE2S
a740374c Jason A. Donenfeld 2018-09-14 21 bool
a740374c Jason A. Donenfeld 2018-09-14 22 select ZINC
a740374c Jason A. Donenfeld 2018-09-14 23
cec5aa7c Jason A. Donenfeld 2018-09-14 24 config ZINC_CURVE25519
cec5aa7c Jason A. Donenfeld 2018-09-14 25 bool
cec5aa7c Jason A. Donenfeld 2018-09-14 26 select ZINC
cec5aa7c Jason A. Donenfeld 2018-09-14 27 select CONFIG_CRYPTO
cec5aa7c Jason A. Donenfeld 2018-09-14 28
32bbe22e Jason A. Donenfeld 2018-09-14 29 config ZINC_DEBUG
32bbe22e Jason A. Donenfeld 2018-09-14 30 bool "Zinc cryptography library debugging and self-tests"
32bbe22e Jason A. Donenfeld 2018-09-14 31 depends on ZINC
32bbe22e Jason A. Donenfeld 2018-09-14 32 help
32bbe22e Jason A. Donenfeld 2018-09-14 33 This builds a series of self-tests for the Zinc crypto library, which
32bbe22e Jason A. Donenfeld 2018-09-14 34 help diagnose any cryptographic algorithm implementation issues that
32bbe22e Jason A. Donenfeld 2018-09-14 35 might be at the root cause of potential bugs. It also adds various
32bbe22e Jason A. Donenfeld 2018-09-14 36 debugging traps.
32bbe22e Jason A. Donenfeld 2018-09-14 37
32bbe22e Jason A. Donenfeld 2018-09-14 38 Unless you're developing and testing cryptographic routines, or are
32bbe22e Jason A. Donenfeld 2018-09-14 39 especially paranoid about correctness on your hardware, you may say
32bbe22e Jason A. Donenfeld 2018-09-14 40 N here.
32bbe22e Jason A. Donenfeld 2018-09-14 41
32bbe22e Jason A. Donenfeld 2018-09-14 @42 config ZINC_ARCH_ARM
32bbe22e Jason A. Donenfeld 2018-09-14 43 def_bool y
32bbe22e Jason A. Donenfeld 2018-09-14 44 depends on ARM
32bbe22e Jason A. Donenfeld 2018-09-14 45 depends on ZINC
32bbe22e Jason A. Donenfeld 2018-09-14 46 imply VFP
32bbe22e Jason A. Donenfeld 2018-09-14 47 imply VFPv3 if CPU_V7
32bbe22e Jason A. Donenfeld 2018-09-14 48 imply NEON if CPU_V7
32bbe22e Jason A. Donenfeld 2018-09-14 49 imply KERNEL_MODE_NEON if CPU_V7
32bbe22e Jason A. Donenfeld 2018-09-14 50

:::::: The code at line 1 was first introduced by commit
:::::: 32bbe22ec6fddc21b70274aab7d2830d69d51578 zinc: introduce minimal cryptography library

:::::: TO: Jason A. Donenfeld <Jason@xxxxxxxxx>
:::::: CC: 0day robot <lkp@xxxxxxxxx>

---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all Intel Corporation

Next message: Kees Cook: "[PATCH 17/18] LSM: Provide init debugging"
Previous message: Kees Cook: "[PATCH 02/18] LSM: Rename .security_initcall section to .lsm_info"
In reply to: Jason A. Donenfeld: "Re: [PATCH net-next v4 17/20] crypto: port Poly1305 to Zinc"
Next in thread: Jason A. Donenfeld: "[PATCH net-next v4 19/20] security/keys: rewrite big_key crypto to use Zinc"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]