From 6db3ac3a8e008128e330295f4fb1d2b2779a25d9 Mon Sep 17 00:00:00 2001
From: GangCao <gang.cao@intel.com>
Date: Wed, 27 Jul 2022 21:44:26 -0400
Subject: [PATCH] Test/Bdev: wait PID to exit on error testing

Fix issue: #2611

Change-Id: I6156598936fa44793d9f4ffc6fe5fb3a7bbc6278
Signed-off-by: GangCao <gang.cao@intel.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/13804
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: Konrad Sztyber <konrad.sztyber@intel.com>
Reviewed-by: Jim Harris <james.r.harris@intel.com>
---
 test/bdev/blockdev.sh | 8 +-------
 1 file changed, 1 insertion(+), 7 deletions(-)

diff --git a/test/bdev/blockdev.sh b/test/bdev/blockdev.sh
index ba4d2293d..93491c087 100755
--- a/test/bdev/blockdev.sh
+++ b/test/bdev/blockdev.sh
@@ -403,13 +403,7 @@ function error_test_suite() {
 	$rpc_py bdev_error_inject_error $ERR_DEV 'all' 'failure' -n 5
 
 	$rootdir/test/bdev/bdevperf/bdevperf.py -t 1 perform_tests &
-	sleep 1
-
-	# Bdevperf is expected to exit when hitting error
-	if kill -0 $ERR_PID; then
-		echo "Process still exists, but was expected to fail due to IO error. Pid: $ERR_PID"
-		exit 1
-	fi
+	NOT wait $ERR_PID
 }
 
 function qd_sampling_function_test() {