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.

Sign up for GitHub

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

Jump to bottom

[action] [PR:16339] [installer] add processor.max_cstate=1 to intel kernel cmdline #16425

Merged
merged 1 commit into from
Sep 3, 2023
Merged

[action] [PR:16339] [installer] add processor.max_cstate=1 to intel kernel cmdline #16425

merged 1 commit into from
Sep 3, 2023

Commits on Sep 3, 2023

  1. add processor.max_cstate=0 to intel cpu cmdline (sonic-net#16339) 

    Why I did it
This is a fix for PR [kernel] Change grub cmdline to set c-states to 0 for "Intel" CPUs by shlomibitton · Pull Request sonic-net#6051 · sonic-net/sonic-buildimage (github.com)

The original PR will disable intel idle driver but it cannot limit the max c-state to 1 due to system will fall back to acpi idle driver.

Currently intel_idle.max_cstate=0 is already present, which will disable intel idle driver. With the added option, common idle driver will be disabled as well, so there will not be idle management. This is to prevent a bug that can be triggered by idle instruction on intel platform.

How I did it
Add the option to installer file beside intel_idle.max_cstate=0
    @Xichen96 @mssonicbld
    Xichen96 authored and mssonicbld committed Sep 3, 2023
    Configuration menu
    Copy the full SHA
    986bfe3 View commit details
    Browse the repository at this point in the history