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

5.10.5 kernel build failure under Debian bookworm #1392

Closed
11 tasks
srgrint opened this issue May 1, 2023 · 4 comments
Closed
11 tasks

5.10.5 kernel build failure under Debian bookworm #1392

srgrint opened this issue May 1, 2023 · 4 comments

Comments

@srgrint
Copy link
Contributor

srgrint commented May 1, 2023

Context of the Build

1. What board are you trying to build?
T440p
2. What repository:branch are you using to build from?

  • [ X] Heads:Master
  • Other (please specify)

3. What version of coreboot are you trying to build

  • 4.13
  • 4.14
  • 4.15
  • [X ] 4.17
  • Other (please specify)

4. In building the rom where did you get the blobs?

  • No blobs required
  • Provided by the company that installed Heads on the device
  • Extracted from a backup rom taken from this device
  • Extracted from another backup rom taken from another device (please identify the board model)
  • [ X] Extracted from the online bios using the automated tools provided in Heads
  • I don't know

5. If using the automated tools to get the blobs did you run the relevant scripts in the blobs directory

  • [X ] Yes
  • No

6. What operating system are you using
Debian bookworm/12

Have tried building the t440p-maximized rom using my test install of Debian 12 (bookworm) which ships with GCC 12

Once I have built the toolchain without gnat support, the build of the 5.10.5 kernel fails due to finding a use after free (due to the new compiler being more aware than what shipped with Debian 11). The bug has already been fixed upstream last year - if I apply the following patch, heads builds as expected:

torvalds/linux@52a9dab

I don't know whether you want to cherry pick this into heads?

@tlaurion
Copy link
Collaborator

tlaurion commented May 1, 2023

@srgrint linux 5.10.178 (LTS) has the fix backported.

@JonathonHall-Purism you might want to backport referred patch, where my helper to prompt devel (who will bump kernel version) works really sweetly under debugging branch.

Note that bookworm on builder host is not officially supported as of now.

I'm personally tired of supporting multiple hosts buildstacks, where goal would be to have Nix layer on top of any linux distrib to fix this as well as building under docker image, which is more interesting on a maintainership perspective.

As of now, it is recommended to build on top of debian-10.

@srgrint
Copy link
Contributor Author

srgrint commented May 1, 2023

I'm personally tired of supporting multiple hosts buildstacks, where goal would be to have Nix layer on top of any linux distrib to fix this as well as building under docker image, which is more interesting on a maintainership perspective.

Completely agree - Nix layer is definitely the way forward.

The use-after-free kernel bug is real though - just Debian 10 buildstack isn't astute enough to catch it

@tlaurion
Copy link
Collaborator

tlaurion commented May 1, 2023

Once I have built the toolchain without gnat support, the build of the 5.10.5 kernel fails due to finding a use after free (due to the new compiler being more aware than what shipped with Debian 11). The bug has already been fixed upstream last year - if I apply the following patch, heads builds as expected:

torvalds/linux@52a9dab

I don't know whether you want to cherry pick this into heads?

@srgrint : Right. Bug is real and currently affects Purism and t440p as of now. Will refer PR to this comment. Thanks.

@srgrint
Copy link
Contributor Author

srgrint commented May 2, 2023

Thanks - not sure whether you wished to also fix boards currently using 4.14.62 which is also affected.

Clearly we want to be bumping all boards to 5.10 in the medium term. Have opened a PR (#1394) in case you want to fix 4.14 branch in the interim.

@tlaurion tlaurion closed this as completed May 3, 2023
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

No branches or pull requests

2 participants