mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git
synced 2024-08-26 10:49:33 +00:00
nvme: simplify the API for getting log pages
Merge nvme_get_log and nvme_get_log_ext into a single helper, which takes a plain nsid instead of the nvme_ns pointer. Also add support for the log specific field while we're at it. Signed-off-by: Christoph Hellwig <hch@lst.de> Reviewed-by: Keith Busch <keith.busch@intel.com> Reviewed-by: Sagi Grimberg <sagi@grimberg.me> Reviewed-by: Martin K. Petersen <martin.petersen@oracle.com> Reviewed-by: Hannes Reinecke <hare@suse.com> Reviewed-by: Johannes Thumshirn <jthumshirn@suse.de>
This commit is contained in:
parent
1a37621658
commit
0e98719b0e
3 changed files with 16 additions and 25 deletions
|
@ -2249,21 +2249,16 @@ static int nvme_init_subsystem(struct nvme_ctrl *ctrl, struct nvme_id_ctrl *id)
|
||||||
return ret;
|
return ret;
|
||||||
}
|
}
|
||||||
|
|
||||||
int nvme_get_log_ext(struct nvme_ctrl *ctrl, struct nvme_ns *ns,
|
int nvme_get_log(struct nvme_ctrl *ctrl, u32 nsid, u8 log_page, u8 lsp,
|
||||||
u8 log_page, void *log,
|
void *log, size_t size, u64 offset)
|
||||||
size_t size, u64 offset)
|
|
||||||
{
|
{
|
||||||
struct nvme_command c = { };
|
struct nvme_command c = { };
|
||||||
unsigned long dwlen = size / 4 - 1;
|
unsigned long dwlen = size / 4 - 1;
|
||||||
|
|
||||||
c.get_log_page.opcode = nvme_admin_get_log_page;
|
c.get_log_page.opcode = nvme_admin_get_log_page;
|
||||||
|
c.get_log_page.nsid = cpu_to_le32(nsid);
|
||||||
if (ns)
|
|
||||||
c.get_log_page.nsid = cpu_to_le32(ns->head->ns_id);
|
|
||||||
else
|
|
||||||
c.get_log_page.nsid = cpu_to_le32(NVME_NSID_ALL);
|
|
||||||
|
|
||||||
c.get_log_page.lid = log_page;
|
c.get_log_page.lid = log_page;
|
||||||
|
c.get_log_page.lsp = lsp;
|
||||||
c.get_log_page.numdl = cpu_to_le16(dwlen & ((1 << 16) - 1));
|
c.get_log_page.numdl = cpu_to_le16(dwlen & ((1 << 16) - 1));
|
||||||
c.get_log_page.numdu = cpu_to_le16(dwlen >> 16);
|
c.get_log_page.numdu = cpu_to_le16(dwlen >> 16);
|
||||||
c.get_log_page.lpol = cpu_to_le32(lower_32_bits(offset));
|
c.get_log_page.lpol = cpu_to_le32(lower_32_bits(offset));
|
||||||
|
@ -2272,12 +2267,6 @@ int nvme_get_log_ext(struct nvme_ctrl *ctrl, struct nvme_ns *ns,
|
||||||
return nvme_submit_sync_cmd(ctrl->admin_q, &c, log, size);
|
return nvme_submit_sync_cmd(ctrl->admin_q, &c, log, size);
|
||||||
}
|
}
|
||||||
|
|
||||||
static int nvme_get_log(struct nvme_ctrl *ctrl, u8 log_page, void *log,
|
|
||||||
size_t size)
|
|
||||||
{
|
|
||||||
return nvme_get_log_ext(ctrl, NULL, log_page, log, size, 0);
|
|
||||||
}
|
|
||||||
|
|
||||||
static int nvme_get_effects_log(struct nvme_ctrl *ctrl)
|
static int nvme_get_effects_log(struct nvme_ctrl *ctrl)
|
||||||
{
|
{
|
||||||
int ret;
|
int ret;
|
||||||
|
@ -2288,8 +2277,8 @@ static int nvme_get_effects_log(struct nvme_ctrl *ctrl)
|
||||||
if (!ctrl->effects)
|
if (!ctrl->effects)
|
||||||
return 0;
|
return 0;
|
||||||
|
|
||||||
ret = nvme_get_log(ctrl, NVME_LOG_CMD_EFFECTS, ctrl->effects,
|
ret = nvme_get_log(ctrl, NVME_NSID_ALL, NVME_LOG_CMD_EFFECTS, 0,
|
||||||
sizeof(*ctrl->effects));
|
ctrl->effects, sizeof(*ctrl->effects), 0);
|
||||||
if (ret) {
|
if (ret) {
|
||||||
kfree(ctrl->effects);
|
kfree(ctrl->effects);
|
||||||
ctrl->effects = NULL;
|
ctrl->effects = NULL;
|
||||||
|
@ -3208,7 +3197,8 @@ static void nvme_clear_changed_ns_log(struct nvme_ctrl *ctrl)
|
||||||
* raced with us in reading the log page, which could cause us to miss
|
* raced with us in reading the log page, which could cause us to miss
|
||||||
* updates.
|
* updates.
|
||||||
*/
|
*/
|
||||||
error = nvme_get_log(ctrl, NVME_LOG_CHANGED_NS, log, log_size);
|
error = nvme_get_log(ctrl, NVME_NSID_ALL, NVME_LOG_CHANGED_NS, 0, log,
|
||||||
|
log_size, 0);
|
||||||
if (error)
|
if (error)
|
||||||
dev_warn(ctrl->device,
|
dev_warn(ctrl->device,
|
||||||
"reading changed ns log failed: %d\n", error);
|
"reading changed ns log failed: %d\n", error);
|
||||||
|
@ -3325,9 +3315,9 @@ static void nvme_get_fw_slot_info(struct nvme_ctrl *ctrl)
|
||||||
if (!log)
|
if (!log)
|
||||||
return;
|
return;
|
||||||
|
|
||||||
if (nvme_get_log(ctrl, NVME_LOG_FW_SLOT, log, sizeof(*log)))
|
if (nvme_get_log(ctrl, NVME_NSID_ALL, 0, NVME_LOG_FW_SLOT, log,
|
||||||
dev_warn(ctrl->device,
|
sizeof(*log), 0))
|
||||||
"Get FW SLOT INFO log error\n");
|
dev_warn(ctrl->device, "Get FW SLOT INFO log error\n");
|
||||||
kfree(log);
|
kfree(log);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
|
@ -604,8 +604,9 @@ static int nvme_nvm_get_chk_meta(struct nvm_dev *ndev,
|
||||||
while (left) {
|
while (left) {
|
||||||
len = min_t(unsigned int, left, max_len);
|
len = min_t(unsigned int, left, max_len);
|
||||||
|
|
||||||
ret = nvme_get_log_ext(ctrl, ns, NVME_NVM_LOG_REPORT_CHUNK,
|
ret = nvme_get_log(ctrl, ns->head->ns_id,
|
||||||
dev_meta, len, offset);
|
NVME_NVM_LOG_REPORT_CHUNK, 0, dev_meta, len,
|
||||||
|
offset);
|
||||||
if (ret) {
|
if (ret) {
|
||||||
dev_err(ctrl->device, "Get REPORT CHUNK log error\n");
|
dev_err(ctrl->device, "Get REPORT CHUNK log error\n");
|
||||||
break;
|
break;
|
||||||
|
|
|
@ -443,8 +443,8 @@ int nvme_reset_ctrl_sync(struct nvme_ctrl *ctrl);
|
||||||
int nvme_delete_ctrl(struct nvme_ctrl *ctrl);
|
int nvme_delete_ctrl(struct nvme_ctrl *ctrl);
|
||||||
int nvme_delete_ctrl_sync(struct nvme_ctrl *ctrl);
|
int nvme_delete_ctrl_sync(struct nvme_ctrl *ctrl);
|
||||||
|
|
||||||
int nvme_get_log_ext(struct nvme_ctrl *ctrl, struct nvme_ns *ns,
|
int nvme_get_log(struct nvme_ctrl *ctrl, u32 nsid, u8 log_page, u8 lsp,
|
||||||
u8 log_page, void *log, size_t size, u64 offset);
|
void *log, size_t size, u64 offset);
|
||||||
|
|
||||||
extern const struct attribute_group nvme_ns_id_attr_group;
|
extern const struct attribute_group nvme_ns_id_attr_group;
|
||||||
extern const struct block_device_operations nvme_ns_head_ops;
|
extern const struct block_device_operations nvme_ns_head_ops;
|
||||||
|
|
Loading…
Reference in a new issue