Commit ff2e6bfe authored by Yankun Li's avatar Yankun Li Committed by Konrad Sztyber
Browse files

lib/lvol: cluster size must be a multiple of bs_dev->blocklen 



Change-Id: Ifabd8fc023b6e2476262dbdd68155b88ca77d2c9
Signed-off-by: default avatarYankun Li <yankun@staff.sina.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/25434


Reviewed-by: default avatarGangCao <gang.cao@intel.com>
Community-CI: Broadcom CI <spdk-ci.pdl@broadcom.com>
Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
Community-CI: Community CI Samsung <spdk.community.ci.samsung@gmail.com>
Reviewed-by: default avatarJim Harris <jim.harris@nvidia.com>
Reviewed-by: default avatarKonrad Sztyber <konrad.sztyber@intel.com>
parent 9885e1d2

lib/lvol/lvol.c

+3 −3
Original line number Diff line number Diff line
@@ -703,9 +703,9 @@ spdk_lvs_init(struct spdk_bs_dev *bs_dev, struct spdk_lvs_opts *o, 
		return -EINVAL;

	}



	if (lvs_opts.cluster_sz < bs_dev->blocklen) {

		SPDK_ERRLOG("Cluster size %" PRIu32 " is smaller than blocklen %" PRIu32 "\n",

			    lvs_opts.cluster_sz, bs_dev->blocklen);

	if (lvs_opts.cluster_sz < bs_dev->blocklen || (lvs_opts.cluster_sz % bs_dev->blocklen) != 0) {

		SPDK_ERRLOG("Cluster size %" PRIu32 " is smaller than blocklen %" PRIu32

			    "Or not an integral multiple\n", lvs_opts.cluster_sz, bs_dev->blocklen);

		return -EINVAL;

	}

	total_clusters = bs_dev->blockcnt / (lvs_opts.cluster_sz / bs_dev->blocklen);