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

initrd: Make compression method an option #195

Merged
merged 3 commits into from Aug 23, 2020

Conversation

samueldr
Copy link
Member

This is a semi-internal option. The end-user may want to change it. Though in reality this is aimed toward device integrators that integrate a device with a small boot partition.

In turn, motorola-addison's initrd is now compressed with xz, which works around its biggest issue: it wouldn't fit in the 16MiB boot partition.

@samueldr samueldr added the 4. type: enhancement New feature or request label Aug 20, 2020
This is a semi-internal option. The end-user may want to change it.
Though in reality this is aimed toward device integrators that integrate
a device with a small boot partition.
(And normalize kernel config)
This makes it go from ~7MiB to ~5MiB, which ends up fitting in the boot
partition.
@samueldr samueldr merged commit b09db9c into NixOS:master Aug 23, 2020
@samueldr samueldr deleted the feature/initrd/compression branch August 23, 2020 01:34
@xaverdh
Copy link

xaverdh commented Aug 25, 2020

you may be interested in NixOS/nixpkgs#96005

@samueldr
Copy link
Member Author

samueldr commented Aug 25, 2020

Thanks for thinking of Mobile NixOS!

Currently the stage-1 for Mobile NixOS doesn't use boot.initrd options, as it is a completely disparate implementation. Though it sure could help in the future when factoring out differences.

Oh, looking at it again, there is a change that will break. Though it looks like there's not much breakage.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4. type: enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants