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

i915 - Transparent Hugepage support is recommended for optimal performance on this platform! #9363

Closed
VxDlH opened this issue Sep 23, 2024 · 1 comment · Fixed by siderolabs/pkgs#1036
Assignees

Comments

@VxDlH
Copy link

VxDlH commented Sep 23, 2024

Feature Request

Can you add the following kernel-level option to Talos images?

CONFIG_TRANSPARENT_HUGEPAGE=y

Description

Why this recommendation with the i915 driver ?

The message suggests enabling Transparent Hugepages to take full advantage of Intel GPU performance. This can improve graphics performance in certain cases, such as :

  • Improved graphics rendering: the GPU can access memory more efficiently when large memory pages are used.
  • Reduced memory fragmentation: The system is able to better organize and manage memory with less fragmentation, which improves overall performance, especially on large or memory-intensive workloads.
  • Lower page management costs: Fewer pages to manage means less overhead from address translation and kernel interrupts.
@smira smira self-assigned this Sep 24, 2024
@smira
Copy link
Member

smira commented Sep 24, 2024

Yes, this setting should be good, and not only for i915.

smira added a commit to smira/pkgs that referenced this issue Oct 7, 2024
Fixes siderolabs/talos#9363

This way THP are only enabled if the application explicitly tries to use
them.

Signed-off-by: Andrey Smirnov <andrey.smirnov@siderolabs.com>
(cherry picked from commit 2c5785b)
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 24, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants