loop: Use blk_validate_block_size() to validate block size
commit af3c570fb0df422b4906ebd11c1bf363d89961d5 upstream. Remove loop_validate_block_size() and use the block layer helper to validate block size. Signed-off-by: Xie Yongji <xieyongji@bytedance.com> Link: https://lore.kernel.org/r/20211026144015.188-4-xieyongji@bytedance.com Signed-off-by: Jens Axboe <axboe@kernel.dk> Cc: Tadeusz Struk <tadeusz.struk@linaro.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:

committed by
Greg Kroah-Hartman

parent
79ff56c613
commit
04e46514fe
@@ -228,19 +228,6 @@ static void __loop_update_dio(struct loop_device *lo, bool dio)
|
|||||||
blk_mq_unfreeze_queue(lo->lo_queue);
|
blk_mq_unfreeze_queue(lo->lo_queue);
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
|
||||||
* loop_validate_block_size() - validates the passed in block size
|
|
||||||
* @bsize: size to validate
|
|
||||||
*/
|
|
||||||
static int
|
|
||||||
loop_validate_block_size(unsigned short bsize)
|
|
||||||
{
|
|
||||||
if (bsize < 512 || bsize > PAGE_SIZE || !is_power_of_2(bsize))
|
|
||||||
return -EINVAL;
|
|
||||||
|
|
||||||
return 0;
|
|
||||||
}
|
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* loop_set_size() - sets device size and notifies userspace
|
* loop_set_size() - sets device size and notifies userspace
|
||||||
* @lo: struct loop_device to set the size for
|
* @lo: struct loop_device to set the size for
|
||||||
@@ -1121,7 +1108,7 @@ static int loop_configure(struct loop_device *lo, fmode_t mode,
|
|||||||
}
|
}
|
||||||
|
|
||||||
if (config->block_size) {
|
if (config->block_size) {
|
||||||
error = loop_validate_block_size(config->block_size);
|
error = blk_validate_block_size(config->block_size);
|
||||||
if (error)
|
if (error)
|
||||||
goto out_unlock;
|
goto out_unlock;
|
||||||
}
|
}
|
||||||
@@ -1617,7 +1604,7 @@ static int loop_set_block_size(struct loop_device *lo, unsigned long arg)
|
|||||||
if (lo->lo_state != Lo_bound)
|
if (lo->lo_state != Lo_bound)
|
||||||
return -ENXIO;
|
return -ENXIO;
|
||||||
|
|
||||||
err = loop_validate_block_size(arg);
|
err = blk_validate_block_size(arg);
|
||||||
if (err)
|
if (err)
|
||||||
return err;
|
return err;
|
||||||
|
|
||||||
|
Reference in New Issue
Block a user