Skip to content

Commit 060f759

Browse files
Xichen96sonic-otn
authored andcommitted
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
1 parent 84c503c commit 060f759

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

installer/default_platform.conf

+1-1
Original file line numberDiff line numberDiff line change
@@ -481,7 +481,7 @@ bootloader_menu_config()
481481
echo "Switch CPU vendor is: $CPUVENDOR"
482482
if echo "$CPUVENDOR" | grep -i 'Intel' >/dev/null 2>&1; then
483483
echo "Switch CPU cstates are: disabled"
484-
CSTATES="intel_idle.max_cstate=0"
484+
CSTATES="processor.max_cstate=1 intel_idle.max_cstate=0"
485485
else
486486
CSTATES=""
487487
fi

0 commit comments

Comments
 (0)