Commit 14bf912d authored by Ben Walker's avatar Ben Walker Committed by Jim Harris
Browse files

bdevperf: Use new zcopy API for reads 



Change-Id: I71023d56f9023f46ff65291d22b1b8d31595a7b5
Signed-off-by: default avatarBen Walker <benjamin.walker@intel.com>
Reviewed-on: https://review.gerrithub.io/c/spdk/spdk/+/416579


Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: default avatarDarek Stojaczyk <dariusz.stojaczyk@intel.com>
Reviewed-by: default avatarShuhei Matsumoto <shuhei.matsumoto.xt@hitachi.com>
Reviewed-by: default avatarJim Harris <james.r.harris@intel.com>
parent 946ef1dc

test/bdev/bdevperf/bdevperf.c

+18 −4
Original line number Diff line number Diff line
@@ -449,6 +449,17 @@ bdevperf_verify_write_complete(struct spdk_bdev_io *bdev_io, bool success, 
	}

}



static void

bdevperf_zcopy_complete(struct spdk_bdev_io *bdev_io, bool success, void *cb_arg)

{

	if (!success) {

		bdevperf_complete(bdev_io, success, cb_arg);

		return;

	}



	spdk_bdev_zcopy_end(bdev_io, false, bdevperf_complete, cb_arg);

}



static __thread unsigned int seed = 0;



static void
@@ -528,7 +539,6 @@ bdevperf_submit_task(void *arg) 
	struct spdk_bdev_desc	*desc;

	struct spdk_io_channel	*ch;

	spdk_bdev_io_completion_cb cb_fn;

	void			*rbuf;

	int			rc = 0;



	desc = target->bdev_desc;
@@ -558,9 +568,13 @@ bdevperf_submit_task(void *arg) 
						   target->io_size_blocks, bdevperf_complete, task);

		break;

	case SPDK_BDEV_IO_TYPE_READ:

		rbuf = g_zcopy ? NULL : task->buf;

		rc = spdk_bdev_read_blocks(desc, ch, rbuf, task->offset_blocks,

		if (g_zcopy) {

			rc = spdk_bdev_zcopy_start(desc, ch, task->offset_blocks, target->io_size_blocks,

						   true, bdevperf_zcopy_complete, task);

		} else {

			rc = spdk_bdev_read_blocks(desc, ch, task->buf, task->offset_blocks,

						   target->io_size_blocks, bdevperf_complete, task);

		}

		break;

	default:

		assert(false);