Re: [PATCH v9 05/13] nvme: zns: Allow ZNS drives that have non-power_of_2 zone size

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

Subject: Re: [PATCH v9 05/13] nvme: zns: Allow ZNS drives that have non-power_of_2 zone size
From: Bart Van Assche <bvanassche@xxxxxxx>
Date: Wed, 3 Aug 2022 10:22:32 -0700
Cc: pankydev8@xxxxxxxxx, gost.dev@xxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx, linux-nvme@xxxxxxxxxxxxxxxxxxx, linux-block@xxxxxxxxxxxxxxx, dm-devel@xxxxxxxxxx, Luis Chamberlain <mcgrof@xxxxxxxxxx>, jaegeuk@xxxxxxxxxx, matias.bjorling@xxxxxxx
In-reply-to: <20220803094801.177490-6-p.raghav@samsung.com>
References: <20220803094801.177490-1-p.raghav@samsung.com> <CGME20220803094808eucas1p1a6a800161d32fe4242d39e669e5f3527@eucas1p1.samsung.com> <20220803094801.177490-6-p.raghav@samsung.com>
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.11.0

On 8/3/22 02:47, Pankaj Raghav wrote:

-	sector &= ~(ns->zsze - 1);
+	/*
+	 * Round down the sector value to the nearest zone start
+	 */
+	div64_u64_rem(sector, ns->zsze, &remainder);
+	sector -= remainder;


Could bdev_offset_from_zone_start() be used here?

Thanks,

Bart.

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

Follow-Ups:
- Re: [PATCH v9 05/13] nvme: zns: Allow ZNS drives that have non-power_of_2 zone size
  - From: Pankaj Raghav

References:
- [PATCH v9 00/13] support zoned block devices with non-power-of-2 zone sizes
  - From: Pankaj Raghav
- [PATCH v9 05/13] nvme: zns: Allow ZNS drives that have non-power_of_2 zone size
  - From: Pankaj Raghav

Prev by Date: Re: [PATCH v9 03/13] block: allow blk-zoned devices to have non-power-of-2 zone size
Next by Date: Re: [PATCH v9 09/13] dm-zone: use generic helpers to calculate offset from zone start
Previous by thread: [PATCH v9 05/13] nvme: zns: Allow ZNS drives that have non-power_of_2 zone size
Next by thread: Re: [PATCH v9 05/13] nvme: zns: Allow ZNS drives that have non-power_of_2 zone size
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]