Skip to content

Commit 61f3715

Browse files
author
Christoph Hellwig
committed
nvmet: use NVME_CMD_EFFECTS_CSUPP instead of open coding it
Use NVME_CMD_EFFECTS_CSUPP instead of open coding it and assign a single value to multiple array entries instead of repeated assignments. Signed-off-by: Christoph Hellwig <[email protected]> Reviewed-by: Keith Busch <[email protected]> Reviewed-by: Sagi Grimberg <[email protected]> Reviewed-by: Kanchan Joshi <[email protected]> Reviewed-by: Chaitanya Kulkarni <[email protected]>
1 parent 685e631 commit 61f3715

File tree

1 file changed

+19
-16
lines changed

1 file changed

+19
-16
lines changed

drivers/nvme/target/admin-cmd.c

Lines changed: 19 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -164,26 +164,29 @@ static void nvmet_execute_get_log_page_smart(struct nvmet_req *req)
164164

165165
static void nvmet_get_cmd_effects_nvm(struct nvme_effects_log *log)
166166
{
167-
log->acs[nvme_admin_get_log_page] = cpu_to_le32(1 << 0);
168-
log->acs[nvme_admin_identify] = cpu_to_le32(1 << 0);
169-
log->acs[nvme_admin_abort_cmd] = cpu_to_le32(1 << 0);
170-
log->acs[nvme_admin_set_features] = cpu_to_le32(1 << 0);
171-
log->acs[nvme_admin_get_features] = cpu_to_le32(1 << 0);
172-
log->acs[nvme_admin_async_event] = cpu_to_le32(1 << 0);
173-
log->acs[nvme_admin_keep_alive] = cpu_to_le32(1 << 0);
174-
175-
log->iocs[nvme_cmd_read] = cpu_to_le32(1 << 0);
176-
log->iocs[nvme_cmd_write] = cpu_to_le32(1 << 0);
177-
log->iocs[nvme_cmd_flush] = cpu_to_le32(1 << 0);
178-
log->iocs[nvme_cmd_dsm] = cpu_to_le32(1 << 0);
179-
log->iocs[nvme_cmd_write_zeroes] = cpu_to_le32(1 << 0);
167+
log->acs[nvme_admin_get_log_page] =
168+
log->acs[nvme_admin_identify] =
169+
log->acs[nvme_admin_abort_cmd] =
170+
log->acs[nvme_admin_set_features] =
171+
log->acs[nvme_admin_get_features] =
172+
log->acs[nvme_admin_async_event] =
173+
log->acs[nvme_admin_keep_alive] =
174+
cpu_to_le32(NVME_CMD_EFFECTS_CSUPP);
175+
176+
log->iocs[nvme_cmd_read] =
177+
log->iocs[nvme_cmd_write] =
178+
log->iocs[nvme_cmd_flush] =
179+
log->iocs[nvme_cmd_dsm] =
180+
log->iocs[nvme_cmd_write_zeroes] =
181+
cpu_to_le32(NVME_CMD_EFFECTS_CSUPP);
180182
}
181183

182184
static void nvmet_get_cmd_effects_zns(struct nvme_effects_log *log)
183185
{
184-
log->iocs[nvme_cmd_zone_append] = cpu_to_le32(1 << 0);
185-
log->iocs[nvme_cmd_zone_mgmt_send] = cpu_to_le32(1 << 0);
186-
log->iocs[nvme_cmd_zone_mgmt_recv] = cpu_to_le32(1 << 0);
186+
log->iocs[nvme_cmd_zone_append] =
187+
log->iocs[nvme_cmd_zone_mgmt_send] =
188+
log->iocs[nvme_cmd_zone_mgmt_recv] =
189+
cpu_to_le32(NVME_CMD_EFFECTS_CSUPP);
187190
}
188191

189192
static void nvmet_execute_get_log_cmd_effects_ns(struct nvmet_req *req)

0 commit comments

Comments
 (0)