For zone block devices, device_area_is_invalid may return an incorrect 
value.

Failure log:
[   19.337657]: device-mapper: table: 254:56: len=836960256 not aligned to
h/w zone size 3244032 of sde
[   19.337665]: device-mapper: core: Cannot calculate initial queue limits
[   19.337667]: device-mapper: ioctl: unable to set up device queue for 
new table.

Actually, the device's zone length is aligned to the zonesize.

Fixes: 5dea271b6d87 ("dm table: pass correct dev area size to 
device_area_is_valid")
Signed-off-by: Li Dong <[email protected]>
---
 drivers/md/dm-table.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/md/dm-table.c b/drivers/md/dm-table.c
index 33b7a1844ed4..0bddae0bee3c 100644
--- a/drivers/md/dm-table.c
+++ b/drivers/md/dm-table.c
@@ -257,7 +257,7 @@ static int device_area_is_invalid(struct dm_target *ti, 
struct dm_dev *dev,
        if (bdev_is_zoned(bdev)) {
                unsigned int zone_sectors = bdev_zone_sectors(bdev);
 
-               if (start & (zone_sectors - 1)) {
+               if (start % zone_sectors) {
                        DMERR("%s: start=%llu not aligned to h/w zone size %u 
of %pg",
                              dm_device_name(ti->table->md),
                              (unsigned long long)start,
@@ -274,7 +274,7 @@ static int device_area_is_invalid(struct dm_target *ti, 
struct dm_dev *dev,
                 * devices do not end up with a smaller zone in the middle of
                 * the sector range.
                 */
-               if (len & (zone_sectors - 1)) {
+               if (len % zone_sectors) {
                        DMERR("%s: len=%llu not aligned to h/w zone size %u of 
%pg",
                              dm_device_name(ti->table->md),
                              (unsigned long long)len,
-- 
2.31.1.windows.1


Reply via email to