Skip to content

crypto: reduce overhead when self-tests disabled

Eric Biggers linux-crypto 2 versions – In review

Get series with b4
b4 am [email protected]

Versions newest first · linked by author and title

  1. v3 Latest6 patches 3 repliesIn review
  2. v26 patches 3 repliesSuperseded

Changes in v3 from the cover letter

- Made sure CRYPTO_MSG_ALG_LOADED still always gets sent.
- Fixed a race condition with larval->test_started.
- Used IS_ENABLED() in a couple places to avoid #ifdefs.

Patches in v3 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/6crypto: optimize algorithm registration when self-tests disabledcrypto/algapi.c crypto/api.c+86 -710 replies
2/6crypto: optimize registration of internal algorithmscrypto/algapi.c crypto/algboss.c+3 -132 replies
3/6crypto: compile out crypto_boot_test_finished when tests disabledcrypto/algapi.c crypto/api.c crypto/internal.h+29 -60 replies
4/6crypto: skip kdf_sp800108 self-test when tests disabledcrypto/kdf_sp800108.c+6 -20 replies
5/6crypto: silence noisy kdf_sp800108 self-testcrypto/kdf_sp800108.c+1 -10 replies
6/6crypto: compile out test-related algboss code when tests disabledcrypto/algboss.c+4 -50 replies

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v3 yet.

Read the v3 thread

Cover letter [PATCH v3 0/6] crypto: reduce overhead when self-tests disabled

This patchset makes it so that the self-test code doesn't still slow things down when self-tests are disabled via the kconfig.

It also optimizes the registration of "internal" algorithms and silences a noisy log message.

Changed in v3:
- Made sure CRYPTO_MSG_ALG_LOADED still always gets sent.
- Fixed a race condition with larval->test_started.
- Used IS_ENABLED() in a couple places to avoid #ifdefs.

Eric Biggers (6):
crypto: optimize algorithm registration when self-tests disabled
crypto: optimize registration of internal algorithms
crypto: compile out crypto_boot_test_finished when tests disabled
crypto: skip kdf_sp800108 self-test when tests disabled
crypto: silence noisy kdf_sp800108 self-test
crypto: compile out test-related algboss code when tests disabled

 crypto/algapi.c       | 160 ++++++++++++++++++++++++------------------
 crypto/algboss.c      |  22 ++----
 crypto/api.c          |  11 ++-
 crypto/internal.h     |  20 +++++-
 crypto/kdf_sp800108.c |  10 ++-
 5 files changed, 127 insertions(+), 96 deletions(-)

base-commit: 557ffd5a4726f8b6f0dd1d4b632ae02c1c063233

Signature · 1 line
2.38.1