Skip to content

Commit 7ee51cf

Browse files
ChaitanayaKulkarniaxboe
authored andcommitted
nvmet: call blk_mq_free_request() directly
Instead of calling blk_put_request() which calls blk_mq_free_request(), call blk_mq_free_request() directly for NVMeOF passthru. This is to mainly avoid an extra function call in the completion path nvmet_passthru_req_done(). Signed-off-by: Chaitanya Kulkarni <[email protected]> Reviewed-by: Keith Busch <[email protected]> Reviewed-by: Logan Gunthorpe <[email protected]> Signed-off-by: Sagi Grimberg <[email protected]> Signed-off-by: Jens Axboe <[email protected]>
1 parent a2138fd commit 7ee51cf

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

drivers/nvme/target/passthru.c

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -165,7 +165,7 @@ static void nvmet_passthru_execute_cmd_work(struct work_struct *w)
165165

166166
req->cqe->result = nvme_req(rq)->result;
167167
nvmet_req_complete(req, status);
168-
blk_put_request(rq);
168+
blk_mq_free_request(rq);
169169
}
170170

171171
static void nvmet_passthru_req_done(struct request *rq,
@@ -175,7 +175,7 @@ static void nvmet_passthru_req_done(struct request *rq,
175175

176176
req->cqe->result = nvme_req(rq)->result;
177177
nvmet_req_complete(req, nvme_req(rq)->status);
178-
blk_put_request(rq);
178+
blk_mq_free_request(rq);
179179
}
180180

181181
static int nvmet_passthru_map_sg(struct nvmet_req *req, struct request *rq)
@@ -274,7 +274,7 @@ static void nvmet_passthru_execute_cmd(struct nvmet_req *req)
274274
return;
275275

276276
out_put_req:
277-
blk_put_request(rq);
277+
blk_mq_free_request(rq);
278278
out_put_ns:
279279
if (ns)
280280
nvme_put_ns(ns);

0 commit comments

Comments
 (0)