Skip to content

kernel: enable CONFIG_CRYPTO_DH in aarch64#11409

Merged
tobiasb-ms merged 2 commits into3.0-devfrom
tobiasb-ms/aarch64-crypto-dh-kernel-module
Jan 8, 2025
Merged

kernel: enable CONFIG_CRYPTO_DH in aarch64#11409
tobiasb-ms merged 2 commits into3.0-devfrom
tobiasb-ms/aarch64-crypto-dh-kernel-module

Conversation

@tobiasb-ms
Copy link
Copy Markdown
Contributor

@tobiasb-ms tobiasb-ms commented Dec 11, 2024

Merge Checklist

All boxes should be checked before merging the PR (just tick any boxes which don't apply to this PR)

  • The toolchain has been rebuilt successfully (or no changes were made to it)
  • The toolchain/worker package manifests are up-to-date
  • Any updated packages successfully build (or no packages were changed)
  • Packages depending on static components modified in this PR (Golang, *-static subpackages, etc.) have had their Release tag incremented.
  • Package tests (%check section) have been verified with RUN_CHECK=y for existing SPEC files, or added to new SPEC files
  • All package sources are available
  • cgmanifest files are up-to-date and sorted (./cgmanifest.json, ./toolkit/scripts/toolchain/cgmanifest.json, .github/workflows/cgmanifest.json)
  • LICENSE-MAP files are up-to-date (./LICENSES-AND-NOTICES/SPECS/data/licenses.json, ./LICENSES-AND-NOTICES/SPECS/LICENSES-MAP.md, ./LICENSES-AND-NOTICES/SPECS/LICENSE-EXCEPTIONS.PHOTON)
  • All source files have up-to-date hashes in the *.signatures.json files
  • sudo make go-tidy-all and sudo make go-test-coverage pass
  • Documentation has been updated to match any changes to the build system
  • Ready to merge

Summary

Add the dh kernel module (CONFIG_CRYPTO_DH) to the aarch64 kernel configuration. This is required for fips images, and is already present in the amd64 config, and has never been present in arm64.

KConfig for CONFIG_CRYPTO_DH is present in many distros.

See "validation" section of PR notes for validations I've done.

Change Log
  • Enabled CONFIG_CRYPTO_DH in module mode in the aarch64 kernel configuration in both kernel and kernel-64k.
  • Bumped version of various kernel-related packages.
Does this affect the toolchain?

NO

Associated Issues
Test Methodology

Various pieces of information (only from arm64 vm):
BEFORE upgrading kernel:

$ lsmod | grep dh
<no output>

$ modinfo dh
modinfo: ERROR: Module dh not found.

$ systemd-analyze
Startup finished in 994ms (kernel) + 1.508s (initrd) + 35.822s (userspace) = 38.324s 
graphical.target reached after 35.264s in userspace.

$ sudo ls -l /boot/ | grep vmlinuz
-rw------- 1 root root 48661352 Dec  5 20:57 vmlinuz-6.6.57.1-5.azl3
$ sudo ls -lh /boot/ | grep vmlinuz
-rw------- 1 root root  47M Dec  5 20:57 vmlinuz-6.6.57.1-5.azl3

AFTER upgrading kernel and rebooting:

$ lsmod | grep dh
<no output>

$ modinfo dh
filename:       /lib/modules/6.6.57.1-8.azl3/kernel/crypto/dh_generic.ko.xz
description:    DH generic algorithm
license:        GPL
alias:          crypto-dh
alias:          dh
depends:
intree:         Y
name:           dh_generic
vermagic:       6.6.57.1-8.azl3 SMP mod_unload modversions aarch64
sig_id:         PKCS#7
signer:
sig_key:
sig_hashalgo:   unknown
signature:

$ systemd-analyze
Startup finished in 942ms (kernel) + 1.524s (initrd) + 7.807s (userspace) = 10.274s
graphical.target reached after 7.397s in userspace.

$ sudo ls -l /boot/ | grep vmlinuz
-rw------- 1 root root 48661352 Dec  5 20:57 vmlinuz-6.6.57.1-5.azl3
-rw------- 1 root root 48650752 Jan  8 16:05 vmlinuz-6.6.57.1-8.azl3
$ sudo ls -lh /boot/ | grep vmlinuz
-rw------- 1 root root  47M Dec  5 20:57 vmlinuz-6.6.57.1-5.azl3
-rw------- 1 root root  47M Jan  8 16:05 vmlinuz-6.6.57.1-8.azl3

@tobiasb-ms tobiasb-ms marked this pull request as ready for review December 13, 2024 19:57
@tobiasb-ms tobiasb-ms requested a review from a team as a code owner December 13, 2024 19:57
@tobiasb-ms tobiasb-ms force-pushed the tobiasb-ms/aarch64-crypto-dh-kernel-module branch from dfe5415 to cfed416 Compare December 16, 2024 15:44
@tobiasb-ms tobiasb-ms force-pushed the tobiasb-ms/aarch64-crypto-dh-kernel-module branch from cfed416 to c958e4d Compare January 7, 2025 18:06
@microsoft-github-policy-service microsoft-github-policy-service Bot added Packaging 3.0-dev PRs Destined for AzureLinux 3.0 labels Jan 7, 2025
@tobiasb-ms tobiasb-ms force-pushed the tobiasb-ms/aarch64-crypto-dh-kernel-module branch 3 times, most recently from a66b411 to e70d342 Compare January 8, 2025 15:16
@tobiasb-ms tobiasb-ms force-pushed the tobiasb-ms/aarch64-crypto-dh-kernel-module branch from e70d342 to d83ee99 Compare January 8, 2025 15:17
@tobiasb-ms tobiasb-ms merged commit db2fd14 into 3.0-dev Jan 8, 2025
@tobiasb-ms tobiasb-ms deleted the tobiasb-ms/aarch64-crypto-dh-kernel-module branch January 8, 2025 22:50
jslobodzian pushed a commit that referenced this pull request Jan 22, 2025
Add the dh kernel module (CONFIG_CRYPTO_DH) to the aarch64 kernel configuration. This is required for fips images, and is already present in the amd64 config, and has never been present in arm64.
SumitJenaHCL pushed a commit to SumitJenaHCL/azurelinux that referenced this pull request Jan 28, 2025
Add the dh kernel module (CONFIG_CRYPTO_DH) to the aarch64 kernel configuration. This is required for fips images, and is already present in the amd64 config, and has never been present in arm64.
SumitJenaHCL pushed a commit to SumitJenaHCL/azurelinux that referenced this pull request Feb 24, 2025
Add the dh kernel module (CONFIG_CRYPTO_DH) to the aarch64 kernel configuration. This is required for fips images, and is already present in the amd64 config, and has never been present in arm64.
KavyaSree2610 pushed a commit to KavyaSree2610/azurelinux that referenced this pull request Mar 24, 2025
Add the dh kernel module (CONFIG_CRYPTO_DH) to the aarch64 kernel configuration. This is required for fips images, and is already present in the amd64 config, and has never been present in arm64.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3.0-dev PRs Destined for AzureLinux 3.0 Packaging

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants