From: Christoph Hellwig <hch@lst.de>
To: axboe@kernel.dk
Cc: linux-block@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: [PATCH 3/7] block: simplify set_init_blocksize
Date: Fri, 26 Jun 2020 10:01:54 +0200 [thread overview]
Message-ID: <20200626080158.1998621-4-hch@lst.de> (raw)
In-Reply-To: <20200626080158.1998621-1-hch@lst.de>
The loop to increase the initial block size doesn't really make any
sense, as the AND operation won't match for powers of two if it didn't
for the initial block size.
Signed-off-by: Christoph Hellwig <hch@lst.de>
---
fs/block_dev.c | 6 ------
1 file changed, 6 deletions(-)
diff --git a/fs/block_dev.c b/fs/block_dev.c
index 0e0d43dc27d331..8b7a9c76d33edf 100644
--- a/fs/block_dev.c
+++ b/fs/block_dev.c
@@ -106,13 +106,7 @@ EXPORT_SYMBOL(invalidate_bdev);
static void set_init_blocksize(struct block_device *bdev)
{
unsigned bsize = bdev_logical_block_size(bdev);
- loff_t size = i_size_read(bdev->bd_inode);
- while (bsize < PAGE_SIZE) {
- if (size & bsize)
- break;
- bsize <<= 1;
- }
bdev->bd_block_size = bsize;
bdev->bd_inode->i_blkbits = blksize_bits(bsize);
}
--
2.26.2
next prev parent reply other threads:[~2020-06-26 8:02 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-06-26 8:01 remove not needed fields from struct block_device Christoph Hellwig
2020-06-26 8:01 ` [PATCH 1/7] floppy: use block_size Christoph Hellwig
2020-06-26 15:48 ` Johannes Thumshirn
2020-06-26 15:50 ` Johannes Thumshirn
2020-06-26 8:01 ` [PATCH 2/7] dcssblk: don't set bd_block_size in ->open Christoph Hellwig
2020-06-26 15:54 ` Johannes Thumshirn
2020-06-26 8:01 ` Christoph Hellwig [this message]
2020-06-26 8:01 ` [PATCH 4/7] block: remove the bd_block_size field from struct block_device Christoph Hellwig
2020-07-01 7:29 ` Johannes Thumshirn
2020-06-26 8:01 ` [PATCH 5/7] block: remove the bd_queue " Christoph Hellwig
2020-07-01 7:30 ` Johannes Thumshirn
2020-06-26 8:01 ` [PATCH 6/7] block: remove the unused bd_private " Christoph Hellwig
2020-07-01 7:31 ` Johannes Thumshirn
2020-06-26 8:01 ` [PATCH 7/7] block: remove the all_bdevs list Christoph Hellwig
2020-07-01 7:33 ` Johannes Thumshirn
2020-07-01 7:11 ` remove not needed fields from struct block_device Christoph Hellwig
2020-07-01 15:03 ` Jens Axboe
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=20200626080158.1998621-4-hch@lst.de \
--to=hch@lst.de \
--cc=axboe@kernel.dk \
--cc=linux-block@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.