Commit 5f146372 authored by Dariusz Stojaczyk's avatar Dariusz Stojaczyk Committed by Jim Harris
Browse files

nvme/pcie: merge physically contiguous SGEs 



There's no need to split a bufffer if it's physically
contiguous. We can now merge buffers that would be
previously split by the nvme_pcie driver and also
separate SGEs provided by the user that happen to be
physically contiguous.

Change-Id: I9c9de31d52a9dc9e384806555cb94609aff0ccf3
Signed-off-by: default avatarDariusz Stojaczyk <dariuszx.stojaczyk@intel.com>
Reviewed-on: https://review.gerrithub.io/417061


Tested-by: default avatarSPDK Automated Test System <sys_sgsw@intel.com>
Reviewed-by: default avatarShuhei Matsumoto <shuhei.matsumoto.xt@hitachi.com>
Reviewed-by: default avatarDaniel Verkamp <daniel.verkamp@intel.com>
parent 3320c06b

lib/nvme/nvme_pcie.c

+7 −0
Original line number Diff line number Diff line
@@ -1818,6 +1818,13 @@ nvme_pcie_qpair_build_hw_sgl_request(struct spdk_nvme_qpair *qpair, struct nvme_ 
			remaining_user_sge_len -= length;

			virt_addr += length;



			if (nseg > 0 && phys_addr ==

			    (*(sgl - 1)).address + (*(sgl - 1)).unkeyed.length) {

				/* extend previous entry */

				(*(sgl - 1)).unkeyed.length += length;

				continue;

			}



			sgl->unkeyed.type = SPDK_NVME_SGL_TYPE_DATA_BLOCK;

			sgl->unkeyed.length = length;

			sgl->address = phys_addr;