From 6e4f5e0b53cef380978ba5116412fa191062b7fd Mon Sep 17 00:00:00 2001 From: Seth Howell Date: Fri, 28 Jun 2019 15:13:52 -0700 Subject: [PATCH] test/nvmf: don't use full disk size in host perf. When we have a really large disk, setting up a nested lvol across the entire drive can actually be quite time consuming. When trying to run this test on an 800G p3700 I was actually getting timeouts on the rpcs when creating the nested lvolstore. By only taking the first 20G of the drive to set up the stores, we end up with a much quicker test and conveniently get rid of the timeouts I was facing. Change-Id: Ifd43c21c66bc77b615da9b0f421e95337b4c7019 Signed-off-by: Seth Howell Reviewed-on: https://review.gerrithub.io/c/spdk/spdk/+/459849 Tested-by: SPDK CI Jenkins Reviewed-by: Paul Luse Reviewed-by: Darek Stojaczyk Reviewed-by: Shuhei Matsumoto Reviewed-by: Ben Walker --- test/nvmf/host/perf.sh | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/test/nvmf/host/perf.sh b/test/nvmf/host/perf.sh index 58f37044b2..804b06b197 100755 --- a/test/nvmf/host/perf.sh +++ b/test/nvmf/host/perf.sh @@ -45,11 +45,20 @@ if [ $RUN_NIGHTLY -eq 1 ]; then if [ -n "$local_nvme_trid" ]; then ls_guid=$($rpc_py construct_lvol_store Nvme0n1 lvs_0) get_lvs_free_mb $ls_guid + # We don't need to create an lvol larger than 20G for this test. + # decreasing the size of the nested lvol allows us to take less time setting up + #before running I/O. + if [ $free_mb -gt 20480 ]; then + free_mb=20480 + fi lb_guid=$($rpc_py construct_lvol_bdev -u $ls_guid lbd_0 $free_mb) # Create lvol bdev for nested lvol stores ls_nested_guid=$($rpc_py construct_lvol_store $lb_guid lvs_n_0) get_lvs_free_mb $ls_nested_guid + if [ $free_mb -gt 20480 ]; then + free_mb=20480 + fi lb_nested_guid=$($rpc_py construct_lvol_bdev -u $ls_nested_guid lbd_nest_0 $free_mb) $rpc_py nvmf_subsystem_create nqn.2016-06.io.spdk:cnode1 -a -s SPDK00000000000001 for bdev in $lb_nested_guid; do