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

Enabling GZIP compression support in initrd for x220 #596

Closed
wants to merge 1 commit into from

Conversation

spacekookie
Copy link

Without this option set by default, images built for
the x220 will kernel panic shortly after boot, failing
to unpack their initrd.

@spacekookie spacekookie changed the title Enabling GZIP compression support in initrd Enabling GZIP compression support in initrd for x220 Jul 22, 2019
@BlackMaria
Copy link

BlackMaria commented Jul 22, 2019

Sorry if I am adding noise, I lost track of this thread. @SebastianMcMillan was looking at an issue where after a power up, the x220 would take 60 seconds before displaying anything. @spacekookie Have you waited 60+ seconds, or do you actually see a segfault or like?

Sebastian perhaps this is a/the regression merged earlier causing the 60 second delay?
Danke

PS your issue refers to the x220 but the diff is for the x230 ( perhaps I am misunderstanding? )

@spacekookie
Copy link
Author

spacekookie commented Jul 22, 2019 via email

@paulmenzel
Copy link
Contributor

I prefer imperative mood in the commit message:

Enable GZIP compression support in initrd for x220

Do you know, how that regression was introduced? The Linux kernel has not been updated for a long time, so the default for that Kconfig symbol should not have been changed.

Also, I thought, that XZ compression is used by default for Heads for smaller images.

@snmcmillan
Copy link
Contributor

snmcmillan commented Jul 22, 2019

I'll have to test these changes as well, but this delay is confirmed to be either a TPM issue or a ME issue, as the delay takes place long before we get to loading the kernel.

Another thing is that X220 isn't panicking on my end without the gzip.

@BlackMaria
Copy link

@SebastianMcMillan Sorry, I seem to have confounded the issue with what you were working on.
FWIW I dont have the gzip issue on my end either ( but haven't built from master for about a month )

@snmcmillan
Copy link
Contributor

Just rebased my build on my changes with an up-to-date master, do not have this problem.

@spacekookie
Copy link
Author

@SebastianMcMillan That's strange. The commit I was using to build my images was 20d79f5, which resulted in unbootable images.

Without this option set by default, images built for
the x220 will kernel panic shortly after boot, failing
to unpack their initrd.
@spacekookie
Copy link
Author

@paulmenzel I updated the commit description. Also, yea I was under the impression that kernel images should be using XZ compression as well. At least that's the feeling one could get from the configs.

The build command I used wasn't customised in anyway, just make BOARD=x220, so not sure what might be going on there 🤷‍♀️

@tlaurion
Copy link
Collaborator

tlaurion commented Jan 2, 2020

@spacekookie @BlackMaria @SebastianMcMillan
Please
make BOARD=x220 real.clean && rm -rf crossgcc build/*/

And report issues in good tickets, including this one. Heads doesn't redownload, doesn't reapply patches, doesn't recompile everything at each builds, so your states are obviously different and you all have nothing to compare against.

A good advice until we have working CIs would be to give a commit ID and the sha256sum hash given for a specific build tested, so you know you are talking about the same built rom. Otherwise, you are all testing different roms.

@spacekookie I confirm xz is supposed to be used. You should't have a kernel panic because of gzip not being available.

Please close if issue not relevant anymore.

@tlaurion
Copy link
Collaborator

tlaurion commented Feb 3, 2020

Is it still relevant @spacekookie?

@tlaurion
Copy link
Collaborator

Ping @spacekookie.
If the issue is still relevant after building with clean environment, please reopen.

cd heads
git fetch
git pull
make BOARD=x220 real.clean && rm -rf crossgcc build/*/

@tlaurion tlaurion closed this Feb 16, 2020
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.

5 participants