From 2250a441c4d2e3ee921b85613ee6111c65f65ca8 Mon Sep 17 00:00:00 2001 From: Ben Walker Date: Thu, 10 Feb 2022 15:13:04 -0700 Subject: [PATCH] test/bdev: In bdev_nvme_ut, count a disconnect as "activity" Count disconnecting a queue pair as activity so that the unit test poll_threads() calls don't bail out until the disconnectedd_qpair_cb is called at least once. Signed-off-by: Ben Walker Change-Id: Idc437d6c589dbf133bfcbb5edba1087f928a718c Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/11507 Community-CI: Broadcom CI Community-CI: Mellanox Build Bot Tested-by: SPDK CI Jenkins Reviewed-by: Aleksey Marchuk Reviewed-by: Jim Harris Reviewed-by: Michael Haeuptle --- test/unit/lib/bdev/nvme/bdev_nvme.c/bdev_nvme_ut.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/test/unit/lib/bdev/nvme/bdev_nvme.c/bdev_nvme_ut.c b/test/unit/lib/bdev/nvme/bdev_nvme.c/bdev_nvme_ut.c index a5026df11..419f6b03a 100644 --- a/test/unit/lib/bdev/nvme/bdev_nvme.c/bdev_nvme_ut.c +++ b/test/unit/lib/bdev/nvme/bdev_nvme.c/bdev_nvme_ut.c @@ -1147,6 +1147,8 @@ spdk_nvme_poll_group_process_completions(struct spdk_nvme_poll_group *group, TAILQ_FOREACH_SAFE(qpair, &group->connected_qpairs, poll_group_tailq, tmp_qpair) { if (qpair->failure_reason != SPDK_NVME_QPAIR_FAILURE_NONE) { spdk_nvme_ctrlr_disconnect_io_qpair(qpair); + /* Bump the number of completions so this counts as "busy" */ + num_completions++; continue; }