diff --git a/lib/nvmf/ctrlr_bdev.c b/lib/nvmf/ctrlr_bdev.c index 4c2feac83..382cd641c 100644 --- a/lib/nvmf/ctrlr_bdev.c +++ b/lib/nvmf/ctrlr_bdev.c @@ -306,11 +306,6 @@ nvmf_bdev_ctrlr_read_cmd(struct spdk_bdev *bdev, struct spdk_bdev_desc *desc, return SPDK_NVMF_REQUEST_EXEC_STATUS_COMPLETE; } - if (req->zcopy_phase == NVMF_ZCOPY_PHASE_EXECUTE) { - /* Return here after checking the lba etc */ - return SPDK_NVMF_REQUEST_EXEC_STATUS_COMPLETE; - } - assert(!spdk_nvmf_request_using_zcopy(req)); rc = spdk_bdev_readv_blocks(desc, ch, req->iov, req->iovcnt, start_lba, num_blocks, @@ -357,11 +352,6 @@ nvmf_bdev_ctrlr_write_cmd(struct spdk_bdev *bdev, struct spdk_bdev_desc *desc, return SPDK_NVMF_REQUEST_EXEC_STATUS_COMPLETE; } - if (req->zcopy_phase == NVMF_ZCOPY_PHASE_EXECUTE) { - /* Return here after checking the lba etc */ - return SPDK_NVMF_REQUEST_EXEC_STATUS_COMPLETE; - } - assert(!spdk_nvmf_request_using_zcopy(req)); rc = spdk_bdev_writev_blocks(desc, ch, req->iov, req->iovcnt, start_lba, num_blocks, diff --git a/test/unit/lib/nvmf/ctrlr_bdev.c/ctrlr_bdev_ut.c b/test/unit/lib/nvmf/ctrlr_bdev.c/ctrlr_bdev_ut.c index 98051b207..924303699 100644 --- a/test/unit/lib/nvmf/ctrlr_bdev.c/ctrlr_bdev_ut.c +++ b/test/unit/lib/nvmf/ctrlr_bdev.c/ctrlr_bdev_ut.c @@ -752,20 +752,6 @@ test_nvmf_bdev_ctrlr_read_write_cmd(void) rc = nvmf_bdev_ctrlr_read_cmd(&bdev, NULL, NULL, &req); CU_ASSERT(rc == SPDK_NVMF_REQUEST_EXEC_STATUS_ASYNCHRONOUS); - /* Execute zero copy */ - req.zcopy_phase = NVMF_ZCOPY_PHASE_EXECUTE; - - rc = nvmf_bdev_ctrlr_read_cmd(&bdev, NULL, NULL, &req); - CU_ASSERT(rc == SPDK_NVMF_REQUEST_EXEC_STATUS_COMPLETE); - - /* Read cmd request length invalid */ - req.length = 4096; - - rc = nvmf_bdev_ctrlr_read_cmd(&bdev, NULL, NULL, &req); - CU_ASSERT(rc == SPDK_NVMF_REQUEST_EXEC_STATUS_COMPLETE); - CU_ASSERT(rsp.nvme_cpl.status.sct == SPDK_NVME_SCT_GENERIC); - CU_ASSERT(rsp.nvme_cpl.status.sc == SPDK_NVME_SC_DATA_SGL_LENGTH_INVALID); - /* Write two blocks, block size 4096 */ cmd.nvme_cmd.cdw12 = 1; bdev.blockcnt = 100; @@ -775,20 +761,6 @@ test_nvmf_bdev_ctrlr_read_write_cmd(void) rc = nvmf_bdev_ctrlr_write_cmd(&bdev, NULL, NULL, &req); CU_ASSERT(rc == SPDK_NVMF_REQUEST_EXEC_STATUS_ASYNCHRONOUS); - - /* Execute zero copy */ - req.zcopy_phase = NVMF_ZCOPY_PHASE_EXECUTE; - - rc = nvmf_bdev_ctrlr_write_cmd(&bdev, NULL, NULL, &req); - CU_ASSERT(rc == SPDK_NVMF_REQUEST_EXEC_STATUS_COMPLETE); - - /* Write cmd request length invalid */ - req.length = 4096; - - rc = nvmf_bdev_ctrlr_read_cmd(&bdev, NULL, NULL, &req); - CU_ASSERT(rc == SPDK_NVMF_REQUEST_EXEC_STATUS_COMPLETE); - CU_ASSERT(rsp.nvme_cpl.status.sct == SPDK_NVME_SCT_GENERIC); - CU_ASSERT(rsp.nvme_cpl.status.sc == SPDK_NVME_SC_DATA_SGL_LENGTH_INVALID); } static void