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

BugLink: https://bugs.launchpad.net/bugs/2083656

[ Upstream commit 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.

Fixes: 2b40e654b7 ("platform/x86/intel/ifs: Add scan test support")
Suggested-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
Reviewed-by: Jithu Joseph <jithu.joseph@intel.com>
Reviewed-by: Ashok Raj <ashok.raj@intel.com>
Signed-off-by: Kuppuswamy Sathyanarayanan <sathyanarayanan.kuppuswamy@linux.intel.com>
Link: https://lore.kernel.org/r/20240730155930.1754744-1-sathyanarayanan.kuppuswamy@linux.intel.com
Reviewed-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
Signed-off-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
Signed-off-by: Sasha Levin <sashal@kernel.org>
Signed-off-by: Koichiro Den <koichiro.den@canonical.com>
Signed-off-by: Roxana Nicolescu <roxana.nicolescu@canonical.com>

This commit is contained in:

Kuppuswamy Sathyanarayanan

2024-07-30 15:59:30 +00:00

committed by

Mehmet Basaran

parent 9e0a8415ca

commit c6af2dd444

1 changed files with 1 additions and 1 deletions

									
										drivers/platform/x86/intel/ifs/runtest.c
									
		+1
		-1
	
												View File
												
				@@ -173,8 +173,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;

					int to_start, to_stop;