Commit ac5578ed authored by Jim Harris's avatar Jim Harris
Browse files

nvme/pcie: remove req_from_current_proc 



This variable was not really needed, we can reduce the code size by
removing it.

Signed-off-by: default avatarJim Harris <jim.harris@samsung.com>
Change-Id: Ia92822408e06a6baf6a9f021ddb5b377c18663d0
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/20645


Reviewed-by: default avatarMichael Haeuptle <michaelhaeuptle@gmail.com>
Reviewed-by: default avatarTomasz Zawadzki <tomasz.zawadzki@intel.com>
Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: default avatarDeepak Abraham Tom <deepak-abraham.tom@hpe.com>
Community-CI: Mellanox Build Bot
Reviewed-by: default avatarChangpeng Liu <changpeng.liu@intel.com>
Reviewed-by: default avatarAleksey Marchuk <alexeymar@nvidia.com>
parent 29773a9c

lib/nvme/nvme_pcie_common.c

+0 −5
Original line number Diff line number Diff line
@@ -672,7 +672,6 @@ nvme_pcie_qpair_complete_tracker(struct spdk_nvme_qpair *qpair, struct nvme_trac 
	struct nvme_pcie_qpair		*pqpair = nvme_pcie_qpair(qpair);

	struct nvme_request		*req;

	bool				retry, error;

	bool				req_from_current_proc = true;

	bool				print_error;



	req = tr->req;
@@ -705,13 +704,9 @@ nvme_pcie_qpair_complete_tracker(struct spdk_nvme_qpair *qpair, struct nvme_trac 


		/* Only check admin requests from different processes. */

		if (nvme_qpair_is_admin_queue(qpair) && req->pid != getpid()) {

			req_from_current_proc = false;

			nvme_pcie_qpair_insert_pending_admin_request(qpair, req, cpl);

		} else {

			nvme_complete_request(tr->cb_fn, tr->cb_arg, qpair, req, cpl);

		}



		if (req_from_current_proc == true) {

			nvme_qpair_free_request(qpair, req);

		}