[PATCH] selftests/cpu-hotplug: Fix check for cpu hotplug not supported

Dmytro Maluka posted 1 patch 2 weeks, 3 days ago
tools/testing/selftests/cpu-hotplug/cpu-on-off-test.sh | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
[PATCH] selftests/cpu-hotplug: Fix check for cpu hotplug not supported
Posted by Dmytro Maluka 2 weeks, 3 days ago
If CONFIG_HOTPLUG_CPU is disabled, /sys/devices/system/cpu/cpu*
directories are still populated, so the test fails to correctly detect
that CPU hotplug is not supported.

Fix this by checking for the presence of 'online' files in those
directories instead. The 'online' node is created for the given CPU if
and only if this CPU supports hotplug. So if none of the CPUs have
'online' nodes, it means CPU hotplug is not supported.

Signed-off-by: Dmytro Maluka <dmaluka@chromium.org>
---
 tools/testing/selftests/cpu-hotplug/cpu-on-off-test.sh | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tools/testing/selftests/cpu-hotplug/cpu-on-off-test.sh b/tools/testing/selftests/cpu-hotplug/cpu-on-off-test.sh
index 6232a46ca6e1..287cfd5809f6 100755
--- a/tools/testing/selftests/cpu-hotplug/cpu-on-off-test.sh
+++ b/tools/testing/selftests/cpu-hotplug/cpu-on-off-test.sh
@@ -24,7 +24,7 @@ prerequisite()
 		exit $ksft_skip
 	fi
 
-	if ! ls $SYSFS/devices/system/cpu/cpu* > /dev/null 2>&1; then
+	if ! ls $SYSFS/devices/system/cpu/cpu*/online > /dev/null 2>&1; then
 		echo $msg cpu hotplug is not supported >&2
 		exit $ksft_skip
 	fi
-- 
2.53.0.851.ga537e3e6e9-goog
Re: [PATCH] selftests/cpu-hotplug: Fix check for cpu hotplug not supported
Posted by Shuah Khan 5 days, 11 hours ago
On 3/19/26 09:38, Dmytro Maluka wrote:
> If CONFIG_HOTPLUG_CPU is disabled, /sys/devices/system/cpu/cpu*
> directories are still populated, so the test fails to correctly detect
> that CPU hotplug is not supported.
> 
> Fix this by checking for the presence of 'online' files in those
> directories instead. The 'online' node is created for the given CPU if
> and only if this CPU supports hotplug. So if none of the CPUs have
> 'online' nodes, it means CPU hotplug is not supported.
> 
> Signed-off-by: Dmytro Maluka <dmaluka@chromium.org>
> ---

Applied to linux_kselftest next for Linux 7.1-rc1.

thanks,
-- Shuah