ivampiresp/Spdk
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity
cddfe002e1
Spdk/test/nvmf/target/nmic.sh

65 lines
1.9 KiB
Bash
Raw Normal View History

test: add nvmf multi-path testing Change-Id: Iea0c8e8c2405d008e77f8bea1a3c30f2f873fa3d Signed-off-by: Liang Yan <liang.z.yan@intel.com> Reviewed-on: https://review.gerrithub.io/420134 Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Chandler-Test-Pool: SPDK Automated Test System <sys_sgsw@intel.com> Reviewed-by: Ben Walker <benjamin.walker@intel.com> Reviewed-by: Jim Harris <james.r.harris@intel.com>
2018-07-23 06:53:41 +00:00
#!/usr/bin/env bash
testdir=$(readlink -f $(dirname $0))
rootdir=$(readlink -f $testdir/../../..)
source $rootdir/test/common/autotest_common.sh
source $rootdir/test/nvmf/common.sh
test/nvmf: make malloc bdevs always the same size A few test scripts were creating 128MB malloc bdevs instead of 64MB - but there's no real reason for the difference. So make them all the same. A future patch will make all of these size variables common instead of duplicated in each file. Signed-off-by: Jim Harris <james.r.harris@intel.com> Change-Id: I865ef08f384e026d0d3e7b064fb3a2d5958f054c Reviewed-on: https://review.gerrithub.io/c/spdk/spdk/+/454688 Reviewed-by: Shuhei Matsumoto <shuhei.matsumoto.xt@hitachi.com> Reviewed-by: Ben Walker <benjamin.walker@intel.com> Reviewed-by: Seth Howell <seth.howell5141@gmail.com> Reviewed-by: Paul Luse <paul.e.luse@intel.com> Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
2019-05-15 23:20:10 +00:00
MALLOC_BDEV_SIZE=64
test: add nvmf multi-path testing Change-Id: Iea0c8e8c2405d008e77f8bea1a3c30f2f873fa3d Signed-off-by: Liang Yan <liang.z.yan@intel.com> Reviewed-on: https://review.gerrithub.io/420134 Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Chandler-Test-Pool: SPDK Automated Test System <sys_sgsw@intel.com> Reviewed-by: Ben Walker <benjamin.walker@intel.com> Reviewed-by: Jim Harris <james.r.harris@intel.com>
2018-07-23 06:53:41 +00:00
MALLOC_BLOCK_SIZE=512
test: don't call python explicitly We use python3 now, not python. This is important because in next patch we will use a new 'aliases' keyword argument in the argparse API that's only available in Python 3. While here, clean up some documentation that was instructing users to call python explicitly instead of just invoking scripts/rpc.py directly. Signed-off-by: Jim Harris <james.r.harris@intel.com> Change-Id: I0c11e131cd56f7c994aa8fe21b4b00ae33489fdd Reviewed-on: https://review.gerrithub.io/c/spdk/spdk/+/453460 Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Reviewed-by: Ben Walker <benjamin.walker@intel.com> Reviewed-by: Pawel Kaminski <pawelx.kaminski@intel.com> Reviewed-by: Darek Stojaczyk <dariusz.stojaczyk@intel.com>
2019-05-06 20:18:07 +00:00
rpc_py="$rootdir/scripts/rpc.py"
test: add nvmf multi-path testing Change-Id: Iea0c8e8c2405d008e77f8bea1a3c30f2f873fa3d Signed-off-by: Liang Yan <liang.z.yan@intel.com> Reviewed-on: https://review.gerrithub.io/420134 Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Chandler-Test-Pool: SPDK Automated Test System <sys_sgsw@intel.com> Reviewed-by: Ben Walker <benjamin.walker@intel.com> Reviewed-by: Jim Harris <james.r.harris@intel.com>
2018-07-23 06:53:41 +00:00
set -e
test/nvmf: reduce duplication between target scripts All of the checking for NVMF_FIRST_TARGET_IP, starting the target, waiting on the target and modprobing nvme-rdma is duplicated in every script. So move a lot of this either to nvmftestinit() or a new nvmfappstart() function in common.sh. Also just kill the nvmf target in nvmftestfini, rather than the script to explicitly kill it. Signed-off-by: Jim Harris <james.r.harris@intel.com> Change-Id: I5864404610a4244473f460d48264de92687ed867 Reviewed-on: https://review.gerrithub.io/c/spdk/spdk/+/454678 Reviewed-by: Shuhei Matsumoto <shuhei.matsumoto.xt@hitachi.com> Reviewed-by: Seth Howell <seth.howell5141@gmail.com> Reviewed-by: Changpeng Liu <changpeng.liu@intel.com> Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
2019-05-13 19:11:00 +00:00
timing_enter nmic
test/nvmf: simplify "iso" parameter Pull this parameter when sourcing common.sh and set a RUN_IN_ISOLATION flag if it was specified. This reduces the overall code a bit plus avoids having to pass the "iso" parameter to nvmftestinit and nvmftestfini. Signed-off-by: Jim Harris <james.r.harris@intel.com> Change-Id: Idbbe27bfb7b62857500176cdb5d90e381fce4ffc Reviewed-on: https://review.gerrithub.io/c/spdk/spdk/+/454814 Reviewed-by: Seth Howell <seth.howell5141@gmail.com> Reviewed-by: Shuhei Matsumoto <shuhei.matsumoto.xt@hitachi.com> Reviewed-by: Changpeng Liu <changpeng.liu@intel.com> Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
2019-05-16 19:38:53 +00:00
nvmftestinit
test/nvmf: reduce duplication between target scripts All of the checking for NVMF_FIRST_TARGET_IP, starting the target, waiting on the target and modprobing nvme-rdma is duplicated in every script. So move a lot of this either to nvmftestinit() or a new nvmfappstart() function in common.sh. Also just kill the nvmf target in nvmftestfini, rather than the script to explicitly kill it. Signed-off-by: Jim Harris <james.r.harris@intel.com> Change-Id: I5864404610a4244473f460d48264de92687ed867 Reviewed-on: https://review.gerrithub.io/c/spdk/spdk/+/454678 Reviewed-by: Shuhei Matsumoto <shuhei.matsumoto.xt@hitachi.com> Reviewed-by: Seth Howell <seth.howell5141@gmail.com> Reviewed-by: Changpeng Liu <changpeng.liu@intel.com> Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
2019-05-13 19:11:00 +00:00
nvmfappstart "-m 0xF"
test: add nvmf multi-path testing Change-Id: Iea0c8e8c2405d008e77f8bea1a3c30f2f873fa3d Signed-off-by: Liang Yan <liang.z.yan@intel.com> Reviewed-on: https://review.gerrithub.io/420134 Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Chandler-Test-Pool: SPDK Automated Test System <sys_sgsw@intel.com> Reviewed-by: Ben Walker <benjamin.walker@intel.com> Reviewed-by: Jim Harris <james.r.harris@intel.com>
2018-07-23 06:53:41 +00:00
NVMF_SECOND_TARGET_IP=$(echo "$RDMA_IP_LIST" | sed -n 2p)
test/nvmf: remove -p num_queues restrictions There's a bug in Linux 5.1 (possibly 5.0 too) where the kernel initiator driver crashes if it can't get one queue per CPU. This will get fixed eventually, but for now we need to remove the cases where we restrict the number of queues per controller so that we can test on newer kernels. Even on cases where we're testing the SPDK initiator, there's no real need to restrict the number of queue pairs. The kernel will eventually get fixed, but we should be testing with default behavior anyways (the kernel wants lots of queues). We'll also want to add some regression tests to make sure the kernel doesn't break again. But that will all come later. Signed-off-by: Jim Harris <james.r.harris@intel.com> Change-Id: I9979e6d94456e075688b822b042936b63e518a4a Reviewed-on: https://review.gerrithub.io/c/spdk/spdk/+/454819 Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Reviewed-by: Darek Stojaczyk <dariusz.stojaczyk@intel.com> Reviewed-by: Shuhei Matsumoto <shuhei.matsumoto.xt@hitachi.com>
2019-05-16 21:39:52 +00:00
$rpc_py nvmf_create_transport -t rdma -u 8192
test: add nvmf multi-path testing Change-Id: Iea0c8e8c2405d008e77f8bea1a3c30f2f873fa3d Signed-off-by: Liang Yan <liang.z.yan@intel.com> Reviewed-on: https://review.gerrithub.io/420134 Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Chandler-Test-Pool: SPDK Automated Test System <sys_sgsw@intel.com> Reviewed-by: Ben Walker <benjamin.walker@intel.com> Reviewed-by: Jim Harris <james.r.harris@intel.com>
2018-07-23 06:53:41 +00:00
# Create subsystems
test/nvmf: simplify more of the nvmf test scripts This should cover all of the cases that we want to changes for now - where it was doing an unnecessarily loop over just one malloc bdev. A few cases remain, but I'm planning to strengthen those tests with more bdevs. Signed-off-by: Jim Harris <james.r.harris@intel.com> Change-Id: I16f9e82b4a551bcf1f491fc506e11c0ee98b60d9 Reviewed-on: https://review.gerrithub.io/c/spdk/spdk/+/452917 Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Reviewed-by: Ben Walker <benjamin.walker@intel.com> Reviewed-by: Seth Howell <seth.howell5141@gmail.com> Reviewed-by: Shuhei Matsumoto <shuhei.matsumoto.xt@hitachi.com> Reviewed-by: Changpeng Liu <changpeng.liu@intel.com>
2019-05-02 21:14:19 +00:00
$rpc_py construct_malloc_bdev $MALLOC_BDEV_SIZE $MALLOC_BLOCK_SIZE -b Malloc0
test: No longer use construct_nvmf_subsystem Use nvmf_subsystem_create instead. Change-Id: Idcfa6526ef47e2839f69acd5d7dc08e65e5150b5 Signed-off-by: Ben Walker <benjamin.walker@intel.com> Reviewed-on: https://review.gerrithub.io/425187 Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Chandler-Test-Pool: SPDK Automated Test System <sys_sgsw@intel.com> Reviewed-by: Jim Harris <james.r.harris@intel.com> Reviewed-by: Changpeng Liu <changpeng.liu@intel.com>
2018-09-10 19:04:31 +00:00
$rpc_py nvmf_subsystem_create nqn.2016-06.io.spdk:cnode1 -a -s SPDK1
test/nvmf: simplify more of the nvmf test scripts This should cover all of the cases that we want to changes for now - where it was doing an unnecessarily loop over just one malloc bdev. A few cases remain, but I'm planning to strengthen those tests with more bdevs. Signed-off-by: Jim Harris <james.r.harris@intel.com> Change-Id: I16f9e82b4a551bcf1f491fc506e11c0ee98b60d9 Reviewed-on: https://review.gerrithub.io/c/spdk/spdk/+/452917 Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Reviewed-by: Ben Walker <benjamin.walker@intel.com> Reviewed-by: Seth Howell <seth.howell5141@gmail.com> Reviewed-by: Shuhei Matsumoto <shuhei.matsumoto.xt@hitachi.com> Reviewed-by: Changpeng Liu <changpeng.liu@intel.com>
2019-05-02 21:14:19 +00:00
$rpc_py nvmf_subsystem_add_ns nqn.2016-06.io.spdk:cnode1 Malloc0
test/nvmf: use "rdma" when creating transport This is a preparatory patch for specifying the transport when calling the script. Better to use "rdma" across the board instead of a mix of "rdma" and "RDMA". Signed-off-by: Jim Harris <james.r.harris@intel.com> Change-Id: If7170cff6ba2ec51e92a0423c0b24cc141054b55 Reviewed-on: https://review.gerrithub.io/c/spdk/spdk/+/454685 Reviewed-by: Shuhei Matsumoto <shuhei.matsumoto.xt@hitachi.com> Reviewed-by: Changpeng Liu <changpeng.liu@intel.com> Reviewed-by: Ben Walker <benjamin.walker@intel.com> Reviewed-by: Seth Howell <seth.howell5141@gmail.com> Reviewed-by: Paul Luse <paul.e.luse@intel.com> Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
2019-05-13 18:56:19 +00:00
$rpc_py nvmf_subsystem_add_listener nqn.2016-06.io.spdk:cnode1 -t rdma -a $NVMF_FIRST_TARGET_IP -s "$NVMF_PORT"
test: add nvmf multi-path testing Change-Id: Iea0c8e8c2405d008e77f8bea1a3c30f2f873fa3d Signed-off-by: Liang Yan <liang.z.yan@intel.com> Reviewed-on: https://review.gerrithub.io/420134 Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Chandler-Test-Pool: SPDK Automated Test System <sys_sgsw@intel.com> Reviewed-by: Ben Walker <benjamin.walker@intel.com> Reviewed-by: Jim Harris <james.r.harris@intel.com>
2018-07-23 06:53:41 +00:00
echo "test case1: single bdev can't be used in multiple subsystems"
set +e
test: No longer use construct_nvmf_subsystem Use nvmf_subsystem_create instead. Change-Id: Idcfa6526ef47e2839f69acd5d7dc08e65e5150b5 Signed-off-by: Ben Walker <benjamin.walker@intel.com> Reviewed-on: https://review.gerrithub.io/425187 Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Chandler-Test-Pool: SPDK Automated Test System <sys_sgsw@intel.com> Reviewed-by: Jim Harris <james.r.harris@intel.com> Reviewed-by: Changpeng Liu <changpeng.liu@intel.com>
2018-09-10 19:04:31 +00:00
$rpc_py nvmf_subsystem_create nqn.2016-06.io.spdk:cnode2 -a -s SPDK2
test/nvmf: use "rdma" when creating transport This is a preparatory patch for specifying the transport when calling the script. Better to use "rdma" across the board instead of a mix of "rdma" and "RDMA". Signed-off-by: Jim Harris <james.r.harris@intel.com> Change-Id: If7170cff6ba2ec51e92a0423c0b24cc141054b55 Reviewed-on: https://review.gerrithub.io/c/spdk/spdk/+/454685 Reviewed-by: Shuhei Matsumoto <shuhei.matsumoto.xt@hitachi.com> Reviewed-by: Changpeng Liu <changpeng.liu@intel.com> Reviewed-by: Ben Walker <benjamin.walker@intel.com> Reviewed-by: Seth Howell <seth.howell5141@gmail.com> Reviewed-by: Paul Luse <paul.e.luse@intel.com> Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
2019-05-13 18:56:19 +00:00
$rpc_py nvmf_subsystem_add_listener nqn.2016-06.io.spdk:cnode2 -t rdma -a $NVMF_FIRST_TARGET_IP -s "$NVMF_PORT"
test/nvmf: simplify more of the nvmf test scripts This should cover all of the cases that we want to changes for now - where it was doing an unnecessarily loop over just one malloc bdev. A few cases remain, but I'm planning to strengthen those tests with more bdevs. Signed-off-by: Jim Harris <james.r.harris@intel.com> Change-Id: I16f9e82b4a551bcf1f491fc506e11c0ee98b60d9 Reviewed-on: https://review.gerrithub.io/c/spdk/spdk/+/452917 Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Reviewed-by: Ben Walker <benjamin.walker@intel.com> Reviewed-by: Seth Howell <seth.howell5141@gmail.com> Reviewed-by: Shuhei Matsumoto <shuhei.matsumoto.xt@hitachi.com> Reviewed-by: Changpeng Liu <changpeng.liu@intel.com>
2019-05-02 21:14:19 +00:00
$rpc_py nvmf_subsystem_add_ns nqn.2016-06.io.spdk:cnode2 Malloc0
nmic_status=$?
if [ $nmic_status -eq 0 ]; then
echo " Adding namespace passed - failure expected."
test/nvmf: reduce duplication between target scripts All of the checking for NVMF_FIRST_TARGET_IP, starting the target, waiting on the target and modprobing nvme-rdma is duplicated in every script. So move a lot of this either to nvmftestinit() or a new nvmfappstart() function in common.sh. Also just kill the nvmf target in nvmftestfini, rather than the script to explicitly kill it. Signed-off-by: Jim Harris <james.r.harris@intel.com> Change-Id: I5864404610a4244473f460d48264de92687ed867 Reviewed-on: https://review.gerrithub.io/c/spdk/spdk/+/454678 Reviewed-by: Shuhei Matsumoto <shuhei.matsumoto.xt@hitachi.com> Reviewed-by: Seth Howell <seth.howell5141@gmail.com> Reviewed-by: Changpeng Liu <changpeng.liu@intel.com> Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
2019-05-13 19:11:00 +00:00
nvmfcleanup
nvmftestfini
test/nvmf: simplify more of the nvmf test scripts This should cover all of the cases that we want to changes for now - where it was doing an unnecessarily loop over just one malloc bdev. A few cases remain, but I'm planning to strengthen those tests with more bdevs. Signed-off-by: Jim Harris <james.r.harris@intel.com> Change-Id: I16f9e82b4a551bcf1f491fc506e11c0ee98b60d9 Reviewed-on: https://review.gerrithub.io/c/spdk/spdk/+/452917 Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Reviewed-by: Ben Walker <benjamin.walker@intel.com> Reviewed-by: Seth Howell <seth.howell5141@gmail.com> Reviewed-by: Shuhei Matsumoto <shuhei.matsumoto.xt@hitachi.com> Reviewed-by: Changpeng Liu <changpeng.liu@intel.com>
2019-05-02 21:14:19 +00:00
exit 1
else
echo " Adding namespace failed - expected result."
fi
test: add nvmf multi-path testing Change-Id: Iea0c8e8c2405d008e77f8bea1a3c30f2f873fa3d Signed-off-by: Liang Yan <liang.z.yan@intel.com> Reviewed-on: https://review.gerrithub.io/420134 Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Chandler-Test-Pool: SPDK Automated Test System <sys_sgsw@intel.com> Reviewed-by: Ben Walker <benjamin.walker@intel.com> Reviewed-by: Jim Harris <james.r.harris@intel.com>
2018-07-23 06:53:41 +00:00
set -e
echo "test case2: host connect to nvmf target in multiple paths"
if [ ! -z $NVMF_SECOND_TARGET_IP ]; then
test/nvmf: use "rdma" when creating transport This is a preparatory patch for specifying the transport when calling the script. Better to use "rdma" across the board instead of a mix of "rdma" and "RDMA". Signed-off-by: Jim Harris <james.r.harris@intel.com> Change-Id: If7170cff6ba2ec51e92a0423c0b24cc141054b55 Reviewed-on: https://review.gerrithub.io/c/spdk/spdk/+/454685 Reviewed-by: Shuhei Matsumoto <shuhei.matsumoto.xt@hitachi.com> Reviewed-by: Changpeng Liu <changpeng.liu@intel.com> Reviewed-by: Ben Walker <benjamin.walker@intel.com> Reviewed-by: Seth Howell <seth.howell5141@gmail.com> Reviewed-by: Paul Luse <paul.e.luse@intel.com> Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
2019-05-13 18:56:19 +00:00
$rpc_py nvmf_subsystem_add_listener nqn.2016-06.io.spdk:cnode1 -t rdma -a $NVMF_SECOND_TARGET_IP -s $NVMF_PORT
test: add nvmf multi-path testing Change-Id: Iea0c8e8c2405d008e77f8bea1a3c30f2f873fa3d Signed-off-by: Liang Yan <liang.z.yan@intel.com> Reviewed-on: https://review.gerrithub.io/420134 Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Chandler-Test-Pool: SPDK Automated Test System <sys_sgsw@intel.com> Reviewed-by: Ben Walker <benjamin.walker@intel.com> Reviewed-by: Jim Harris <james.r.harris@intel.com>
2018-07-23 06:53:41 +00:00
nvme connect -t rdma -n "nqn.2016-06.io.spdk:cnode1" -a "$NVMF_FIRST_TARGET_IP" -s "$NVMF_PORT"
nvme connect -t rdma -n "nqn.2016-06.io.spdk:cnode1" -a "$NVMF_SECOND_TARGET_IP" -s "$NVMF_PORT"
waitforblk "nvme0n1"
scripts/fio.py: add argparse for command line parameters This script was getting a bit unwieldy. adding argparse will make it easier to add more options in the future or to set default values and validate arguments going forward. Change-Id: I1ffdbdf2082287ceb8a88cd3eb6ecf9bbd6c9e11 Signed-off-by: Seth Howell <seth.howell@intel.com> Reviewed-on: https://review.gerrithub.io/c/spdk/spdk/+/455724 Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Reviewed-by: Jim Harris <james.r.harris@intel.com> Reviewed-by: Shuhei Matsumoto <shuhei.matsumoto.xt@hitachi.com> Reviewed-by: Darek Stojaczyk <dariusz.stojaczyk@intel.com>
2019-05-24 22:58:19 +00:00
$rootdir/scripts/fio.py -p nvmf -i 4096 -d 1 -t write -r 1 -v
test: add nvmf multi-path testing Change-Id: Iea0c8e8c2405d008e77f8bea1a3c30f2f873fa3d Signed-off-by: Liang Yan <liang.z.yan@intel.com> Reviewed-on: https://review.gerrithub.io/420134 Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Chandler-Test-Pool: SPDK Automated Test System <sys_sgsw@intel.com> Reviewed-by: Ben Walker <benjamin.walker@intel.com> Reviewed-by: Jim Harris <james.r.harris@intel.com>
2018-07-23 06:53:41 +00:00
fi
nvme disconnect -n "nqn.2016-06.io.spdk:cnode1" || true
trap - SIGINT SIGTERM EXIT
nvmfcleanup
test/nvmf: simplify "iso" parameter Pull this parameter when sourcing common.sh and set a RUN_IN_ISOLATION flag if it was specified. This reduces the overall code a bit plus avoids having to pass the "iso" parameter to nvmftestinit and nvmftestfini. Signed-off-by: Jim Harris <james.r.harris@intel.com> Change-Id: Idbbe27bfb7b62857500176cdb5d90e381fce4ffc Reviewed-on: https://review.gerrithub.io/c/spdk/spdk/+/454814 Reviewed-by: Seth Howell <seth.howell5141@gmail.com> Reviewed-by: Shuhei Matsumoto <shuhei.matsumoto.xt@hitachi.com> Reviewed-by: Changpeng Liu <changpeng.liu@intel.com> Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
2019-05-16 19:38:53 +00:00
nvmftestfini
test: add nvmf multi-path testing Change-Id: Iea0c8e8c2405d008e77f8bea1a3c30f2f873fa3d Signed-off-by: Liang Yan <liang.z.yan@intel.com> Reviewed-on: https://review.gerrithub.io/420134 Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Chandler-Test-Pool: SPDK Automated Test System <sys_sgsw@intel.com> Reviewed-by: Ben Walker <benjamin.walker@intel.com> Reviewed-by: Jim Harris <james.r.harris@intel.com>
2018-07-23 06:53:41 +00:00
timing_exit nmic
Reference in New Issue Copy Permalink