renegade-project/linux-next
2
0
Fork 0
mirror of https://github.com/edk2-porting/linux-next.git synced 2025-01-11 07:04:04 +08:00
Code

nvme: avoid an Identify Controller command for each namespace scan

Browse Source 
The namespace lists are 0-terminated, so we don't really need the NN value
execept for the legacy sequential scan.

Signed-off-by: Christoph Hellwig <hch@lst.de>
Reviewed-by: Keith Busch <kbusch@kernel.org>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
This commit is contained in:
Christoph Hellwig 2020-04-04 10:31:35 +02:00 committed by Jens Axboe
parent 4450ba3bbb
commit 4005f28d25
1 changed files with 18 additions and 22 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

40
drivers/nvme/host/core.c
View File

@ -3740,12 +3740,11 @@ static void nvme_remove_invalid_namespaces(struct nvme_ctrl *ctrl,
} }
static int nvme_scan_ns_list(struct nvme_ctrl *ctrl, unsigned nn) static int nvme_scan_ns_list(struct nvme_ctrl *ctrl)
{ {
__le32 *ns_list; __le32 *ns_list;
unsigned i, j, nsid, prev = 0; u32 prev = 0;
unsigned num_lists = DIV_ROUND_UP_ULL((u64)nn, 1024); int ret = 0, i;
int ret = 0;
if (nvme_ctrl_limited_cns(ctrl)) if (nvme_ctrl_limited_cns(ctrl))
return -EOPNOTSUPP; return -EOPNOTSUPP;
@ -3754,22 +3753,20 @@ static int nvme_scan_ns_list(struct nvme_ctrl *ctrl, unsigned nn)
if (!ns_list) if (!ns_list)
return -ENOMEM; return -ENOMEM;
for (i = 0; i < num_lists; i++) { for (;;) {
ret = nvme_identify_ns_list(ctrl, prev, ns_list); ret = nvme_identify_ns_list(ctrl, prev, ns_list);
if (ret) if (ret)
goto free; goto free;
for (j = 0; j < min(nn, 1024U); j++) { for (i = 0; i < 1024; i++) {
nsid = le32_to_cpu(ns_list[j]); u32 nsid = le32_to_cpu(ns_list[i]);
if (!nsid)
if (!nsid) /* end of the list? */
goto out; goto out;
nvme_validate_ns(ctrl, nsid); nvme_validate_ns(ctrl, nsid);
while (++prev < nsid) while (++prev < nsid)
nvme_ns_remove_by_nsid(ctrl, prev); nvme_ns_remove_by_nsid(ctrl, prev);
} }
nn -= j;
} }
out: out:
nvme_remove_invalid_namespaces(ctrl, prev); nvme_remove_invalid_namespaces(ctrl, prev);
@ -3778,9 +3775,15 @@ static int nvme_scan_ns_list(struct nvme_ctrl *ctrl, unsigned nn)
return ret; return ret;
} }
static void nvme_scan_ns_sequential(struct nvme_ctrl *ctrl, unsigned nn) static void nvme_scan_ns_sequential(struct nvme_ctrl *ctrl)
{ {
unsigned i; struct nvme_id_ctrl *id;
u32 nn, i;
if (nvme_identify_ctrl(ctrl, &id))
return;
nn = le32_to_cpu(id->nn);
kfree(id);
for (i = 1; i <= nn; i++) for (i = 1; i <= nn; i++)
nvme_validate_ns(ctrl, i); nvme_validate_ns(ctrl, i);
@ -3817,8 +3820,6 @@ static void nvme_scan_work(struct work_struct *work)
{ {
struct nvme_ctrl *ctrl = struct nvme_ctrl *ctrl =
container_of(work, struct nvme_ctrl, scan_work); container_of(work, struct nvme_ctrl, scan_work);
struct nvme_id_ctrl *id;
unsigned nn;
/* No tagset on a live ctrl means IO queues could not created */ /* No tagset on a live ctrl means IO queues could not created */
if (ctrl->state != NVME_CTRL_LIVE || !ctrl->tagset) if (ctrl->state != NVME_CTRL_LIVE || !ctrl->tagset)
@ -3829,14 +3830,9 @@ static void nvme_scan_work(struct work_struct *work)
nvme_clear_changed_ns_log(ctrl); nvme_clear_changed_ns_log(ctrl);
} }
if (nvme_identify_ctrl(ctrl, &id))
return;
nn = le32_to_cpu(id->nn);
kfree(id);
mutex_lock(&ctrl->scan_lock); mutex_lock(&ctrl->scan_lock);
if (nvme_scan_ns_list(ctrl, nn) != 0) if (nvme_scan_ns_list(ctrl) != 0)
nvme_scan_ns_sequential(ctrl, nn); nvme_scan_ns_sequential(ctrl);
mutex_unlock(&ctrl->scan_lock); mutex_unlock(&ctrl->scan_lock);
down_write(&ctrl->namespaces_rwsem); down_write(&ctrl->namespaces_rwsem);