From ad672ce24923cb5d40057ba6292d6160ab354baf Mon Sep 17 00:00:00 2001
From: Michal Berger <michalx.berger@intel.com>
Date: Mon, 25 Oct 2021 14:05:21 +0200
Subject: [PATCH] test/interrupt: Fix syntax

return was not used in proper context. Let the errexit do the job to
signal a failure.

Signed-off-by: Michal Berger <michalx.berger@intel.com>
Change-Id: I731545c913858286009168041030b4b362cebc32
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/9987
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: Shuhei Matsumoto <shuhei.matsumoto.xt@hitachi.com>
Reviewed-by: Jim Harris <james.r.harris@intel.com>
---
 test/interrupt/reap_unregistered_poller.sh | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/test/interrupt/reap_unregistered_poller.sh b/test/interrupt/reap_unregistered_poller.sh
index 88c13a69f..162de33cf 100755
--- a/test/interrupt/reap_unregistered_poller.sh
+++ b/test/interrupt/reap_unregistered_poller.sh
@@ -29,9 +29,7 @@ remaining_pollers+=$(jq -r '.timed_pollers[].name' <<< $app_thread)
 
 # Since bdev_aio created pollers were already unregistered, so
 # remaining_pollers should be same with native_pollers.
-if [[ "$remaining_pollers" != "$native_pollers" ]]; then
-	return 1
-fi
+[[ "$remaining_pollers" == "$native_pollers" ]]
 
 trap - SIGINT SIGTERM EXIT
 killprocess $intr_tgt_pid