All of lore.kernel.org
 help / color / mirror / Atom feed
From: Eric Biggers <ebiggers@kernel.org>
To: linux-crypto@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Subject: [PATCH v3 0/7] Clean up the crypto testing options
Date: Mon,  5 May 2025 13:33:38 -0700	[thread overview]
Message-ID: <20250505203345.802740-1-ebiggers@kernel.org> (raw)

This series reworks the crypto testing kconfig options to fix some
longstanding issues:

- Replace the inverted option CONFIG_CRYPTO_MANAGER_DISABLE_TESTS with a
  regular option CONFIG_CRYPTO_SELFTESTS.

- Make CONFIG_CRYPTO_SELFTESTS enable the full set of tests by default,
  removing CONFIG_CRYPTO_MANAGER_EXTRA_TESTS.

- Automatically enable CONFIG_CRYPTO_MANAGER when needed for the tests.

- Rename cryptomgr.noextratests to cryptomgr.noslowtests.

- Remove cryptomgr.panic_on_fail, as panic_on_warn can be used instead.

- Rename CONFIG_CRYPTO_TEST to CONFIG_CRYPTO_BENCHMARK.

Changed in v3:
  - Keep options in defconfigs.
  - Make CRYPTO_SELFTESTS depend on DEBUG_KERNEL.

Changed in v2:
  - Moved selftest lines in lib/crypto/Makefile to appropriate places.
  - Fixed a commit message.

Eric Biggers (7):
  crypto: tcrypt - rename CRYPTO_TEST to CRYPTO_BENCHMARK
  crypto: testmgr - remove panic_on_fail
  crypto: testmgr - replace CRYPTO_MANAGER_DISABLE_TESTS with
    CRYPTO_SELFTESTS
  crypto: testmgr - make it easier to enable the full set of tests
  crypto: testmgr - rename noextratests to noslowtests
  crypto: Kconfig - make CRYPTO_MANAGER a hidden symbol
  crypto: testmgr - enable CRYPTO_MANAGER when CRYPTO_SELFTESTS

 arch/arm/configs/exynos_defconfig           |   2 +-
 arch/arm/configs/milbeaut_m10v_defconfig    |   3 +-
 arch/arm/configs/pxa_defconfig              |   3 +-
 arch/arm/configs/spitz_defconfig            |   2 +-
 arch/arm64/configs/defconfig                |   2 +-
 arch/loongarch/configs/loongson3_defconfig  |   2 +-
 arch/m68k/configs/amiga_defconfig           |   2 +-
 arch/m68k/configs/apollo_defconfig          |   2 +-
 arch/m68k/configs/atari_defconfig           |   2 +-
 arch/m68k/configs/bvme6000_defconfig        |   2 +-
 arch/m68k/configs/hp300_defconfig           |   2 +-
 arch/m68k/configs/mac_defconfig             |   2 +-
 arch/m68k/configs/multi_defconfig           |   2 +-
 arch/m68k/configs/mvme147_defconfig         |   2 +-
 arch/m68k/configs/mvme16x_defconfig         |   2 +-
 arch/m68k/configs/q40_defconfig             |   2 +-
 arch/m68k/configs/sun3_defconfig            |   2 +-
 arch/m68k/configs/sun3x_defconfig           |   2 +-
 arch/mips/configs/decstation_64_defconfig   |   1 -
 arch/mips/configs/decstation_defconfig      |   1 -
 arch/mips/configs/decstation_r4k_defconfig  |   1 -
 arch/mips/configs/gpr_defconfig             |   2 +-
 arch/mips/configs/ip28_defconfig            |   1 -
 arch/mips/configs/lemote2f_defconfig        |   2 +-
 arch/mips/configs/mtx1_defconfig            |   2 +-
 arch/mips/configs/rb532_defconfig           |   2 +-
 arch/parisc/configs/generic-32bit_defconfig |   2 +-
 arch/parisc/configs/generic-64bit_defconfig |   1 -
 arch/powerpc/configs/g5_defconfig           |   2 +-
 arch/powerpc/configs/powernv_defconfig      |   2 +-
 arch/powerpc/configs/ppc64_defconfig        |   2 +-
 arch/powerpc/configs/ppc64e_defconfig       |   2 +-
 arch/powerpc/configs/ppc6xx_defconfig       |   2 +-
 arch/s390/configs/debug_defconfig           |   4 +-
 arch/s390/configs/defconfig                 |   4 +-
 arch/sh/configs/migor_defconfig             |   1 -
 arch/sparc/configs/sparc64_defconfig        |   2 +-
 crypto/Kconfig                              |  45 ++++----
 crypto/Makefile                             |   2 +-
 crypto/algapi.c                             |   5 +-
 crypto/algboss.c                            |   2 +-
 crypto/api.c                                |   3 +-
 crypto/hkdf.c                               |   2 +-
 crypto/internal.h                           |   5 +-
 crypto/kdf_sp800108.c                       |   2 +-
 crypto/tcrypt.c                             |   8 +-
 crypto/tcrypt.h                             |   4 +-
 crypto/testmgr.c                            | 115 +++++---------------
 include/crypto/internal/simd.h              |   6 +-
 lib/crypto/Makefile                         |  11 +-
 lib/crypto/aescfb.c                         |   2 +-
 lib/crypto/aesgcm.c                         |   2 +-
 lib/crypto/blake2s.c                        |   2 +-
 lib/crypto/chacha20poly1305.c               |   2 +-
 lib/crypto/curve25519.c                     |   2 +-
 55 files changed, 111 insertions(+), 183 deletions(-)


base-commit: 64745a9ca890ed60d78162ec511e1983e1946d73
-- 
2.49.0


             reply	other threads:[~2025-05-05 20:37 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-05-05 20:33 Eric Biggers [this message]
2025-05-05 20:33 ` [PATCH v3 1/7] crypto: tcrypt - rename CRYPTO_TEST to CRYPTO_BENCHMARK Eric Biggers
2025-05-08 12:59   ` Geert Uytterhoeven
2025-05-05 20:33 ` [PATCH v3 2/7] crypto: testmgr - remove panic_on_fail Eric Biggers
2025-05-05 20:33 ` [PATCH v3 3/7] crypto: testmgr - replace CRYPTO_MANAGER_DISABLE_TESTS with CRYPTO_SELFTESTS Eric Biggers
2025-06-11 11:41   ` Diederik de Haas
2025-06-11 11:41     ` Diederik de Haas
2025-06-11 16:34     ` Eric Biggers
2025-06-11 16:34       ` Eric Biggers
2025-06-13  7:57       ` Diederik de Haas
2025-06-13  7:57         ` Diederik de Haas
2025-05-05 20:33 ` [PATCH v3 4/7] crypto: testmgr - make it easier to enable the full set of tests Eric Biggers
2025-05-05 20:33 ` [PATCH v3 5/7] crypto: testmgr - rename noextratests to noslowtests Eric Biggers
2025-05-05 20:33 ` [PATCH v3 6/7] crypto: Kconfig - make CRYPTO_MANAGER a hidden symbol Eric Biggers
2025-05-05 20:33 ` [PATCH v3 7/7] crypto: testmgr - enable CRYPTO_MANAGER when CRYPTO_SELFTESTS Eric Biggers
2025-05-12  5:46 ` [PATCH v3 0/7] Clean up the crypto testing options Herbert Xu

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20250505203345.802740-1-ebiggers@kernel.org \
    --to=ebiggers@kernel.org \
    --cc=linux-crypto@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.