Commit c650e144 authored Oct 29, 2019 by Karol Latecki Committed by Tomasz Zawadzki Nov 05, 2019

test/vhost: fix respecting additional params for vhost_run



Reinstating "shift" removed in commit a2ab5eb0 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 <karol.latecki@intel.com>
Reviewed-on: https://review.gerrithub.io/c/spdk/spdk/+/472681


Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: Maciej Wawryk <maciejx.wawryk@intel.com>
Reviewed-by: Ben Walker <benjamin.walker@intel.com>
Reviewed-by: Shuhei Matsumoto <shuhei.matsumoto.xt@hitachi.com>

parent 456d60a7

test/vhost/common.sh

+3 −1

Original line number	Diff line number	Diff line
		@@ -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"