diff options
author | Christoph Hellwig <hch@lst.de> | 2020-09-28 12:03:13 +0200 |
---|---|---|
committer | Christoph Hellwig <hch@lst.de> | 2020-10-07 07:56:18 +0200 |
commit | 13f0b26bbf0a8660d23cad22ed0c4b58b5675dc4 (patch) | |
tree | 8bdfd52872064450b66c71e7016f60276ddc0893 /drivers/nvme | |
parent | f9d5f4579feafa721dba2f350fc064a1852c6f8c (diff) | |
download | linux-13f0b26bbf0a8660d23cad22ed0c4b58b5675dc4.tar.gz linux-13f0b26bbf0a8660d23cad22ed0c4b58b5675dc4.tar.bz2 linux-13f0b26bbf0a8660d23cad22ed0c4b58b5675dc4.zip |
nvme: clean up the check for too large logic block sizes
Use a single statement to set both the capacity and fake block size
instead of two.
Signed-off-by: Christoph Hellwig <hch@lst.de>
Reviewed-by: Keith Busch <kbusch@kernel.org>
Diffstat (limited to 'drivers/nvme')
-rw-r--r-- | drivers/nvme/host/core.c | 13 |
1 files changed, 5 insertions, 8 deletions
diff --git a/drivers/nvme/host/core.c b/drivers/nvme/host/core.c index 82cd03c0ba21..0114fe47de35 100644 --- a/drivers/nvme/host/core.c +++ b/drivers/nvme/host/core.c @@ -2017,8 +2017,12 @@ static void nvme_update_disk_info(struct gendisk *disk, unsigned short bs = 1 << ns->lba_shift; u32 atomic_bs, phys_bs, io_opt = 0; + /* + * The block layer can't support LBA sizes larger than the page size + * yet, so catch this early and don't allow block I/O. + */ if (ns->lba_shift > PAGE_SHIFT) { - /* unsupported block size, set capacity to 0 later */ + capacity = 0; bs = (1 << 9); } @@ -2056,13 +2060,6 @@ static void nvme_update_disk_info(struct gendisk *disk, blk_queue_io_opt(disk->queue, io_opt); /* - * The block layer can't support LBA sizes larger than the page size - * yet, so catch this early and don't allow block I/O. - */ - if (ns->lba_shift > PAGE_SHIFT) - capacity = 0; - - /* * Register a metadata profile for PI, or the plain non-integrity NVMe * metadata masquerading as Type 0 if supported, otherwise reject block * I/O to namespaces with metadata except when the namespace supports |