Commit 1a53c8a5 authored by Ziv Hirsch's avatar Ziv Hirsch Committed by Tomasz Zawadzki
Browse files

nvmf: fix ctrlr_bdev passthru for commands without data 



There are commands without data, like Verify, possibly some vendor specifics opcodes, etc.

Change-Id: Ia190eb0c4ff2b79f7f323c52c29aa2b99b02682c
Signed-off-by: default avatarZiv Hirsch <zivhirsch13@gmail.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/19174


Community-CI: Mellanox Build Bot
Reviewed-by: default avatarChangpeng Liu <changpeng.liu@intel.com>
Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: default avatarTomasz Zawadzki <tomasz.zawadzki@intel.com>
parent ebc052af

lib/nvmf/ctrlr_bdev.c

+2 −2
Original line number Diff line number Diff line
@@ -735,7 +735,7 @@ nvmf_bdev_ctrlr_nvme_passthru_io(struct spdk_bdev *bdev, struct spdk_bdev_desc * 
{

	int rc;



	if (spdk_unlikely(req->iovcnt != 1)) {

	if (spdk_unlikely(req->iovcnt > 1)) {

		req->rsp->nvme_cpl.status.sct = SPDK_NVME_SCT_GENERIC;

		req->rsp->nvme_cpl.status.sc = SPDK_NVME_SC_INTERNAL_DEVICE_ERROR;

		req->rsp->nvme_cpl.status.dnr = 1;
@@ -765,7 +765,7 @@ spdk_nvmf_bdev_ctrlr_nvme_passthru_admin(struct spdk_bdev *bdev, struct spdk_bde 
{

	int rc;



	if (spdk_unlikely(req->iovcnt != 1)) {

	if (spdk_unlikely(req->iovcnt > 1)) {

		req->rsp->nvme_cpl.status.sct = SPDK_NVME_SCT_GENERIC;

		req->rsp->nvme_cpl.status.sc = SPDK_NVME_SC_INTERNAL_DEVICE_ERROR;

		req->rsp->nvme_cpl.status.dnr = 1;