From eb5789ceaed06bc973e9962343164e8a9359f731 Mon Sep 17 00:00:00 2001 From: Changpeng Liu Date: Wed, 4 Jan 2023 15:59:03 +0800 Subject: [PATCH] test/vfio_user: use 2048MiB static memory size for bdevperf Previously we use 1024MiB static memory for bdevperf, but it may invoke DPDK dynamic memory allocation when calling `spdk_zmalloc`, and this part of new memory region isn't registered to remote target process, vfio-user like solution is designed for pre-allocated memory, so here we can increase the static memory size as a workaround. Also add debug log when testing. Fix issue #2846. Change-Id: I509093a12a63db2c9e9797da10eab9b5ee0b3aac Signed-off-by: Changpeng Liu Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/16141 Tested-by: SPDK CI Jenkins Reviewed-by: Jim Harris Reviewed-by: Shuhei Matsumoto --- test/vfio_user/virtio/initiator_bdevperf.sh | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/test/vfio_user/virtio/initiator_bdevperf.sh b/test/vfio_user/virtio/initiator_bdevperf.sh index 78f95de13..dabc815f7 100755 --- a/test/vfio_user/virtio/initiator_bdevperf.sh +++ b/test/vfio_user/virtio/initiator_bdevperf.sh @@ -13,7 +13,7 @@ rm -rf $vfu_dir mkdir -p $vfu_dir # Start `spdk_tgt` and configure it -$SPDK_BIN_DIR/spdk_tgt -m 0xf & +$SPDK_BIN_DIR/spdk_tgt -m 0xf -L vfu_virtio & spdk_tgt_pid=$! waitforlisten $spdk_tgt_pid @@ -37,7 +37,7 @@ $rpc_py vfu_virtio_scsi_add_target vfu.scsi --scsi-target-num=1 --bdev-name mall bdevperf=$rootdir/build/examples/bdevperf bdevperf_rpc_sock=/tmp/bdevperf.sock -$bdevperf -r $bdevperf_rpc_sock -g -s 1024 -q 256 -o 4096 -w randrw -M 50 -t 30 -m 0xf0 & +$bdevperf -r $bdevperf_rpc_sock -g -s 2048 -q 256 -o 4096 -w randrw -M 50 -t 30 -m 0xf0 -L vfio_pci -L virtio_vfio_user & bdevperf_pid=$! trap 'killprocess $bdevperf_pid; exit 1' SIGINT SIGTERM EXIT waitforlisten $bdevperf_pid $bdevperf_rpc_sock