From c650e144348d00782f2cbcca837a6e152d7ccd8f Mon Sep 17 00:00:00 2001 From: Karol Latecki Date: Tue, 29 Oct 2019 10:20:23 +0100 Subject: [PATCH] test/vhost: fix respecting additional params for vhost_run Reinstating "shift" removed in commit a2ab5eb as it was actually needed. Change "cmd" command string to use $@ instead of $2 so that we're able to use more than one additional param for vhost instance. Change-Id: If656c4319d7acfecddd2ff73104d3c79b799689f Signed-off-by: Karol Latecki Reviewed-on: https://review.gerrithub.io/c/spdk/spdk/+/472681 Tested-by: SPDK CI Jenkins Reviewed-by: Maciej Wawryk Reviewed-by: Ben Walker Reviewed-by: Shuhei Matsumoto --- test/vhost/common.sh | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/test/vhost/common.sh b/test/vhost/common.sh index 1b390fb90..030dbd7c8 100644 --- a/test/vhost/common.sh +++ b/test/vhost/common.sh @@ -116,6 +116,8 @@ function vhost_run() return 1 fi + shift + local vhost_dir="$(get_vhost_dir $vhost_name)" local vhost_app="$rootdir/app/vhost/vhost" local vhost_log_file="$vhost_dir/vhost.log" @@ -131,7 +133,7 @@ function vhost_run() return 1 fi - local cmd="$vhost_app -r $vhost_dir/rpc.sock $2" + local cmd="$vhost_app -r $vhost_dir/rpc.sock $@" notice "Loging to: $vhost_log_file" notice "Socket: $vhost_socket"