This repository was archived by the owner on May 30, 2023. It is now read-only.
sys-kernel/coreos-modules: switch to gzip/zstd compression#1136
Merged
Conversation
With the default gzip compression the 60 MB limit for the vmlinuz bundle of kernel+initramfs was reached. The limit comes from the size of the /boot partition which is 128 MB large and the kernel needs to fit twice, in addition to GRUB. Use zstd for the initramfs as it provides a similar speed but better compression. For the kernel we can't switch yet to zstd for arm64 but for amd64 it works.
510ab85 to
cc94163
Compare
dongsupark
approved these changes
Jul 28, 2021
Contributor
dongsupark
left a comment
There was a problem hiding this comment.
Looks good.
Side note: I was wondering if we should do the same thing for arm64.
However, apparently arm64 still needs uncompressed Kernel, according to the discussion. https://bugs.gentoo.org/784599
Contributor
Author
|
Thanks for the correction, I assumed it uses GZIP and was confused why switching to something similarly supported didn't work |
pothos
added a commit
that referenced
this pull request
Aug 9, 2021
sys-kernel/coreos-modules: switch to gzip/zstd compression
pothos
added a commit
that referenced
this pull request
Aug 9, 2021
sys-kernel/coreos-modules: switch to gzip/zstd compression
Contributor
Author
|
Ported to 2905 and 2942 maintenance branches |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
sys-kernel/coreos-modules: switch to gzip/zstd compression
Fixes flatcar/Flatcar#438
How to use/testing done
With zstd for both on amd64 I got a reduction of 13 MB for the vmlinuz file (new size is 49 MB, Alpha 2920.0.0 is 62 MB). Since that didn't work with arm64 I changed arm64 to continue to use no compression (default) for the kernel and zstd for initramfs, this just just a 5 MB reduction (new size is 54 MB, old is 59 MB).
http://jenkins.infra.kinvolk.io:8080/job/os/job/manifest/3138/cldsv/