Commit 64eb2cc1 authored by Jim Harris's avatar Jim Harris Committed by Tomasz Zawadzki
Browse files

bdev/nvme: account for abort completion with cdw0=1 



When an abort command completes successfully,
cdw0 bit 0 may be set to indicate that the controller
was unable to abort the specified cid.  In that case
the bdev nvme abort completion callback needs
to reset the controller.

Signed-off-by: default avatarJim Harris <james.r.harris@intel.com>
Change-Id: I42a5e96e19e113c38dec67c2d8575a11f39d41a9

Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/9320


Community-CI: Broadcom CI <spdk-ci.pdl@broadcom.com>
Community-CI: Mellanox Build Bot
Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: default avatarChangpeng Liu <changpeng.liu@intel.com>
Reviewed-by: default avatarZiye Yang <ziye.yang@intel.com>
Reviewed-by: default avatarKonrad Sztyber <konrad.sztyber@intel.com>
Reviewed-by: default avatarTomasz Zawadzki <tomasz.zawadzki@intel.com>
parent cda7ddeb

module/bdev/nvme/bdev_nvme.c

+3 −0
Original line number Diff line number Diff line
@@ -1684,6 +1684,9 @@ nvme_abort_cpl(void *ctx, const struct spdk_nvme_cpl *cpl) 
		SPDK_WARNLOG("Abort failed. Resetting controller. sc is %u, sct is %u.\n", cpl->status.sc,

			     cpl->status.sct);

		bdev_nvme_reset(nvme_ctrlr);

	} else if (cpl->cdw0 & 0x1) {

		SPDK_WARNLOG("Specified command could not be aborted.\n");

		bdev_nvme_reset(nvme_ctrlr);

	}

}