Re: [PATCH 11/29] btrfs: use bdev_nr_sectors instead of open coding it

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Subject: Re: [PATCH 11/29] btrfs: use bdev_nr_sectors instead of open coding it
From: Keith Busch <kbusch@xxxxxxxxxx>
Date: Wed, 13 Oct 2021 19:44:38 -0700
Cc: Dave Kleikamp <shaggy@xxxxxxxxxx>, jfs-discussion@xxxxxxxxxxxxxxxxxxxxx, Mike Snitzer <snitzer@xxxxxxxxxx>, linux-nvme@xxxxxxxxxxxxxxxxxxx, Konstantin Komarov <almaz.alexandrovich@xxxxxxxxxxxxxxxxxxxx>, Song Liu <song@xxxxxxxxxx>, dm-devel@xxxxxxxxxx, target-devel@xxxxxxxxxxxxxxx, linux-mtd@xxxxxxxxxxxxxxxxxxx, reiserfs-devel@xxxxxxxxxxxxxxx, drbd-dev@xxxxxxxxxxxxxxxx, linux-nilfs@xxxxxxxxxxxxxxx, linux-scsi@xxxxxxxxxxxxxxx, OGAWA Hirofumi <hirofumi@xxxxxxxxxxxxxxxxxx>, linux-ext4@xxxxxxxxxxxxxxx, Kees Cook <keescook@xxxxxxxxxxxx>, Josef Bacik <josef@xxxxxxxxxxxxxx>, Coly Li <colyli@xxxxxxx>, linux-raid@xxxxxxxxxxxxxxx, linux-bcache@xxxxxxxxxxxxxxx, David Sterba <dsterba@xxxxxxxx>, Ryusuke Konishi <konishi.ryusuke@xxxxxxxxx>, Anton Altaparmakov <anton@xxxxxxxxxx>, Jens Axboe <axboe@xxxxxxxxx>, linux-block@xxxxxxxxxxxxxxx, linux-nfs@xxxxxxxxxxxxxxx, "Theodore Ts'o" <tytso@xxxxxxx>, linux-ntfs-dev@xxxxxxxxxxxxxxxxxxxxx, Jan Kara <jack@xxxxxxxx>, linux-fsdevel@xxxxxxxxxxxxxxx, Phillip Lougher <phillip@xxxxxxxxxxxxxxx>, ntfs3@xxxxxxxxxxxxxxx, linux-btrfs@xxxxxxxxxxxxxxx
In-reply-to: <20211013051042.1065752-12-hch@lst.de>
References: <20211013051042.1065752-1-hch@lst.de> <20211013051042.1065752-12-hch@lst.de>

On Wed, Oct 13, 2021 at 07:10:24AM +0200, Christoph Hellwig wrote:
> Use the proper helper to read the block device size.

Just IMO, this patch looks like it wants a new bdev_nr_bytes() helper
instead of using the double shifting sectors back to bytes.

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

References:
- don't use ->bd_inode to access the block device size
  - From: Christoph Hellwig
- [PATCH 11/29] btrfs: use bdev_nr_sectors instead of open coding it
  - From: Christoph Hellwig

Prev by Date: Re: [PATCH 07/29] target/iblock: use bdev_nr_sectors instead of open coding it
Next by Date: Re: [PATCH 04/29] md: use bdev_nr_sectors instead of open coding it
Previous by thread: Re: [PATCH 11/29] btrfs: use bdev_nr_sectors instead of open coding it
Next by thread: [PATCH 12/29] cramfs: use bdev_nr_sectors instead of open coding it
Index(es):
- Date
- Thread

[Index of Archives] [DM Crypt] [Fedora Desktop] [ATA RAID] [Fedora Marketing] [Fedora Packaging] [Fedora SELinux] [Yosemite Discussion] [KDE Users] [Fedora Docs]