From d47b0d2d7d1d55189bcf6d74ea1fe412786a4945 Mon Sep 17 00:00:00 2001 From: Tomasz Zawadzki Date: Thu, 15 Jul 2021 07:13:44 -0400 Subject: [PATCH] test/scheduler: remove unnecessary assignments of thread names The "threadX_name" variables are not used anywhere except creating the thread. Instead use the names directly. Signed-off-by: Tomasz Zawadzki Change-Id: I6bdb2eb1d1e96fe888a4d5d9765fc0b0dfa74272 Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/8794 Community-CI: Broadcom CI Tested-by: SPDK CI Jenkins Reviewed-by: Michal Berger Reviewed-by: Shuhei Matsumoto Reviewed-by: Jim Harris Reviewed-by: Konrad Sztyber --- test/scheduler/load_balancing.sh | 9 +++------ 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/test/scheduler/load_balancing.sh b/test/scheduler/load_balancing.sh index 3f867ce93..ce4dd1e2c 100755 --- a/test/scheduler/load_balancing.sh +++ b/test/scheduler/load_balancing.sh @@ -23,10 +23,8 @@ busy() { fold_list_onto_array selected_cpus "${cpus[@]:1:2}" - thread0_name=thread0 - thread0=$(create_thread -n "$thread0_name" -m "$(mask_cpus "${selected_cpus[@]}")" -a 100) - thread1_name=thread1 - thread1=$(create_thread -n "$thread1_name" -m "$(mask_cpus "${selected_cpus[@]}")" -a 100) + thread0=$(create_thread -n "thread0" -m "$(mask_cpus "${selected_cpus[@]}")" -a 100) + thread1=$(create_thread -n "thread1" -m "$(mask_cpus "${selected_cpus[@]}")" -a 100) local samples=0 @@ -94,8 +92,7 @@ balanced() { # Exclude main cpu fold_list_onto_array selected_cpus "${cpus[@]:1}" - thread0_name=thread0 - thread0=$(create_thread -n "$thread0_name" -m "$(mask_cpus "${selected_cpus[@]}")" -a 0) + thread0=$(create_thread -n "thread0" -m "$(mask_cpus "${selected_cpus[@]}")" -a 0) for cpu in "${selected_cpus[@]::${#selected_cpus[@]}-1}"; do extra_threads+=("$(create_thread -n "thread_cpu_$cpu" -m "$(mask_cpus "$cpu")" -a 100)") done