From 8d4ecf1e222d2cd1c2073dab19852281a04dc060 Mon Sep 17 00:00:00 2001 From: Pawel Kaminski Date: Thu, 28 Nov 2019 09:46:29 -0500 Subject: [PATCH] test/vhost: Fix. vm_run takes vm numbers as separate arguments. In patch fdc82d3 'vm_run $@' was changed to 'vm_run "$@"' but arguments like 'vm_run "0 1"' should be passed as 'vm_run 0 1'. Change-Id: I2a51a31fc0b0b55079963b11587be55d9bf86a87 Signed-off-by: Pawel Kaminski Reviewed-on: https://review.gerrithub.io/c/spdk/spdk/+/476087 Tested-by: SPDK CI Jenkins Reviewed-by: Karol Latecki Reviewed-by: Shuhei Matsumoto Reviewed-by: Tomasz Zawadzki Reviewed-by: Jim Harris Reviewed-by: Ben Walker --- test/vhost/hotplug/common.sh | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/test/vhost/hotplug/common.sh b/test/vhost/hotplug/common.sh index 1619ebba8..adf7cd44b 100644 --- a/test/vhost/hotplug/common.sh +++ b/test/vhost/hotplug/common.sh @@ -105,13 +105,15 @@ function vms_setup() { } function vm_run_with_arg() { - vm_run "$@" - vm_wait_for_boot 300 "$@" + local vms_to_run="$*" + vm_run $vms_to_run + vm_wait_for_boot 300 $vms_to_run } function vms_setup_and_run() { + local vms_to_run="$*" vms_setup - vm_run_with_arg "$@" + vm_run_with_arg $vms_to_run } function vms_prepare() {