Skip to content

Commit 07f0a8a

Browse files
Xichen96mssonicbld
authored andcommitted
add processor.max_cstate=0 to intel cpu cmdline (#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 #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 acad2e6 commit 07f0a8a

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
@@ -472,7 +472,7 @@ bootloader_menu_config()
472472
echo "Switch CPU vendor is: $CPUVENDOR"
473473
if echo "$CPUVENDOR" | grep -i 'Intel' >/dev/null 2>&1; then
474474
echo "Switch CPU cstates are: disabled"
475-
CSTATES="intel_idle.max_cstate=0"
475+
CSTATES="processor.max_cstate=1 intel_idle.max_cstate=0"
476476
else
477477
CSTATES=""
478478
fi

0 commit comments

Comments
 (0)