From a5e180552f99b1ea42f0edc671cc018df6386fe5 Mon Sep 17 00:00:00 2001 From: Xichen96 Date: Thu, 31 Aug 2023 23:47:46 +0800 Subject: [PATCH] add processor.max_cstate=0 to intel cpu cmdline (#16339) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit 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 --- installer/default_platform.conf | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/installer/default_platform.conf b/installer/default_platform.conf index 559e7ab3cf89..47ae1e67069d 100755 --- a/installer/default_platform.conf +++ b/installer/default_platform.conf @@ -481,7 +481,7 @@ bootloader_menu_config() echo "Switch CPU vendor is: $CPUVENDOR" if echo "$CPUVENDOR" | grep -i 'Intel' >/dev/null 2>&1; then echo "Switch CPU cstates are: disabled" - CSTATES="intel_idle.max_cstate=0" + CSTATES="processor.max_cstate=1 intel_idle.max_cstate=0" else CSTATES="" fi