Skip to content

feat: add hugepages support#952

Draft
piyush-jena wants to merge 2 commits into
bottlerocket-os:developfrom
piyush-jena:hugepages-final
Draft

feat: add hugepages support#952
piyush-jena wants to merge 2 commits into
bottlerocket-os:developfrom
piyush-jena:hugepages-final

Conversation

@piyush-jena

@piyush-jena piyush-jena commented Jun 19, 2026

Copy link
Copy Markdown
Contributor

Description of changes:

  • Add support for hugepages and transparent hugepages

Testing done:

  1. Kits and AMI build ✅
  2. cat /proc/meminfo shows correct values ✅
  3. kubectl describe node advertises correct values ✅
  4. Pods are able to use it and we can benchmark with pointer chasing experiments
  5. EFA Test
  6. Test both arches - x86 and aarch64 support diffferent set of sizes

Terms of contribution:

By submitting this pull request, I agree that this contribution is dual-licensed under the terms of both the Apache License, version 2.0, and the MIT license.

@piyush-jena piyush-jena marked this pull request as draft June 19, 2026 17:21
@piyush-jena piyush-jena force-pushed the hugepages-final branch 9 times, most recently from 2c7149a to b784d20 Compare June 24, 2026 20:59
Signed-off-by: Piyush Jena <jepiyush@amazon.com>
Signed-off-by: Piyush Jena <jepiyush@amazon.com>
Comment on lines +134 to +140
ensure!(
dir.exists(),
error::UnsupportedHugepageSizeSnafu {
size_kib: page_size,
}
);

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This error could also be that the requested numa node doesn't exist. Can you distinguish between the two, or add the numa node info to this snafu report?

@ginglis13 ginglis13 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Please also update the corndog README to reflect these changes

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.

2 participants