Skip to content

Commit e666a1f

Browse files
igawigaw
committed
src: drop result argument from submit API
The command submit API doesn't need to return explicitly the return value, it's already in struct nvme_passthru_cmd. Signed-off-by: Daniel Wagner <wagi@kernel.org>
1 parent bc68c2b commit e666a1f

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

53 files changed

+803
-925
lines changed

‎libnvme/examples/mi-mctp-csi-test.c‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -91,7 +91,7 @@ int do_get_log_page(nvme_mi_ep_t ep, int argc, char **argv)
9191

9292
nvme_init_get_log(&cmd, NVME_NSID_NONE, lid, NVME_CSI_NVM,
9393
buf, sizeof(buf));
94-
rc = nvme_get_log(hdl, &cmd, false, NVME_LOG_PAGE_PDU_SIZE, NULL);
94+
rc = nvme_get_log(hdl, &cmd, false, NVME_LOG_PAGE_PDU_SIZE);
9595
if (rc) {
9696
warn("can't perform Get Log page command");
9797
return -1;

‎libnvme/examples/mi-mctp.c‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -216,7 +216,7 @@ int do_identify(nvme_mi_ep_t ep, int argc, char **argv)
216216
if (partial)
217217
cmd.data_len = offsetof(struct nvme_id_ctrl, rab);
218218

219-
rc = nvme_submit_admin_passthru(hdl, &cmd, NULL);
219+
rc = nvme_submit_admin_passthru(hdl, &cmd);
220220
if (rc) {
221221
warn("can't perform Admin Identify command");
222222
return -1;
@@ -399,7 +399,7 @@ int do_get_log_page(nvme_mi_ep_t ep, int argc, char **argv)
399399

400400
nvme_init_get_log(&cmd, NVME_NSID_NONE, lid, NVME_CSI_NVM,
401401
buf, sizeof(buf));
402-
rc = nvme_get_log(hdl, &cmd, true, NVME_LOG_PAGE_PDU_SIZE, NULL);
402+
rc = nvme_get_log(hdl, &cmd, true, NVME_LOG_PAGE_PDU_SIZE);
403403
if (rc) {
404404
warn("can't perform Get Log page command");
405405
return -1;
@@ -566,7 +566,7 @@ int do_security_info(nvme_mi_ep_t ep, int argc, char **argv)
566566
}
567567

568568
nvme_init_security_receive(&cmd, 0, 0, 0, 0, 0, data, data_len);
569-
rc = nvme_submit_admin_passthru(hdl, &cmd, NULL);
569+
rc = nvme_submit_admin_passthru(hdl, &cmd);
570570
if (rc) {
571571
warnx("can't perform Security Receive command: rc %d", rc);
572572
return -1;

‎libnvme/libnvme/nvme.i‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -824,7 +824,7 @@ struct nvme_ns {
824824

825825
Py_BEGIN_ALLOW_THREADS /* Release Python GIL */
826826
nvme_init_get_log_supported_log_pages(&cmd, NVME_CSI_NVM, &log);
827-
ret = nvme_get_log(nvme_ctrl_get_transport_handle($self), &cmd, rae, NVME_LOG_PAGE_PDU_SIZE, NULL);
827+
ret = nvme_get_log(nvme_ctrl_get_transport_handle($self), &cmd, rae, NVME_LOG_PAGE_PDU_SIZE);
828828
Py_END_ALLOW_THREADS /* Reacquire Python GIL */
829829

830830
if (ret) {

‎libnvme/src/nvme/fabrics.c‎

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1134,7 +1134,7 @@ static int nvme_discovery_log(const struct nvme_get_discovery_args *args,
11341134
nvme_msg(ctx, LOG_DEBUG, "%s: get header (try %d/%d)\n",
11351135
name, retries, args->max_retries);
11361136
nvme_init_get_log_discovery(&cmd, 0, log, DISCOVERY_HEADER_LEN);
1137-
err = nvme_get_log(hdl, &cmd, false, DISCOVERY_HEADER_LEN, NULL);
1137+
err = nvme_get_log(hdl, &cmd, false, DISCOVERY_HEADER_LEN);
11381138
if (err) {
11391139
nvme_msg(ctx, LOG_INFO,
11401140
"%s: discover try %d/%d failed, errno %d status 0x%x\n",
@@ -1168,8 +1168,7 @@ static int nvme_discovery_log(const struct nvme_get_discovery_args *args,
11681168
cmd.cdw10 |= NVME_FIELD_ENCODE(args->lsp,
11691169
NVME_LOG_CDW10_LSP_SHIFT,
11701170
NVME_LOG_CDW10_LSP_MASK);
1171-
err = nvme_get_log(hdl, &cmd, false,
1172-
NVME_LOG_PAGE_PDU_SIZE, NULL);
1171+
err = nvme_get_log(hdl, &cmd, false, NVME_LOG_PAGE_PDU_SIZE);
11731172
if (err) {
11741173
nvme_msg(ctx, LOG_INFO,
11751174
"%s: discover try %d/%d failed, errno %d status 0x%x\n",
@@ -1184,7 +1183,7 @@ static int nvme_discovery_log(const struct nvme_get_discovery_args *args,
11841183
nvme_msg(ctx, LOG_DEBUG, "%s: get header again\n", name);
11851184

11861185
nvme_init_get_log_discovery(&cmd, 0, log, DISCOVERY_HEADER_LEN);
1187-
err = nvme_get_log(hdl, &cmd, false, DISCOVERY_HEADER_LEN, NULL);
1186+
err = nvme_get_log(hdl, &cmd, false, DISCOVERY_HEADER_LEN);
11881187
if (err) {
11891188
nvme_msg(ctx, LOG_INFO,
11901189
"%s: discover try %d/%d failed, errno %d status 0x%x\n",
@@ -1669,7 +1668,7 @@ static int nvmf_dim(nvme_ctrl_t c, enum nvmf_dim_tas tas, __u8 trtype,
16691668
nvmf_fill_die(die, c->s->h, tel, trtype, adrfam, reg_addr, tsas);
16701669

16711670
nvme_init_dim_send(&cmd, tas, dim, tdl);
1672-
return nvme_submit_admin_passthru(hdl, &cmd, NULL);
1671+
return nvme_submit_admin_passthru(hdl, &cmd);
16731672
}
16741673

16751674
/**

‎libnvme/src/nvme/ioctl.c‎

Lines changed: 13 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -118,8 +118,7 @@ bool nvme_decide_retry(struct nvme_transport_handle *hdl,
118118
}
119119

120120
int nvme_submit_passthru(struct nvme_transport_handle *hdl,
121-
unsigned long ioctl_cmd, struct nvme_passthru_cmd *cmd,
122-
__u64 *result)
121+
unsigned long ioctl_cmd, struct nvme_passthru_cmd *cmd)
123122
{
124123
void *user_data;
125124
int err = 0;
@@ -134,25 +133,22 @@ int nvme_submit_passthru(struct nvme_transport_handle *hdl,
134133
break;
135134
} while (!nvme_decide_retry(hdl, cmd, errno));
136135

136+
if (err)
137+
err = -errno;
138+
137139
out:
138140
nvme_submit_exit(hdl, cmd, err, user_data);
139-
140-
if (err >= 0 && result)
141-
*result = cmd->result;
142-
if (err < 0)
143-
return -errno;
144141
return err;
145142
}
146143

147144
int nvme_submit_admin_passthru(struct nvme_transport_handle *hdl,
148-
struct nvme_passthru_cmd *cmd, __u64 *result)
145+
struct nvme_passthru_cmd *cmd)
149146
{
150147
switch (hdl->type) {
151148
case NVME_TRANSPORT_HANDLE_TYPE_DIRECT:
152-
return nvme_submit_passthru(hdl, NVME_IOCTL_ADMIN64_CMD,
153-
cmd, result);
149+
return nvme_submit_passthru(hdl, NVME_IOCTL_ADMIN64_CMD, cmd);
154150
case NVME_TRANSPORT_HANDLE_TYPE_MI:
155-
return nvme_mi_admin_admin_passthru(hdl, cmd, result);
151+
return nvme_mi_admin_admin_passthru(hdl, cmd);
156152
default:
157153
break;
158154
}
@@ -277,7 +273,7 @@ static bool nvme_uring_is_usable(struct nvme_transport_handle *hdl)
277273

278274
int nvme_get_log(struct nvme_transport_handle *hdl,
279275
struct nvme_passthru_cmd *cmd, bool rae,
280-
__u32 xfer_len, __u64 *result)
276+
__u32 xfer_len)
281277
{
282278
__u64 offset = 0, xfer, data_len = cmd->data_len;
283279
__u64 start = (__u64)cmd->cdw13 << 32 | cmd->cdw12;
@@ -352,12 +348,12 @@ int nvme_get_log(struct nvme_transport_handle *hdl,
352348
if (ret)
353349
nvme_uring_cmd_exit(&ring);
354350
} else {
355-
ret = nvme_submit_admin_passthru(hdl, cmd, result);
351+
ret = nvme_submit_admin_passthru(hdl, cmd);
356352
if (ret)
357353
return ret;
358354
}
359355
#else /* CONFIG_LIBURING */
360-
ret = nvme_submit_admin_passthru(hdl, cmd, result);
356+
ret = nvme_submit_admin_passthru(hdl, cmd);
361357
#endif /* CONFIG_LIBURING */
362358
if (ret)
363359
return ret;
@@ -391,7 +387,7 @@ static int read_ana_chunk(struct nvme_transport_handle *hdl, enum nvme_log_ana_l
391387
int ret;
392388

393389
nvme_init_get_log_ana(&cmd, lsp, *read - log, *read, len);
394-
ret = nvme_get_log(hdl, &cmd, rae, NVME_LOG_PAGE_PDU_SIZE, NULL);
390+
ret = nvme_get_log(hdl, &cmd, rae, NVME_LOG_PAGE_PDU_SIZE);
395391
if (ret)
396392
return ret;
397393

@@ -503,7 +499,7 @@ int nvme_get_ana_log_atomic(struct nvme_transport_handle *hdl, bool rae, bool rg
503499
}
504500

505501
int nvme_submit_io_passthru(struct nvme_transport_handle *hdl,
506-
struct nvme_passthru_cmd *cmd, __u64 *result)
502+
struct nvme_passthru_cmd *cmd)
507503
{
508-
return nvme_submit_passthru(hdl, NVME_IOCTL_IO64_CMD, cmd, result);
504+
return nvme_submit_passthru(hdl, NVME_IOCTL_IO64_CMD, cmd);
509505
}

0 commit comments

Comments
 (0)