Skip to content

Bump minimal supported kernel version from 4.19 to 5.4#440

Merged
florianl merged 2 commits intomainfrom
drop-4.19
Apr 17, 2025
Merged

Bump minimal supported kernel version from 4.19 to 5.4#440
florianl merged 2 commits intomainfrom
drop-4.19

Conversation

@florianl
Copy link
Copy Markdown
Member

While the Linux kernel LTS support for 4.19 was dropped for some time the oldest supported Linux kernel LTS version is currently 5.4. Update references to use the currently oldest supported Linux kernel LTS version.

Proposed change based on the discussion in https://cloud-native.slack.com/archives/C03J794L0BV/p1744317888826789 - FYI @gnurizen @umanwizard

While the Linux kernel LTS support for 4.19 was dropped for some time the oldest supported Linux kernel LTS version is currently 5.4.
Update references to use the currently oldest supported Linux kernel LTS version.

Signed-off-by: Florian Lehner <florian.lehner@elastic.co>
@florianl florianl requested review from a team as code owners April 16, 2025 12:39
@gnurizen
Copy link
Copy Markdown
Contributor

Awesome! My understanding is 5.4 has bounded loops and we can start removing loop unrolling which should make programs smaller and faster to load.

@florianl
Copy link
Copy Markdown
Member Author

Awesome! My understanding is 5.4 has bounded loops and we can start removing loop unrolling which should make programs smaller and faster to load.

Right - some things in the eBPF programs can change and simplified. I didn't do it with this PR, as I wanted first to get consensus on this bump of the minimal supported kernel version.

@umanwizard
Copy link
Copy Markdown
Contributor

This is great from my perspective. It will make it much easier for Parca to collaborate with the rest of the community by upstreaming our work.

Signed-off-by: Florian Lehner <florian.lehner@elastic.co>
@rockdaboot
Copy link
Copy Markdown
Contributor

From what I read, with 5.2+ the instruction limit has been raised from 4096 to 1M.

Copy link
Copy Markdown
Contributor

@korniltsev korniltsev left a comment

Choose a reason for hiding this comment

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

🥳

Copy link
Copy Markdown
Contributor

@fabled fabled left a comment

Choose a reason for hiding this comment

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

🎉

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.

7 participants