[dm-devel] [PATCH 10/29] dm: Convert to bdev_open_by_dev()

[Jan Kara]

Convert device mapper to use bdev_open_by_dev() and pass the handle
around.

CC: Alasdair Kergon <a...@redhat.com>
CC: Mike Snitzer <snit...@kernel.org>
CC: dm-devel@redhat.com
Acked-by: Christoph Hellwig <h...@lst.de>
Signed-off-by: Jan Kara <j...@suse.cz>
---
 drivers/md/dm.c               | 20 +++++++++++---------
 include/linux/device-mapper.h |  1 +
 2 files changed, 12 insertions(+), 9 deletions(-)

diff --git a/drivers/md/dm.c b/drivers/md/dm.c
index f0f118ab20fa..efde77d34f15 100644
--- a/drivers/md/dm.c
+++ b/drivers/md/dm.c
@@ -742,7 +742,7 @@ static struct table_device *open_table_device(struct 
mapped_device *md,
                dev_t dev, blk_mode_t mode)
 {
        struct table_device *td;
-       struct block_device *bdev;
+       struct bdev_handle *bdev_handle;
        u64 part_off;
        int r;
 
@@ -751,9 +751,9 @@ static struct table_device *open_table_device(struct 
mapped_device *md,
                return ERR_PTR(-ENOMEM);
        refcount_set(&td->count, 1);
 
-       bdev = blkdev_get_by_dev(dev, mode, _dm_claim_ptr, NULL);
-       if (IS_ERR(bdev)) {
-               r = PTR_ERR(bdev);
+       bdev_handle = bdev_open_by_dev(dev, mode, _dm_claim_ptr, NULL);
+       if (IS_ERR(bdev_handle)) {
+               r = PTR_ERR(bdev_handle);
                goto out_free_td;
        }
 
@@ -763,20 +763,22 @@ static struct table_device *open_table_device(struct 
mapped_device *md,
         * called.
         */
        if (md->disk->slave_dir) {
-               r = bd_link_disk_holder(bdev, md->disk);
+               r = bd_link_disk_holder(bdev_handle->bdev, md->disk);
                if (r)
                        goto out_blkdev_put;
        }
 
        td->dm_dev.mode = mode;
-       td->dm_dev.bdev = bdev;
-       td->dm_dev.dax_dev = fs_dax_get_by_bdev(bdev, &part_off, NULL, NULL);
+       td->dm_dev.bdev = bdev_handle->bdev;
+       td->dm_dev.bdev_handle = bdev_handle;
+       td->dm_dev.dax_dev = fs_dax_get_by_bdev(bdev_handle->bdev, &part_off,
+                                               NULL, NULL);
        format_dev_t(td->dm_dev.name, dev);
        list_add(&td->list, &md->table_devices);
        return td;
 
 out_blkdev_put:
-       blkdev_put(bdev, _dm_claim_ptr);
+       bdev_release(bdev_handle);
 out_free_td:
        kfree(td);
        return ERR_PTR(r);
@@ -789,7 +791,7 @@ static void close_table_device(struct table_device *td, 
struct mapped_device *md
 {
        if (md->disk->slave_dir)
                bd_unlink_disk_holder(td->dm_dev.bdev, md->disk);
-       blkdev_put(td->dm_dev.bdev, _dm_claim_ptr);
+       bdev_release(td->dm_dev.bdev_handle);
        put_dax(td->dm_dev.dax_dev);
        list_del(&td->list);
        kfree(td);
diff --git a/include/linux/device-mapper.h b/include/linux/device-mapper.h
index 69d0435c7ebb..772ab4d74d94 100644
--- a/include/linux/device-mapper.h
+++ b/include/linux/device-mapper.h
@@ -165,6 +165,7 @@ void dm_error(const char *message);
 
 struct dm_dev {
        struct block_device *bdev;
+       struct bdev_handle *bdev_handle;
        struct dax_device *dax_dev;
        blk_mode_t mode;
        char name[16];
-- 
2.35.3

--
dm-devel mailing list
dm-devel@redhat.com
https://listman.redhat.com/mailman/listinfo/dm-devel