From e2025217a7aeb3bc89035eb49dffe19da345bf0e Mon Sep 17 00:00:00 2001 From: Michal Berger Date: Fri, 5 Feb 2021 11:42:52 +0100 Subject: [PATCH] test/scheduler: Wait for a bit for the governor to change the freq Issues like https://github.com/spdk/spdk/issues/1784 showed that governor may need some more time to properly adjust cpufreq's sysfs knobs. In case we see that frequency setting hasn't changed after given iteration, wait for 0.5s to make sure it finally settles. Signed-off-by: Michal Berger Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/6283 (master) (cherry picked from commit 92e49c00446de7c8a3bd6b2bbc2a8053f189ee09) Change-Id: Ibf25d0d1962bf2b07b13d60d6096c5cc185c1279 Signed-off-by: Krzysztof Karas Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/9967 Reviewed-by: Tomasz Zawadzki Reviewed-by: Ben Walker Reviewed-by: Jim Harris Tested-by: Tomasz Zawadzki --- test/scheduler/governor.sh | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/test/scheduler/governor.sh b/test/scheduler/governor.sh index c259fa1776..0ead29cf23 100755 --- a/test/scheduler/governor.sh +++ b/test/scheduler/governor.sh @@ -86,7 +86,8 @@ verify_dpdk_governor() { elif ((main_core_setspeed < old_main_core_setspeed)); then dir=0 elif ((main_core_setspeed == old_main_core_setspeed)); then - # Frequency didn't change, skip + # Frequency didn't change, skip and wait for a bit + sleep 0.5s continue fi