From: Jan Kara <jack@suse.cz>
To: Yu Kuai <yukuai1@huaweicloud.com>
Cc: jack@suse.cz, hch@lst.de, brauner@kernel.org, axboe@kernel.dk,
linux-fsdevel@vger.kernel.org, linux-block@vger.kernel.org,
yukuai3@huawei.com, yi.zhang@huawei.com, yangerkun@huawei.com
Subject: Re: [RFC v4 linux-next 17/19] dm-vdo: prevent direct access of bd_inode
Date: Mon, 18 Mar 2024 10:19:58 +0100 [thread overview]
Message-ID: <20240318091958.u3yqy2ab7rbqbroq@quack3> (raw)
In-Reply-To: <20240222124555.2049140-18-yukuai1@huaweicloud.com>
On Thu 22-02-24 20:45:53, Yu Kuai wrote:
> From: Yu Kuai <yukuai3@huawei.com>
>
> Now that dm upper layer already statsh the file of opened device in
> 'dm_dev->bdev_file', it's ok to get inode from the file.
>
> Signed-off-by: Yu Kuai <yukuai3@huawei.com>
Given there are like three real uses of ->bdev in dm-vdo, I suspect it
might be better to just replace bdev with bdev_file in struct io_factory
and in struct uds_parameters.
Honza
> ---
> drivers/md/dm-vdo/dedupe.c | 3 ++-
> drivers/md/dm-vdo/dm-vdo-target.c | 5 +++--
> drivers/md/dm-vdo/indexer/config.c | 1 +
> drivers/md/dm-vdo/indexer/config.h | 3 +++
> drivers/md/dm-vdo/indexer/index-layout.c | 6 +++---
> drivers/md/dm-vdo/indexer/index-layout.h | 2 +-
> drivers/md/dm-vdo/indexer/index-session.c | 13 +++++++------
> drivers/md/dm-vdo/indexer/index.c | 4 ++--
> drivers/md/dm-vdo/indexer/index.h | 2 +-
> drivers/md/dm-vdo/indexer/indexer.h | 4 +++-
> drivers/md/dm-vdo/indexer/io-factory.c | 13 ++++++++-----
> drivers/md/dm-vdo/indexer/io-factory.h | 4 ++--
> drivers/md/dm-vdo/indexer/volume.c | 4 ++--
> drivers/md/dm-vdo/indexer/volume.h | 2 +-
> 14 files changed, 39 insertions(+), 27 deletions(-)
>
> diff --git a/drivers/md/dm-vdo/dedupe.c b/drivers/md/dm-vdo/dedupe.c
> index a9b189395592..532294a15174 100644
> --- a/drivers/md/dm-vdo/dedupe.c
> +++ b/drivers/md/dm-vdo/dedupe.c
> @@ -2592,7 +2592,8 @@ static void resume_index(void *context, struct vdo_completion *parent)
> int result;
>
> zones->parameters.bdev = config->owned_device->bdev;
> - result = uds_resume_index_session(zones->index_session, zones->parameters.bdev);
> + zones->parameters.bdev_file = config->owned_device->bdev_file;
> + result = uds_resume_index_session(zones->index_session, zones->parameters.bdev_file);
> if (result != UDS_SUCCESS)
> vdo_log_error_strerror(result, "Error resuming dedupe index");
>
> diff --git a/drivers/md/dm-vdo/dm-vdo-target.c b/drivers/md/dm-vdo/dm-vdo-target.c
> index 89d00be9f075..b2d7f68e70be 100644
> --- a/drivers/md/dm-vdo/dm-vdo-target.c
> +++ b/drivers/md/dm-vdo/dm-vdo-target.c
> @@ -883,7 +883,7 @@ static int parse_device_config(int argc, char **argv, struct dm_target *ti,
> }
>
> if (config->version == 0) {
> - u64 device_size = i_size_read(config->owned_device->bdev->bd_inode);
> + u64 device_size = i_size_read(file_inode(config->owned_device->bdev_file));
>
> config->physical_blocks = device_size / VDO_BLOCK_SIZE;
> }
> @@ -1018,7 +1018,8 @@ static void vdo_status(struct dm_target *ti, status_type_t status_type,
>
> static block_count_t __must_check get_underlying_device_block_count(const struct vdo *vdo)
> {
> - return i_size_read(vdo_get_backing_device(vdo)->bd_inode) / VDO_BLOCK_SIZE;
> + return i_size_read(file_inode(vdo->device_config->owned_device->bdev_file)) /
> + VDO_BLOCK_SIZE;
> }
>
> static int __must_check process_vdo_message_locked(struct vdo *vdo, unsigned int argc,
> diff --git a/drivers/md/dm-vdo/indexer/config.c b/drivers/md/dm-vdo/indexer/config.c
> index 260993ce1944..f1f66e232b54 100644
> --- a/drivers/md/dm-vdo/indexer/config.c
> +++ b/drivers/md/dm-vdo/indexer/config.c
> @@ -347,6 +347,7 @@ int uds_make_configuration(const struct uds_parameters *params,
> config->sparse_sample_rate = (params->sparse ? DEFAULT_SPARSE_SAMPLE_RATE : 0);
> config->nonce = params->nonce;
> config->bdev = params->bdev;
> + config->bdev_file = params->bdev_file;
> config->offset = params->offset;
> config->size = params->size;
>
> diff --git a/drivers/md/dm-vdo/indexer/config.h b/drivers/md/dm-vdo/indexer/config.h
> index fe7958263ed6..688f7450183e 100644
> --- a/drivers/md/dm-vdo/indexer/config.h
> +++ b/drivers/md/dm-vdo/indexer/config.h
> @@ -28,6 +28,9 @@ struct uds_configuration {
> /* Storage device for the index */
> struct block_device *bdev;
>
> + /* Opened device fot the index */
> + struct file *bdev_file;
> +
> /* The maximum allowable size of the index */
> size_t size;
>
> diff --git a/drivers/md/dm-vdo/indexer/index-layout.c b/drivers/md/dm-vdo/indexer/index-layout.c
> index 1453fddaa656..6dd80a432fe5 100644
> --- a/drivers/md/dm-vdo/indexer/index-layout.c
> +++ b/drivers/md/dm-vdo/indexer/index-layout.c
> @@ -1672,7 +1672,7 @@ static int create_layout_factory(struct index_layout *layout,
> size_t writable_size;
> struct io_factory *factory = NULL;
>
> - result = uds_make_io_factory(config->bdev, &factory);
> + result = uds_make_io_factory(config->bdev_file, &factory);
> if (result != UDS_SUCCESS)
> return result;
>
> @@ -1745,9 +1745,9 @@ void vdo_free_index_layout(struct index_layout *layout)
> }
>
> int uds_replace_index_layout_storage(struct index_layout *layout,
> - struct block_device *bdev)
> + struct file *bdev_file)
> {
> - return uds_replace_storage(layout->factory, bdev);
> + return uds_replace_storage(layout->factory, bdev_file);
> }
>
> /* Obtain a dm_bufio_client for the volume region. */
> diff --git a/drivers/md/dm-vdo/indexer/index-layout.h b/drivers/md/dm-vdo/indexer/index-layout.h
> index bd9b90c84a70..9b0c850fe9a7 100644
> --- a/drivers/md/dm-vdo/indexer/index-layout.h
> +++ b/drivers/md/dm-vdo/indexer/index-layout.h
> @@ -24,7 +24,7 @@ int __must_check uds_make_index_layout(struct uds_configuration *config, bool ne
> void vdo_free_index_layout(struct index_layout *layout);
>
> int __must_check uds_replace_index_layout_storage(struct index_layout *layout,
> - struct block_device *bdev);
> + struct file *bdev_file);
>
> int __must_check uds_load_index_state(struct index_layout *layout,
> struct uds_index *index);
> diff --git a/drivers/md/dm-vdo/indexer/index-session.c b/drivers/md/dm-vdo/indexer/index-session.c
> index 1949a2598656..df8f8122a22d 100644
> --- a/drivers/md/dm-vdo/indexer/index-session.c
> +++ b/drivers/md/dm-vdo/indexer/index-session.c
> @@ -460,15 +460,16 @@ int uds_suspend_index_session(struct uds_index_session *session, bool save)
> return uds_status_to_errno(result);
> }
>
> -static int replace_device(struct uds_index_session *session, struct block_device *bdev)
> +static int replace_device(struct uds_index_session *session, struct file *bdev_file)
> {
> int result;
>
> - result = uds_replace_index_storage(session->index, bdev);
> + result = uds_replace_index_storage(session->index, bdev_file);
> if (result != UDS_SUCCESS)
> return result;
>
> - session->parameters.bdev = bdev;
> + session->parameters.bdev = file_bdev(bdev_file);
> + session->parameters.bdev_file = bdev_file;
> return UDS_SUCCESS;
> }
>
> @@ -477,7 +478,7 @@ static int replace_device(struct uds_index_session *session, struct block_device
> * device differs from the current backing store, the index will start using the new backing store.
> */
> int uds_resume_index_session(struct uds_index_session *session,
> - struct block_device *bdev)
> + struct file *bdev_file)
> {
> int result = UDS_SUCCESS;
> bool no_work = false;
> @@ -502,8 +503,8 @@ int uds_resume_index_session(struct uds_index_session *session,
> if (no_work)
> return result;
>
> - if ((session->index != NULL) && (bdev != session->parameters.bdev)) {
> - result = replace_device(session, bdev);
> + if ((session->index != NULL) && (bdev_file != session->parameters.bdev_file)) {
> + result = replace_device(session, bdev_file);
> if (result != UDS_SUCCESS) {
> mutex_lock(&session->request_mutex);
> session->state &= ~IS_FLAG_WAITING;
> diff --git a/drivers/md/dm-vdo/indexer/index.c b/drivers/md/dm-vdo/indexer/index.c
> index bd2405738c50..3600a169ca98 100644
> --- a/drivers/md/dm-vdo/indexer/index.c
> +++ b/drivers/md/dm-vdo/indexer/index.c
> @@ -1334,9 +1334,9 @@ int uds_save_index(struct uds_index *index)
> return result;
> }
>
> -int uds_replace_index_storage(struct uds_index *index, struct block_device *bdev)
> +int uds_replace_index_storage(struct uds_index *index, struct file *bdev_file)
> {
> - return uds_replace_volume_storage(index->volume, index->layout, bdev);
> + return uds_replace_volume_storage(index->volume, index->layout, bdev_file);
> }
>
> /* Accessing statistics should be safe from any thread. */
> diff --git a/drivers/md/dm-vdo/indexer/index.h b/drivers/md/dm-vdo/indexer/index.h
> index 7fbc63db4131..9428ee025cda 100644
> --- a/drivers/md/dm-vdo/indexer/index.h
> +++ b/drivers/md/dm-vdo/indexer/index.h
> @@ -72,7 +72,7 @@ int __must_check uds_save_index(struct uds_index *index);
> void vdo_free_index(struct uds_index *index);
>
> int __must_check uds_replace_index_storage(struct uds_index *index,
> - struct block_device *bdev);
> + struct file *bdev_file);
>
> void uds_get_index_stats(struct uds_index *index, struct uds_index_stats *counters);
>
> diff --git a/drivers/md/dm-vdo/indexer/indexer.h b/drivers/md/dm-vdo/indexer/indexer.h
> index a832a34d9436..5dd2c93f12c2 100644
> --- a/drivers/md/dm-vdo/indexer/indexer.h
> +++ b/drivers/md/dm-vdo/indexer/indexer.h
> @@ -130,6 +130,8 @@ struct uds_volume_record {
> struct uds_parameters {
> /* The block_device used for storage */
> struct block_device *bdev;
> + /* Then opened block_device */
> + struct file *bdev_file;
> /* The maximum allowable size of the index on storage */
> size_t size;
> /* The offset where the index should start */
> @@ -314,7 +316,7 @@ int __must_check uds_suspend_index_session(struct uds_index_session *session, bo
> * start using the new backing store instead.
> */
> int __must_check uds_resume_index_session(struct uds_index_session *session,
> - struct block_device *bdev);
> + struct file *bdev_file);
>
> /* Wait until all outstanding index operations are complete. */
> int __must_check uds_flush_index_session(struct uds_index_session *session);
> diff --git a/drivers/md/dm-vdo/indexer/io-factory.c b/drivers/md/dm-vdo/indexer/io-factory.c
> index 61104d5ccd61..a855c3ac73bc 100644
> --- a/drivers/md/dm-vdo/indexer/io-factory.c
> +++ b/drivers/md/dm-vdo/indexer/io-factory.c
> @@ -23,6 +23,7 @@
> */
> struct io_factory {
> struct block_device *bdev;
> + struct file *bdev_file;
> atomic_t ref_count;
> };
>
> @@ -59,7 +60,7 @@ static void uds_get_io_factory(struct io_factory *factory)
> atomic_inc(&factory->ref_count);
> }
>
> -int uds_make_io_factory(struct block_device *bdev, struct io_factory **factory_ptr)
> +int uds_make_io_factory(struct file *bdev_file, struct io_factory **factory_ptr)
> {
> int result;
> struct io_factory *factory;
> @@ -68,16 +69,18 @@ int uds_make_io_factory(struct block_device *bdev, struct io_factory **factory_p
> if (result != VDO_SUCCESS)
> return result;
>
> - factory->bdev = bdev;
> + factory->bdev = file_bdev(bdev_file);
> + factory->bdev_file = bdev_file;
> atomic_set_release(&factory->ref_count, 1);
>
> *factory_ptr = factory;
> return UDS_SUCCESS;
> }
>
> -int uds_replace_storage(struct io_factory *factory, struct block_device *bdev)
> +int uds_replace_storage(struct io_factory *factory, struct file *bdev_file)
> {
> - factory->bdev = bdev;
> + factory->bdev = file_bdev(bdev_file);
> + factory->bdev_file = bdev_file;
> return UDS_SUCCESS;
> }
>
> @@ -90,7 +93,7 @@ void uds_put_io_factory(struct io_factory *factory)
>
> size_t uds_get_writable_size(struct io_factory *factory)
> {
> - return i_size_read(factory->bdev->bd_inode);
> + return i_size_read(file_inode(factory->bdev_file));
> }
>
> /* Create a struct dm_bufio_client for an index region starting at offset. */
> diff --git a/drivers/md/dm-vdo/indexer/io-factory.h b/drivers/md/dm-vdo/indexer/io-factory.h
> index 60749a9ff756..e5100ab57754 100644
> --- a/drivers/md/dm-vdo/indexer/io-factory.h
> +++ b/drivers/md/dm-vdo/indexer/io-factory.h
> @@ -24,11 +24,11 @@ enum {
> SECTORS_PER_BLOCK = UDS_BLOCK_SIZE >> SECTOR_SHIFT,
> };
>
> -int __must_check uds_make_io_factory(struct block_device *bdev,
> +int __must_check uds_make_io_factory(struct file *bdev_file,
> struct io_factory **factory_ptr);
>
> int __must_check uds_replace_storage(struct io_factory *factory,
> - struct block_device *bdev);
> + struct file *bdev_file);
>
> void uds_put_io_factory(struct io_factory *factory);
>
> diff --git a/drivers/md/dm-vdo/indexer/volume.c b/drivers/md/dm-vdo/indexer/volume.c
> index 8b21ec93f3bc..a292840a83e3 100644
> --- a/drivers/md/dm-vdo/indexer/volume.c
> +++ b/drivers/md/dm-vdo/indexer/volume.c
> @@ -1467,12 +1467,12 @@ int uds_find_volume_chapter_boundaries(struct volume *volume, u64 *lowest_vcn,
>
> int __must_check uds_replace_volume_storage(struct volume *volume,
> struct index_layout *layout,
> - struct block_device *bdev)
> + struct file *bdev_file)
> {
> int result;
> u32 i;
>
> - result = uds_replace_index_layout_storage(layout, bdev);
> + result = uds_replace_index_layout_storage(layout, bdev_file);
> if (result != UDS_SUCCESS)
> return result;
>
> diff --git a/drivers/md/dm-vdo/indexer/volume.h b/drivers/md/dm-vdo/indexer/volume.h
> index 7fdd44464db2..5861654d837e 100644
> --- a/drivers/md/dm-vdo/indexer/volume.h
> +++ b/drivers/md/dm-vdo/indexer/volume.h
> @@ -131,7 +131,7 @@ void vdo_free_volume(struct volume *volume);
>
> int __must_check uds_replace_volume_storage(struct volume *volume,
> struct index_layout *layout,
> - struct block_device *bdev);
> + struct file *bdev_file);
>
> int __must_check uds_find_volume_chapter_boundaries(struct volume *volume,
> u64 *lowest_vcn, u64 *highest_vcn,
> --
> 2.39.2
>
--
Jan Kara <jack@suse.com>
SUSE Labs, CR
next prev parent reply other threads:[~2024-03-18 9:20 UTC|newest]
Thread overview: 97+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-02-22 12:45 [RFC v4 linux-next 00/19] fs & block: remove bdev->bd_inode Yu Kuai
2024-02-22 12:45 ` [RFC v4 linux-next 01/19] block: move two helpers into bdev.c Yu Kuai
2024-03-15 14:31 ` Jan Kara
2024-03-17 21:19 ` Christoph Hellwig
2024-02-22 12:45 ` [RFC v4 linux-next 02/19] block: remove sync_blockdev_nowait() Yu Kuai
2024-03-15 14:34 ` Jan Kara
2024-03-17 21:19 ` Christoph Hellwig
2024-02-22 12:45 ` [RFC v4 linux-next 03/19] block: remove sync_blockdev_range() Yu Kuai
2024-03-15 14:37 ` Jan Kara
2024-03-17 21:21 ` Christoph Hellwig
2024-02-22 12:45 ` [RFC v4 linux-next 04/19] block: prevent direct access of bd_inode Yu Kuai
2024-03-15 14:44 ` Jan Kara
2024-03-17 21:23 ` Christoph Hellwig
2024-03-22 5:44 ` Al Viro
2024-02-22 12:45 ` [RFC v4 linux-next 05/19] bcachefs: remove dead function bdev_sectors() Yu Kuai
2024-03-15 14:42 ` Jan Kara
2024-03-17 21:23 ` Christoph Hellwig
2024-02-22 12:45 ` [RFC v4 linux-next 06/19] cramfs: prevent direct access of bd_inode Yu Kuai
2024-03-15 14:44 ` Jan Kara
2024-03-17 21:23 ` Christoph Hellwig
2024-02-22 12:45 ` [RFC v4 linux-next 07/19] erofs: " Yu Kuai
2024-03-15 14:45 ` Jan Kara
2024-03-17 21:24 ` Christoph Hellwig
2024-03-18 2:39 ` Gao Xiang
2024-02-22 12:45 ` [RFC v4 linux-next 08/19] nilfs2: " Yu Kuai
2024-03-15 14:49 ` Jan Kara
2024-03-17 21:24 ` Christoph Hellwig
2024-02-22 12:45 ` [RFC v4 linux-next 09/19] gfs2: " Yu Kuai
2024-03-15 14:54 ` Jan Kara
2024-03-17 21:24 ` Christoph Hellwig
2024-02-22 12:45 ` [RFC v4 linux-next 10/19] s390/dasd: use bdev api in dasd_format() Yu Kuai
2024-03-15 14:55 ` Jan Kara
2024-03-17 21:25 ` Christoph Hellwig
2024-02-22 12:45 ` [RFC v4 linux-next 11/19] btrfs: prevent direct access of bd_inode Yu Kuai
2024-03-15 15:09 ` Jan Kara
2024-03-17 21:25 ` Christoph Hellwig
2024-02-22 12:45 ` [RFC v4 linux-next 12/19] ext4: remove block_device_ejected() Yu Kuai
2024-02-22 12:45 ` [RFC v4 linux-next 13/19] ext4: prevent direct access of bd_inode Yu Kuai
2024-03-15 14:58 ` Jan Kara
2024-03-17 21:25 ` Christoph Hellwig
2024-02-22 12:45 ` [RFC v4 linux-next 14/19] jbd2: " Yu Kuai
2024-03-15 15:06 ` Jan Kara
2024-03-17 21:26 ` Christoph Hellwig
2024-03-18 1:10 ` Yu Kuai
2024-02-22 12:45 ` [RFC v4 linux-next 15/19] bcache: " Yu Kuai
2024-03-15 15:11 ` Jan Kara
2024-03-17 21:34 ` Christoph Hellwig
2024-02-22 12:45 ` [RFC v4 linux-next 16/19] block2mtd: " Yu Kuai
2024-03-15 15:12 ` Jan Kara
2024-03-17 21:36 ` Christoph Hellwig
2024-02-22 12:45 ` [RFC v4 linux-next 17/19] dm-vdo: " Yu Kuai
2024-02-28 13:41 ` Christoph Hellwig
2024-03-18 9:11 ` Jan Kara
2024-03-18 9:19 ` Jan Kara [this message]
2024-03-18 13:38 ` Yu Kuai
2024-03-19 2:00 ` Matthew Sakai
2024-02-22 12:45 ` [RFC v4 linux-next 18/19] scsi: factor out a helper bdev_read_folio() from scsi_bios_ptable() Yu Kuai
2024-03-17 21:36 ` Christoph Hellwig
2024-03-18 1:12 ` Yu Kuai
2024-03-18 9:22 ` Jan Kara
2024-02-22 12:45 ` [RFC v4 linux-next 19/19] fs & block: remove bdev->bd_inode Yu Kuai
2024-03-17 21:38 ` Christoph Hellwig
2024-03-18 1:26 ` Yu Kuai
2024-03-18 1:32 ` Christoph Hellwig
2024-03-18 1:51 ` Yu Kuai
2024-03-18 7:19 ` Yu Kuai
2024-03-18 10:07 ` Christian Brauner
2024-03-18 10:29 ` Christian Brauner
2024-03-18 10:46 ` Christian Brauner
2024-03-18 11:57 ` Yu Kuai
2024-03-18 23:35 ` Christoph Hellwig
2024-03-18 23:22 ` Christoph Hellwig
2024-03-19 8:26 ` Yu Kuai
2024-03-21 11:27 ` Jan Kara
2024-03-21 12:15 ` Yu Kuai
2024-03-22 6:37 ` Al Viro
2024-03-22 6:39 ` Al Viro
2024-03-22 6:52 ` Yu Kuai
2024-03-22 12:57 ` Jan Kara
2024-03-22 13:57 ` Christian Brauner
2024-03-22 15:43 ` Al Viro
2024-03-22 16:16 ` Al Viro
2024-03-22 6:33 ` Al Viro
2024-03-22 7:09 ` Yu Kuai
2024-03-22 16:01 ` Al Viro
2024-03-22 13:10 ` Jan Kara
2024-03-22 14:57 ` Al Viro
2024-03-25 1:06 ` Christoph Hellwig
2024-02-28 13:42 ` [RFC v4 linux-next 00/19] " Christoph Hellwig
2024-03-15 12:08 ` Yu Kuai
2024-03-15 13:54 ` Christian Brauner
2024-03-16 2:49 ` Yu Kuai
2024-03-18 9:39 ` Christian Brauner
2024-03-19 1:18 ` Yu Kuai
2024-03-19 1:43 ` Yu Kuai
2024-03-19 2:13 ` Matthew Sakai
2024-03-19 2:27 ` Yu Kuai
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20240318091958.u3yqy2ab7rbqbroq@quack3 \
--to=jack@suse.cz \
--cc=axboe@kernel.dk \
--cc=brauner@kernel.org \
--cc=hch@lst.de \
--cc=linux-block@vger.kernel.org \
--cc=linux-fsdevel@vger.kernel.org \
--cc=yangerkun@huawei.com \
--cc=yi.zhang@huawei.com \
--cc=yukuai1@huaweicloud.com \
--cc=yukuai3@huawei.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox