diff options
author | Chaitanya Kulkarni <chaitanya.kulkarni@wdc.com> | 2021-06-14 03:58:50 +0200 |
---|---|---|
committer | Christoph Hellwig <hch@lst.de> | 2021-06-17 15:51:20 +0200 |
commit | 8bb6cb9b97ef0b0ae4a492db5a90f8156d2cbe85 (patch) | |
tree | 3dff07241740365bdfa5201964df0656b97ee776 /drivers/nvme | |
parent | nvmet: use u32 type for the local variable nsid (diff) | |
download | linux-8bb6cb9b97ef0b0ae4a492db5a90f8156d2cbe85.tar.xz linux-8bb6cb9b97ef0b0ae4a492db5a90f8156d2cbe85.zip |
nvmet: use nvme status value directly
There is no point in keeping the status variable that is used only once
in the function nvmet_async_events_failall().
Remove the variable and use the value directly.
Signed-off-by: Chaitanya Kulkarni <chaitanya.kulkarni@wdc.com>
Signed-off-by: Christoph Hellwig <hch@lst.de>
Diffstat (limited to 'drivers/nvme')
-rw-r--r-- | drivers/nvme/target/core.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/nvme/target/core.c b/drivers/nvme/target/core.c index 8494a132da35..45a5b273b525 100644 --- a/drivers/nvme/target/core.c +++ b/drivers/nvme/target/core.c @@ -141,14 +141,13 @@ static u32 nvmet_async_event_result(struct nvmet_async_event *aen) static void nvmet_async_events_failall(struct nvmet_ctrl *ctrl) { - u16 status = NVME_SC_INTERNAL | NVME_SC_DNR; struct nvmet_req *req; mutex_lock(&ctrl->lock); while (ctrl->nr_async_event_cmds) { req = ctrl->async_event_cmds[--ctrl->nr_async_event_cmds]; mutex_unlock(&ctrl->lock); - nvmet_req_complete(req, status); + nvmet_req_complete(req, NVME_SC_INTERNAL | NVME_SC_DNR); mutex_lock(&ctrl->lock); } mutex_unlock(&ctrl->lock); |