Commit 552a26d9 authored by Damiano Cipriani's avatar Damiano Cipriani Committed by Jim Harris
Browse files

spdk_dd: Add check on block size 



If passed as a parameter, block size must be a multiple of input
native block size, so a check to control this requirement has been added

Signed-off-by: default avatarDamiano Cipriani <damiano.cipriani@suse.com>
Change-Id: I38c41afd421083ba0864d5066d9978cb29bdaed6
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/14554


Community-CI: Mellanox Build Bot
Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: default avatarJim Harris <james.r.harris@intel.com>
Reviewed-by: default avatarBen Walker <benjamin.walker@intel.com>
Reviewed-by: default avatarAleksey Marchuk <alexeymar@nvidia.com>
parent 03843f73

app/spdk_dd/spdk_dd.c

+7 −0
Original line number Diff line number Diff line
@@ -884,6 +884,13 @@ dd_run(void *arg1) 
		return;

	}



	if (g_opts.input_bdev && g_opts.io_unit_size % g_job.input.block_size != 0) {

		SPDK_ERRLOG("--bs value must be a multiple of input native block size (%d)\n",

			    g_job.input.block_size);

		dd_exit(-EINVAL);

		return;

	}



	g_job.ios = calloc(g_opts.queue_depth, sizeof(struct dd_io));

	if (g_job.ios == NULL) {

		SPDK_ERRLOG("%s\n", strerror(ENOMEM));