nvmet: use nvme status value directly
authorChaitanya Kulkarni <chaitanya.kulkarni@wdc.com>
Mon, 14 Jun 2021 01:58:50 +0000 (18:58 -0700)
committerChristoph Hellwig <hch@lst.de>
Thu, 17 Jun 2021 13:51:20 +0000 (15:51 +0200)
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>
drivers/nvme/target/core.c

index 8494a13..45a5b27 100644 (file)
@@ -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);