From 7478e053e1782d44d0a91c17d344b32d876ec9c3 Mon Sep 17 00:00:00 2001 From: Michal Berger Date: Thu, 22 Apr 2021 18:33:46 +0200 Subject: [PATCH] test/scheduler: Don't use extra time while waiting for idle cpus It's not needed anymore since we pick up now last idle sample in comparision to remaining states given cpu spent time in. This way we get the faster view if in given moment cpu is busy or not. Signed-off-by: Michal Berger Change-Id: I3677c03ee3c2e42662df35dd0a6e7b5b6b1f98ed Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/7565 Tested-by: SPDK CI Jenkins Community-CI: Broadcom CI Reviewed-by: Tomasz Zawadzki Reviewed-by: Jim Harris Reviewed-by: Karol Latecki --- test/scheduler/interrupt.sh | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/test/scheduler/interrupt.sh b/test/scheduler/interrupt.sh index e5b20cb58..4d76c33bc 100755 --- a/test/scheduler/interrupt.sh +++ b/test/scheduler/interrupt.sh @@ -60,8 +60,7 @@ interrupt() { for cpu in "${!threads[@]}"; do active_thread "${threads[cpu]}" 0 cpus_to_collect=("$cpu") - # Give some extra time for the cpu to spin down - collect_cpu_idle 10 + collect_cpu_idle reactor_framework=$(rpc_cmd framework_get_reactors | jq -r '.reactors[]') [[ -z $(jq -r "select(.lcore == $cpu) | .lw_threads[].id" <<< "$reactor_framework") ]] [[ -n $(jq -r "select(.lcore == $spdk_main_core) | .lw_threads[] | select(.name == \"thread$cpu\")" <<< "$reactor_framework") ]]