Skip to content

Conversation

ssssam
Copy link
Contributor

@ssssam ssssam commented Oct 20, 2025

Main thing in this PR is disabling kernel debug symbols, which brings the size of payg-image.efi down from 392MB to 98.2MB. This is below the 100MB limit imposed by the sb-signer (just).

About re-enabling kernel debug symbols, see tracking issues #176 and #178.

Other changes in this PR:

  • Remove some dead code
  • Add howdo doc about inspecting the UKI
  • Detect errors properly during the UKI build
  • Speed up linux.bst build process
  • Fix a build failure in eos-payg-nonfree

@ssssam ssssam mentioned this pull request Oct 20, 2025
Copy link
Contributor

@starnight starnight left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! LGTM!

Only the typo :)


The UKI is only used for PAYG devices.

Note that building the UKI is only possible within Endless. But uou can find
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

uou -> you?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good spot!

- freedesktop-sdk.bst:components/dracut.bst
- freedesktop-sdk.bst:components/ostree.bst
- freedesktop-sdk.bst:components/python3-pefile.bst
- freedesktop-sdk.bst:components/zstd.bst
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Was this addition intentional?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, as the initramfs is zstd compressed, but i didnt put that in the commit message. will update

@ssssam ssssam force-pushed the sam/payg-uki-fixes branch from 1d3fcd2 to 8d54ab1 Compare October 21, 2025 08:44
@ssssam
Copy link
Contributor Author

ssssam commented Oct 21, 2025

Updated with some small changes, I'll merge once the CI is complete.

Dracut errors were not being correctly detected.

Add zstd, which is needed for compression.
This allows signing to happen on many cores in parallel.
This matches the eos6 kernel build which does not compress modules.
They will still be compressed within the initramfs (which is compressed
with zstd as a single blob) and within the ostree (where each object
is gzipped individually).
Kernel modules are otherwise huge, which causes problems in particular
for the PAYG UKI.

A better solution would be to build and then split out the debug info.
There are a couple of ways to do this, see:
<#176>

Fixes #178
by bringing the UKI down to a mere 98.6MB.
@ssssam ssssam force-pushed the sam/payg-uki-fixes branch from 8d54ab1 to 81f07e9 Compare October 21, 2025 08:46
@ssssam
Copy link
Contributor Author

ssssam commented Oct 21, 2025

Updated again to resolve conflicts after landing #180

@ssssam ssssam merged commit 08baeaf into main Oct 21, 2025
1 check passed
@ssssam ssssam deleted the sam/payg-uki-fixes branch October 21, 2025 09:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants