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

Kernel: T5499: update arm64 kernel configuration to v6.6.15 #495

Merged
merged 1 commit into from
Feb 9, 2024

Conversation

Schamper
Copy link
Contributor

@Schamper Schamper commented Feb 5, 2024

Change Summary

As a follow-up/improvement to #421, I've meticulously compared the default Debian arm64 configuration with the VyOS x86 configuration and cherry-picked all of the changes made by Debian (for a proper working arm64 kernel) and VyOS (for all the specific VyOS tweaks). I'm not a kernel developer by any means so I may have missed one or two, but I do believe I got all the important bits.

I've verified that the resulting kernel (with some other small tweaks to make the rest of the build work) results in a stable VyOS Sagitta that has been running for a few weeks now in a VMware VM on Apple Sillicon. I specifically built a Sagitta ISO because that's what I intend to run long-term. I expect the current branch to also run fine.

If the team is interested, the changes required to get a happy Sagitta on arm64 mostly boils down to some package compilations for arm64 to be added to the sagitta package mirror, and a backport of vyos/vyos-1x@37bd574.

The kernel builds in a few minutes on an M3 Max, I don't currently have access to a x86 instance that's similarly specced as the build server to compare build times under QEMU. I unfortunately also don't have access to other ARM hardware to test.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes)
  • Migration from an old Vyatta component to vyos-1x, please link to related PR inside obsoleted component
  • Other (please describe):

Related Task(s)

Component(s) name

Kernel

Proposed changes

How to test

Checklist:

  • I have read the CONTRIBUTING document
  • I have linked this PR to one or more Phabricator Task(s)
  • My commit headlines contain a valid Task id
  • My change requires a change to the documentation
  • I have updated the documentation accordingly

@vyosbot vyosbot requested review from a team, dmbaturin, sarthurdev, zdc, jestabro, sever-sever and c-po and removed request for a team February 5, 2024 23:10
@c-po
Copy link
Member

c-po commented Feb 9, 2024

Can we close #421?

@c-po c-po merged commit bb8e7d7 into vyos:current Feb 9, 2024
4 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

2 participants