Commit 8cad22dc authored by Kuppuswamy Sathyanarayanan's avatar Kuppuswamy Sathyanarayanan Committed by Aichun Shi
Browse files

platform/x86/intel/ifs: Initialize union ifs_status to zero 

mainline inclusion
from mainline-v6.11-rc3
commit 3114f77e9453daa292ec0906f313a715c69b5943
category: feature
bugzilla: https://gitee.com/openeuler/intel-kernel/issues/IB4HZ4
CVE: N/A
Reference: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=3114f77e9453daa292ec0906f313a715c69b5943



-------------------------------------

If the IFS scan test exits prematurely due to a timeout before
completing a single run, the union ifs_status remains uninitialized,
leading to incorrect test status reporting. To prevent this, always
initialize the union ifs_status to zero.

Intel-SIG: commit 3114f77e9453 platform/x86/intel/ifs: Initialize union ifs_status to zero
Backport to fix IFS(In Field Scan) SAF(Scan At Field)

Fixes: 2b40e654 ("platform/x86/intel/ifs: Add scan test support")
Suggested-by: default avatarIlpo Järvinen <ilpo.jarvinen@linux.intel.com>
Reviewed-by: default avatarJithu Joseph <jithu.joseph@intel.com>
Reviewed-by: default avatarAshok Raj <ashok.raj@intel.com>
Signed-off-by: default avatarKuppuswamy Sathyanarayanan <sathyanarayanan.kuppuswamy@linux.intel.com>
Link: https://lore.kernel.org/r/20240730155930.1754744-1-sathyanarayanan.kuppuswamy@linux.intel.com


Reviewed-by: default avatarIlpo Järvinen <ilpo.jarvinen@linux.intel.com>
Signed-off-by: default avatarIlpo Järvinen <ilpo.jarvinen@linux.intel.com>
[ Aichun Shi: amend commit log ]
Signed-off-by: default avatarAichun Shi <aichun.shi@intel.com>
parent e7aff647

drivers/platform/x86/intel/ifs/runtest.c

+1 −1
Original line number Diff line number Diff line
@@ -167,8 +167,8 @@ static int doscan(void *data) 
 */

static void ifs_test_core(int cpu, struct device *dev)

{

	union ifs_status status = {};

	union ifs_scan activate;

	union ifs_status status;

	unsigned long timeout;

	struct ifs_data *ifsd;

	u64 msrvals[2];