Commit c85164bd authored May 28, 2019 by Jim Harris

nvme: add explicit "inline" keyword to a couple of functions



Profiling showed these weren't getting inlined - so add
the inline keyword to make sure it happens.  This helps
improve performance a bit.

Signed-off-by: Jim Harris <james.r.harris@intel.com>
Change-Id: Ia86edccc9163258efdcddcce6989a71fb180caf6

Reviewed-on: https://review.gerrithub.io/c/spdk/spdk/+/456099


Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: Ben Walker <benjamin.walker@intel.com>
Reviewed-by: Shuhei Matsumoto <shuhei.matsumoto.xt@hitachi.com>
Reviewed-by: Paul Luse <paul.e.luse@intel.com>

parent 6c820f84

lib/nvme/nvme_pcie.c

+1 −1

Original line number	Diff line number	Diff line
		@@ -1695,7 +1695,7 @@ nvme_pcie_fail_request_bad_vtophys(struct spdk_nvme_qpair *qpair, struct nvme_tr
		*
		* *prp_index will be updated to account for the number of PRP entries used.
		*/
		static int
		static inline int
		nvme_pcie_prp_list_append(struct nvme_tracker tr, uint32_t prp_index, void *virt_addr, size_t len,
		uint32_t page_size)
		{

lib/nvme/nvme_qpair.c

+1 −1

Original line number	Diff line number	Diff line
		@@ -403,7 +403,7 @@ nvme_qpair_abort_queued_reqs(struct spdk_nvme_qpair *qpair, uint32_t dnr)
		}
		}

		static bool
		static inline bool
		nvme_qpair_check_enabled(struct spdk_nvme_qpair *qpair)
		{
		if (!qpair->is_enabled && !qpair->ctrlr->is_resetting) {