Commit 161eb083 authored by Karol Latecki's avatar Karol Latecki Committed by Jim Harris
Browse files

test/vhost: remove config-ro Qemu option from config 



Vhost-blk readonly "config-ro" option was
removed from Qemu arguments after merging
pull request f67c9b693a into Qemu master.

"config-ro" Qemu option needs to be removed
from scripts in order for tests to pass.

Nightly test for read-only feature has to be
commented out until Qemu is updated on test
servers.

Change-Id: I0cdc9c2fd3b23ef77a63b54e280e1bb9500229ee
Signed-off-by: default avatarKarol Latecki <karol.latecki@intel.com>
Reviewed-on: https://review.gerrithub.io/413745


Tested-by: default avatarSPDK Automated Test System <sys_sgsw@intel.com>
Reviewed-by: default avatarChangpeng Liu <changpeng.liu@intel.com>
Reviewed-by: default avatarJim Harris <james.r.harris@intel.com>
parent 5d3d5a49

autotest.sh

+3 −3
Original line number Diff line number Diff line
@@ -167,9 +167,9 @@ if [ $SPDK_TEST_VHOST -eq 1 ]; then 
		run_test ./test/vhost/spdk_vhost.sh --migration

		timing_exit vhost_migration



		timing_enter readonly

		run_test ./test/vhost/spdk_vhost.sh --readonly

		timing_exit readonly

		# timing_enter readonly

		# run_test ./test/vhost/spdk_vhost.sh --readonly

		# timing_exit readonly

	fi



	timing_enter integrity_lvol_scsi

test/vhost/common/common.sh

+1 −1
Original line number Diff line number Diff line
@@ -725,7 +725,7 @@ function vm_setup() 
			spdk_vhost_blk)

				notice "using socket $vhost_dir/naa.$disk.$vm_num"

				cmd+="-chardev socket,id=char_$disk,path=$vhost_dir/naa.$disk.$vm_num ${eol}"

				cmd+="-device vhost-user-blk-pci,num-queues=$queue_number,chardev=char_$disk,config-ro=$read_only ${eol}"

				cmd+="-device vhost-user-blk-pci,num-queues=$queue_number,chardev=char_$disk ${eol}"

				;;

			kernel_vhost)

				if [[ -z $disk ]]; then