mbox series

[v2,0/2] relax crypto Kconfig dependencies for fsverity/fscrypt

Message ID 20210421075511.45321-1-ardb@kernel.org (mailing list archive)
Headers show
Series relax crypto Kconfig dependencies for fsverity/fscrypt | expand

Message

Ard Biesheuvel April 21, 2021, 7:55 a.m. UTC 
Relax 'select' dependencies to 'imply' for crypto algorithms that are
fulfilled only at runtime, and which may be implemented by other drivers
than the generic ones implemented in C. This permits, e.g., arm64 builds
to omit the generic CRYPTO_SHA256 and CRYPTO_AES drivers, both of which
are superseded by optimized scalar versions at the very least,

Changes since v1:
- use Eric's suggested comment text in patch #1
- add Eric's ack to partch #2

Cc: "Theodore Y. Ts'o" <tytso@mit.edu>
Cc: Jaegeuk Kim <jaegeuk@kernel.org>
Cc: Eric Biggers <ebiggers@kernel.org>

Ard Biesheuvel (2):
  fscrypt: relax Kconfig dependencies for crypto API algorithms
  fsverity: relax build time dependency on CRYPTO_SHA256

 fs/crypto/Kconfig | 30 ++++++++++++++------
 fs/verity/Kconfig |  8 ++++--
 2 files changed, 28 insertions(+), 10 deletions(-)

Comments

Herbert Xu April 22, 2021, 7:48 a.m. UTC | #1 
Ard Biesheuvel <ardb@kernel.org> wrote:
> Relax 'select' dependencies to 'imply' for crypto algorithms that are
> fulfilled only at runtime, and which may be implemented by other drivers
> than the generic ones implemented in C. This permits, e.g., arm64 builds
> to omit the generic CRYPTO_SHA256 and CRYPTO_AES drivers, both of which
> are superseded by optimized scalar versions at the very least,
> 
> Changes since v1:
> - use Eric's suggested comment text in patch #1
> - add Eric's ack to partch #2
> 
> Cc: "Theodore Y. Ts'o" <tytso@mit.edu>
> Cc: Jaegeuk Kim <jaegeuk@kernel.org>
> Cc: Eric Biggers <ebiggers@kernel.org>
> 
> Ard Biesheuvel (2):
>  fscrypt: relax Kconfig dependencies for crypto API algorithms
>  fsverity: relax build time dependency on CRYPTO_SHA256
> 
> fs/crypto/Kconfig | 30 ++++++++++++++------
> fs/verity/Kconfig |  8 ++++--
> 2 files changed, 28 insertions(+), 10 deletions(-)

All applied.  Thanks.