nvmet-fcloop: add missing fcloop_callback_host_done

Add the missing fcloop_call_host_done calls so that the caller
frees resources when something goes wrong.

Signed-off-by: Daniel Wagner <wagi@kernel.org>
Reviewed-by: Hannes Reinecke <hare@kernel.org>
Signed-off-by: Christoph Hellwig <hch@lst.de>

diff --git a/drivers/nvme/target/fcloop.c b/drivers/nvme/target/fcloop.c
index 4c60eaf0b653b9d86e91e48e520d525a58b7688c..bac60c6de6cf16e59a1245263bba3971e97cabf0 100644 (file)
--- a/drivers/nvme/target/fcloop.c
+++ b/drivers/nvme/target/fcloop.c
@@ -1002,9 +1002,10 @@ fcloop_fcp_abort(struct nvme_fc_local_port *localport,
        }
        spin_unlock(&inireq->inilock);
 
-       if (!tfcp_req)
+       if (!tfcp_req) {
                /* abort has already been called */
-               return;
+               goto out_host_done;
+       }
 
        /* break initiator/target relationship for io */
        spin_lock_irqsave(&tfcp_req->reqlock, flags);
@@ -1019,7 +1020,7 @@ fcloop_fcp_abort(struct nvme_fc_local_port *localport,
        default:
                spin_unlock_irqrestore(&tfcp_req->reqlock, flags);
                WARN_ON(1);
-               return;
+               goto out_host_done;
        }
        spin_unlock_irqrestore(&tfcp_req->reqlock, flags);
 
@@ -1033,6 +1034,11 @@ fcloop_fcp_abort(struct nvme_fc_local_port *localport,
                 */
                fcloop_tfcp_req_put(tfcp_req);
        }
+
+       return;
+
+out_host_done:
+       fcloop_call_host_done(fcpreq, tfcp_req, -ECANCELED);
 }
 
 static void