block: remove unnecessary argument from blk_execute_rq_nowait
ANBZ: #11744 i commit8eeed0b554
upstream. The 'q' is not used since commita1ce35fa49
("block: remove dead elevator code"), also update the comment of the function. And more importantly it never really was needed to start with given that we can trivial derive it from struct request. Cc: target-devel@vger.kernel.org Cc: linux-scsi@vger.kernel.org Cc: virtualization@lists.linux-foundation.org Cc: linux-ide@vger.kernel.org Cc: linux-mmc@vger.kernel.org Cc: linux-nvme@lists.infradead.org Cc: linux-nfs@vger.kernel.org Signed-off-by: Guoqing Jiang <guoqing.jiang@cloud.ionos.com> Signed-off-by: Jens Axboe <axboe@kernel.dk> [joe: also do the corresponding change in drivers/nvme/host/ioctl.c] Signed-off-by: Joseph Qi <joseph.qi@linux.alibaba.com> Reviewed-by: Ferry Meng <mengferry@linux.alibaba.com> Reviewed-by: Guixin Liu <kanie@linux.alibaba.com> Link: https://gitee.com/anolis/cloud-kernel/pulls/4084
This commit is contained in:
parent
6f6d9a0bda
commit
7f9dc9d81b
|
@ -41,8 +41,7 @@ bool blk_rq_is_poll(struct request *rq)
|
|||
EXPORT_SYMBOL_GPL(blk_rq_is_poll);
|
||||
|
||||
/**
|
||||
* blk_execute_rq_nowait - insert a request into queue for execution
|
||||
* @q: queue to insert the request in
|
||||
* blk_execute_rq_nowait - insert a request to I/O scheduler for execution
|
||||
* @bd_disk: matching gendisk
|
||||
* @rq: request to insert
|
||||
* @at_head: insert request at head or tail of queue
|
||||
|
@ -55,9 +54,8 @@ EXPORT_SYMBOL_GPL(blk_rq_is_poll);
|
|||
* Note:
|
||||
* This function will invoke @done directly if the queue is dead.
|
||||
*/
|
||||
void blk_execute_rq_nowait(struct request_queue *q, struct gendisk *bd_disk,
|
||||
struct request *rq, int at_head,
|
||||
rq_end_io_fn *done)
|
||||
void blk_execute_rq_nowait(struct gendisk *bd_disk, struct request *rq,
|
||||
int at_head, rq_end_io_fn *done)
|
||||
{
|
||||
WARN_ON(irqs_disabled());
|
||||
WARN_ON(!blk_rq_is_passthrough(rq));
|
||||
|
@ -93,7 +91,7 @@ void blk_execute_rq(struct request_queue *q, struct gendisk *bd_disk,
|
|||
unsigned long hang_check;
|
||||
|
||||
rq->end_io_data = &wait;
|
||||
blk_execute_rq_nowait(q, bd_disk, rq, at_head, blk_end_sync_rq);
|
||||
blk_execute_rq_nowait(bd_disk, rq, at_head, blk_end_sync_rq);
|
||||
|
||||
/* Prevent hang_check timer from firing at us during very long I/O */
|
||||
hang_check = sysctl_hung_task_timeout_secs;
|
||||
|
|
|
@ -539,7 +539,7 @@ static int carm_array_info (struct carm_host *host, unsigned int array_idx)
|
|||
spin_unlock_irq(&host->lock);
|
||||
|
||||
DPRINTK("blk_execute_rq_nowait, tag == %u\n", rq->tag);
|
||||
blk_execute_rq_nowait(host->oob_q, NULL, rq, true, NULL);
|
||||
blk_execute_rq_nowait(NULL, rq, true, NULL);
|
||||
|
||||
return 0;
|
||||
|
||||
|
@ -578,7 +578,7 @@ static int carm_send_special (struct carm_host *host, carm_sspc_t func)
|
|||
crq->msg_bucket = (u32) rc;
|
||||
|
||||
DPRINTK("blk_execute_rq_nowait, tag == %u\n", rq->tag);
|
||||
blk_execute_rq_nowait(host->oob_q, NULL, rq, true, NULL);
|
||||
blk_execute_rq_nowait(NULL, rq, true, NULL);
|
||||
|
||||
return 0;
|
||||
}
|
||||
|
|
|
@ -1040,7 +1040,7 @@ static void nvme_execute_rq_polled(struct request_queue *q,
|
|||
|
||||
rq->cmd_flags |= REQ_HIPRI;
|
||||
rq->end_io_data = &wait;
|
||||
blk_execute_rq_nowait(q, bd_disk, rq, at_head, nvme_end_sync_rq);
|
||||
blk_execute_rq_nowait(bd_disk, rq, at_head, nvme_end_sync_rq);
|
||||
|
||||
while (!completion_done(&wait)) {
|
||||
blk_poll(q, request_to_qc_t(rq->mq_hctx, rq), true);
|
||||
|
@ -1227,7 +1227,7 @@ static int nvme_keep_alive(struct nvme_ctrl *ctrl)
|
|||
rq->timeout = ctrl->kato * HZ;
|
||||
rq->end_io_data = ctrl;
|
||||
|
||||
blk_execute_rq_nowait(rq->q, NULL, rq, 0, nvme_keep_alive_end_io);
|
||||
blk_execute_rq_nowait(NULL, rq, 0, nvme_keep_alive_end_io);
|
||||
|
||||
return 0;
|
||||
}
|
||||
|
|
|
@ -492,7 +492,7 @@ retry:
|
|||
pdu->meta_buffer = nvme_to_user_ptr(d.metadata);
|
||||
pdu->meta_len = d.metadata_len;
|
||||
|
||||
blk_execute_rq_nowait(req->q, NULL, req, 0, nvme_uring_cmd_end_io);
|
||||
blk_execute_rq_nowait(NULL, req, 0, nvme_uring_cmd_end_io);
|
||||
return -EIOCBQUEUED;
|
||||
}
|
||||
|
||||
|
|
|
@ -696,7 +696,7 @@ static int nvme_nvm_submit_io(struct nvm_dev *dev, struct nvm_rq *rqd,
|
|||
|
||||
rq->end_io_data = rqd;
|
||||
|
||||
blk_execute_rq_nowait(q, NULL, rq, 0, nvme_nvm_end_io);
|
||||
blk_execute_rq_nowait(NULL, rq, 0, nvme_nvm_end_io);
|
||||
|
||||
return 0;
|
||||
|
||||
|
|
|
@ -1386,7 +1386,7 @@ static enum blk_eh_timer_return nvme_timeout(struct request *req, bool reserved)
|
|||
nvme_init_request(abort_req, &cmd);
|
||||
|
||||
abort_req->end_io_data = NULL;
|
||||
blk_execute_rq_nowait(abort_req->q, NULL, abort_req, 0, abort_endio);
|
||||
blk_execute_rq_nowait(NULL, abort_req, 0, abort_endio);
|
||||
|
||||
/*
|
||||
* The aborted req will be completed on receiving the abort req.
|
||||
|
@ -2403,7 +2403,7 @@ static int nvme_delete_queue(struct nvme_queue *nvmeq, u8 opcode)
|
|||
req->end_io_data = nvmeq;
|
||||
|
||||
init_completion(&nvmeq->delete_done);
|
||||
blk_execute_rq_nowait(q, NULL, req, false,
|
||||
blk_execute_rq_nowait(NULL, req, false,
|
||||
opcode == nvme_admin_delete_cq ?
|
||||
nvme_del_cq_end : nvme_del_queue_end);
|
||||
return 0;
|
||||
|
|
|
@ -278,7 +278,7 @@ static void nvmet_passthru_execute_cmd(struct nvmet_req *req)
|
|||
schedule_work(&req->p.work);
|
||||
} else {
|
||||
rq->end_io_data = req;
|
||||
blk_execute_rq_nowait(rq->q, ns ? ns->disk : NULL, rq, 0,
|
||||
blk_execute_rq_nowait(ns ? ns->disk : NULL, rq, 0,
|
||||
nvmet_passthru_req_done);
|
||||
}
|
||||
|
||||
|
|
|
@ -2003,7 +2003,7 @@ static void scsi_eh_lock_door(struct scsi_device *sdev)
|
|||
req->timeout = 10 * HZ;
|
||||
rq->retries = 5;
|
||||
|
||||
blk_execute_rq_nowait(req->q, NULL, req, 1, eh_lock_door_done);
|
||||
blk_execute_rq_nowait(NULL, req, 1, eh_lock_door_done);
|
||||
}
|
||||
|
||||
/**
|
||||
|
|
|
@ -829,8 +829,7 @@ sg_common_write(Sg_fd * sfp, Sg_request * srp,
|
|||
|
||||
srp->rq->timeout = timeout;
|
||||
kref_get(&sfp->f_ref); /* sg_rq_end_io() does kref_put(). */
|
||||
blk_execute_rq_nowait(sdp->device->request_queue, sdp->disk,
|
||||
srp->rq, at_head, sg_rq_end_io);
|
||||
blk_execute_rq_nowait(sdp->disk, srp->rq, at_head, sg_rq_end_io);
|
||||
return 0;
|
||||
}
|
||||
|
||||
|
|
|
@ -585,7 +585,7 @@ static int st_scsi_execute(struct st_request *SRpnt, const unsigned char *cmd,
|
|||
rq->retries = retries;
|
||||
req->end_io_data = SRpnt;
|
||||
|
||||
blk_execute_rq_nowait(req->q, NULL, req, 1, st_scsi_execute_end);
|
||||
blk_execute_rq_nowait(NULL, req, 1, st_scsi_execute_end);
|
||||
return 0;
|
||||
}
|
||||
|
||||
|
|
|
@ -1009,8 +1009,7 @@ pscsi_execute_cmd(struct se_cmd *cmd)
|
|||
req->timeout = PS_TIMEOUT_OTHER;
|
||||
scsi_req(req)->retries = PS_RETRY;
|
||||
|
||||
blk_execute_rq_nowait(pdv->pdv_sd->request_queue, NULL, req,
|
||||
(cmd->sam_task_attr == TCM_HEAD_TAG),
|
||||
blk_execute_rq_nowait(NULL, req, (cmd->sam_task_attr == TCM_HEAD_TAG),
|
||||
pscsi_req_done);
|
||||
|
||||
return 0;
|
||||
|
|
|
@ -981,7 +981,7 @@ extern int blk_rq_map_user_iov(struct request_queue *, struct request *,
|
|||
gfp_t);
|
||||
extern void blk_execute_rq(struct request_queue *, struct gendisk *,
|
||||
struct request *, int);
|
||||
extern void blk_execute_rq_nowait(struct request_queue *, struct gendisk *,
|
||||
extern void blk_execute_rq_nowait(struct gendisk *,
|
||||
struct request *, int, rq_end_io_fn *);
|
||||
extern bool blk_rq_is_poll(struct request *);
|
||||
|
||||
|
|
Loading…
Reference in New Issue