Skip to content

Commit c4cfe8c

Browse files
calebsanderkeithbusch
authored andcommitted
nvmet: report NPDGL and NPDAL
A block device with a very large discard_granularity queue limit may not be able to report it in the 16-bit NPDG and NPDA fields in the Identify Namespace data structure. For this reason, version 2.1 of the NVMe specs added 32-bit fields NPDGL and NPDAL to the NVM Command Set Specific Identify Namespace structure. So report the discard_granularity there too and set OPTPERF to 11b to indicate those fields are supported. Signed-off-by: Caleb Sander Mateos <csander@purestorage.com> Reviewed-by: Christoph Hellwig <hch@lst.de> Signed-off-by: Keith Busch <kbusch@kernel.org>
1 parent e0d56e7 commit c4cfe8c

3 files changed

Lines changed: 19 additions & 4 deletions

File tree

drivers/nvme/target/admin-cmd.c

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1057,6 +1057,8 @@ static void nvme_execute_identify_ns_nvm(struct nvmet_req *req)
10571057
status = NVME_SC_INTERNAL;
10581058
goto out;
10591059
}
1060+
if (req->ns->bdev)
1061+
nvmet_bdev_set_nvm_limits(req->ns->bdev, id);
10601062
status = nvmet_copy_to_sgl(req, 0, id, sizeof(*id));
10611063
kfree(id);
10621064
out:

drivers/nvme/target/io-cmd-bdev.c

Lines changed: 15 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -30,11 +30,11 @@ void nvmet_bdev_set_limits(struct block_device *bdev, struct nvme_id_ns *id)
3030
id->nacwu = lpp0b;
3131

3232
/*
33-
* OPTPERF = 01b indicates that the fields NPWG, NPWA, NPDG, NPDA, and
34-
* NOWS are defined for this namespace and should be used by
35-
* the host for I/O optimization.
33+
* OPTPERF = 11b indicates that the fields NPWG, NPWA, NPDG, NPDA,
34+
* NPDGL, NPDAL, and NOWS are defined for this namespace and should be
35+
* used by the host for I/O optimization.
3636
*/
37-
id->nsfeat |= 0x1 << NVME_NS_FEAT_OPTPERF_SHIFT;
37+
id->nsfeat |= 0x3 << NVME_NS_FEAT_OPTPERF_SHIFT;
3838
/* NPWG = Namespace Preferred Write Granularity. 0's based */
3939
id->npwg = to0based(bdev_io_min(bdev) / bdev_logical_block_size(bdev));
4040
/* NPWA = Namespace Preferred Write Alignment. 0's based */
@@ -52,6 +52,17 @@ void nvmet_bdev_set_limits(struct block_device *bdev, struct nvme_id_ns *id)
5252
id->dlfeat = (1 << 3) | 0x1;
5353
}
5454

55+
void nvmet_bdev_set_nvm_limits(struct block_device *bdev,
56+
struct nvme_id_ns_nvm *id)
57+
{
58+
/*
59+
* NPDGL = Namespace Preferred Deallocate Granularity Large
60+
* NPDAL = Namespace Preferred Deallocate Alignment Large
61+
*/
62+
id->npdgl = id->npdal = cpu_to_le32(bdev_discard_granularity(bdev) /
63+
bdev_logical_block_size(bdev));
64+
}
65+
5566
void nvmet_bdev_ns_disable(struct nvmet_ns *ns)
5667
{
5768
if (ns->bdev_file) {

drivers/nvme/target/nvmet.h

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -549,6 +549,8 @@ void nvmet_stop_keep_alive_timer(struct nvmet_ctrl *ctrl);
549549
u16 nvmet_parse_connect_cmd(struct nvmet_req *req);
550550
u32 nvmet_connect_cmd_data_len(struct nvmet_req *req);
551551
void nvmet_bdev_set_limits(struct block_device *bdev, struct nvme_id_ns *id);
552+
void nvmet_bdev_set_nvm_limits(struct block_device *bdev,
553+
struct nvme_id_ns_nvm *id);
552554
u16 nvmet_bdev_parse_io_cmd(struct nvmet_req *req);
553555
u16 nvmet_file_parse_io_cmd(struct nvmet_req *req);
554556
u16 nvmet_bdev_zns_parse_io_cmd(struct nvmet_req *req);

0 commit comments

Comments
 (0)