Commit b5e08f54 authored by Konrad Sztyber's avatar Konrad Sztyber Committed by Tomasz Zawadzki
Browse files

nvmf: group FLUSH with other mandatory commands 



The spec says that the FLUSH command is mandatory, so group it with
other mandatory commands.

Signed-off-by: default avatarKonrad Sztyber <konrad.sztyber@intel.com>
Change-Id: I9ea97107ae9186ebd47742663444f3853f87ff27
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/23375


Reviewed-by: default avatarJacek Kalwas <jacek.kalwas@intel.com>
Community-CI: Mellanox Build Bot
Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: default avatarTomasz Zawadzki <tomasz.zawadzki@intel.com>
Reviewed-by: default avatarJim Harris <jim.harris@samsung.com>
parent 7d19bf23

lib/nvmf/ctrlr.c

+2 −2
Original line number Diff line number Diff line
@@ -4415,6 +4415,8 @@ nvmf_ctrlr_process_io_cmd(struct spdk_nvmf_request *req) 
			return nvmf_bdev_ctrlr_read_cmd(bdev, desc, ch, req);

		case SPDK_NVME_OPC_WRITE:

			return nvmf_bdev_ctrlr_write_cmd(bdev, desc, ch, req);

		case SPDK_NVME_OPC_FLUSH:

			return nvmf_bdev_ctrlr_flush_cmd(bdev, desc, ch, req);

		case SPDK_NVME_OPC_COMPARE:

			if (spdk_unlikely(!ctrlr->cdata.oncs.compare)) {

				goto invalid_opcode;
@@ -4425,8 +4427,6 @@ nvmf_ctrlr_process_io_cmd(struct spdk_nvmf_request *req) 
				goto invalid_opcode;

			}

			return nvmf_bdev_ctrlr_write_zeroes_cmd(bdev, desc, ch, req);

		case SPDK_NVME_OPC_FLUSH:

			return nvmf_bdev_ctrlr_flush_cmd(bdev, desc, ch, req);

		case SPDK_NVME_OPC_DATASET_MANAGEMENT:

			if (spdk_unlikely(!ctrlr->cdata.oncs.dsm)) {

				goto invalid_opcode;