Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

rockchip64/uboot: increase rng-seed size to make it sufficient for modern linux #7365

Merged
merged 1 commit into from
Oct 13, 2024

Conversation

alex3d
Copy link
Contributor

@alex3d alex3d commented Oct 12, 2024

Description

Modern linux requires 32 byte seed to initialise random pool, but u-boot currently provides only 8 bytes. Increase rng-seed size to make linux happy and initialise instantly.
This significantly speeds-up encrypted overlayroot mount (by 5 sec or so), and I expect cryptroot would benefit either.

Without patch:

# dmesg | grep crng
[   12.089286] random: crng init done

With patch:

# dmesg | grep crng
[    0.000000] random: crng init done

How Has This Been Tested?

  • Ruild and run on rk3318-box

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code

@github-actions github-actions bot added size/small PR with less then 50 lines Hardware Hardware related like kernel, U-Boot, ... Patches Patches related to kernel, U-Boot, ... labels Oct 12, 2024
@alex3d alex3d changed the title rockchip64/uboot: increase rng-seed size to make it sufficient for mo… rockchip64/uboot: increase rng-seed size to make it sufficient for modern linux Oct 12, 2024
@igorpecovnik igorpecovnik added Ready to merge Reviewed, tested and ready for merge 11 Milestone: Fourth quarter release labels Oct 13, 2024
@igorpecovnik igorpecovnik merged commit 864d7f0 into armbian:main Oct 13, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
11 Milestone: Fourth quarter release Hardware Hardware related like kernel, U-Boot, ... Patches Patches related to kernel, U-Boot, ... Ready to merge Reviewed, tested and ready for merge size/small PR with less then 50 lines
Development

Successfully merging this pull request may close these issues.

2 participants