From 0190e71eb6fb44e2ea4bb1103c1326bfb66cf951 Mon Sep 17 00:00:00 2001 From: Sven Breuner Date: Wed, 10 Jun 2020 21:30:51 +0300 Subject: [PATCH] module/bdev_compress: Fix use logical block size in vbdev_compress_claim Use logical block size instead of base bdev block size in vbdev_compress_claim. This is relevant to enable the compress bdev to present a logical block size that is different from the block size of the base bdev (e.g. 512B logical on 4KB base bdev). However, this is just an internal fix/preparation and there is currently not yet a way for users to specify a different logical block size, e.g. through an rpc.py parameter. Signed-off-by: Sven Breuner Change-Id: Id7562f0da234ce11242b7ee78837a317ee51c211 Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/2858 Tested-by: SPDK CI Jenkins Community-CI: Mellanox Build Bot Community-CI: Broadcom CI Reviewed-by: Shuhei Matsumoto Reviewed-by: Paul Luse Reviewed-by: Jim Harris --- module/bdev/compress/vbdev_compress.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/module/bdev/compress/vbdev_compress.c b/module/bdev/compress/vbdev_compress.c index 399a695b4..6ac5ffae7 100644 --- a/module/bdev/compress/vbdev_compress.c +++ b/module/bdev/compress/vbdev_compress.c @@ -1595,7 +1595,7 @@ vbdev_compress_claim(struct vbdev_compress *comp_bdev) comp_bdev->comp_bdev.split_on_optimal_io_boundary = true; - comp_bdev->comp_bdev.blocklen = comp_bdev->base_bdev->blocklen; + comp_bdev->comp_bdev.blocklen = comp_bdev->params.logical_block_size; comp_bdev->comp_bdev.blockcnt = comp_bdev->params.vol_size / comp_bdev->comp_bdev.blocklen; assert(comp_bdev->comp_bdev.blockcnt > 0);