Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
IronPort-SDR: 5kUEnbAvK/Mx1yZRyTG6Q2TETDkYNkp+4Jc6PS4ZVc6s/PfyQahzaHIJgcs1WYvgJlcP7QVnne
 w3DGibsAAfXQ==
IronPort-SDR: NKlEa2tHbkW20Gfo3x7x+9Z+3A45Evl4V9/jfkTylSugrhsAx4kodqRLxGM+K9ve0DiT+boOEV
 5FX8N7/4lYmA==
From:   Like Xu <like.xu@linux.intel.com>
To:     Giovanni Gherdovich <ggherdovich@suse.cz>
Cc:     linux-kernel@vger.kernel.org, Like Xu <like.xu@linux.intel.com>
Subject: [PATCH] x86, smpboot: Disable frequency invariance when it's unsupported
Date:   Thu, 16 Apr 2020 10:12:10 +0800
Message-Id: <20200416021210.170736-1-like.xu@linux.intel.com>
MIME-Version: 1.0
Content-Transfer-Encoding: 8bit
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

On the Intel SNR processors such as "Intel Atom(R) C6562", the
turbo_freq for 4C turbo may be zero which causes a divide by zero
exception and blocks the boot process when arch_scale_freq_tick().

When one of the preset base_freq or turbo_freq is meaningless,
we may disable frequency invariance.

Fixes: 1567c3e3467c ("x86, sched: Add support for frequency invariance")
Signed-off-by: Like Xu <like.xu@linux.intel.com>
---
 arch/x86/kernel/smpboot.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/arch/x86/kernel/smpboot.c b/arch/x86/kernel/smpboot.c
index fe3ab9632f3b..741367ce4d14 100644
--- a/arch/x86/kernel/smpboot.c
+++ b/arch/x86/kernel/smpboot.c
@@ -1958,6 +1958,9 @@ static bool core_set_max_freq_ratio(u64 *base_freq, u64 *turbo_freq)
 	*base_freq = (*base_freq >> 8) & 0xFF;      /* max P state */
 	*turbo_freq = (*turbo_freq >> 24) & 0xFF;   /* 4C turbo    */
 
+	if (*turbo_freq == 0 || *base_freq == 0)
+		return false;
+
 	return true;
 }
 
-- 
2.21.1