Skip to content

Commit

Permalink
block: Change argument type of scsi_req_init()
Browse files Browse the repository at this point in the history
Since scsi_req_init() works on a struct scsi_request, change the
argument type into struct scsi_request *.

Signed-off-by: Bart Van Assche <[email protected]>
Reviewed-by: Christoph Hellwig <[email protected]>
Reviewed-by: Hannes Reinecke <[email protected]>
Reviewed-by: Martin K. Petersen <[email protected]>
Signed-off-by: Jens Axboe <[email protected]>
  • Loading branch information
Bart Van Assche authored and axboe committed Jun 21, 2017
1 parent ca18d6f commit c8d9cf2
Show file tree
Hide file tree
Showing 5 changed files with 13 additions and 7 deletions.
10 changes: 7 additions & 3 deletions block/scsi_ioctl.c
Original file line number Diff line number Diff line change
Expand Up @@ -741,10 +741,14 @@ int scsi_cmd_blk_ioctl(struct block_device *bd, fmode_t mode,
}
EXPORT_SYMBOL(scsi_cmd_blk_ioctl);

void scsi_req_init(struct request *rq)
/**
* scsi_req_init - initialize certain fields of a scsi_request structure
* @req: Pointer to a scsi_request structure.
* Initializes .__cmd[], .cmd, .cmd_len and .sense_len but no other members
* of struct scsi_request.
*/
void scsi_req_init(struct scsi_request *req)
{
struct scsi_request *req = scsi_req(rq);

memset(req->__cmd, 0, sizeof(req->__cmd));
req->cmd = req->__cmd;
req->cmd_len = BLK_MAX_CDB;
Expand Down
2 changes: 1 addition & 1 deletion drivers/ide/ide-atapi.c
Original file line number Diff line number Diff line change
Expand Up @@ -199,7 +199,7 @@ void ide_prep_sense(ide_drive_t *drive, struct request *rq)
memset(sense, 0, sizeof(*sense));

blk_rq_init(rq->q, sense_rq);
scsi_req_init(sense_rq);
scsi_req_init(req);

err = blk_rq_map_kern(drive->queue, sense_rq, sense, sense_len,
GFP_NOIO);
Expand Down
2 changes: 1 addition & 1 deletion drivers/ide/ide-probe.c
Original file line number Diff line number Diff line change
Expand Up @@ -745,7 +745,7 @@ static void ide_initialize_rq(struct request *rq)
{
struct ide_request *req = blk_mq_rq_to_pdu(rq);

scsi_req_init(rq);
scsi_req_init(&req->sreq);
req->sreq.sense = req->sense;
}

Expand Down
4 changes: 3 additions & 1 deletion drivers/scsi/scsi_lib.c
Original file line number Diff line number Diff line change
Expand Up @@ -1123,7 +1123,9 @@ EXPORT_SYMBOL(scsi_init_io);
*/
void scsi_initialize_rq(struct request *rq)
{
scsi_req_init(rq);
struct scsi_cmnd *cmd = blk_mq_rq_to_pdu(rq);

scsi_req_init(&cmd->req);
}
EXPORT_SYMBOL(scsi_initialize_rq);

Expand Down
2 changes: 1 addition & 1 deletion include/scsi/scsi_request.h
Original file line number Diff line number Diff line change
Expand Up @@ -27,6 +27,6 @@ static inline void scsi_req_free_cmd(struct scsi_request *req)
kfree(req->cmd);
}

void scsi_req_init(struct request *);
void scsi_req_init(struct scsi_request *req);

#endif /* _SCSI_SCSI_REQUEST_H */

0 comments on commit c8d9cf2

Please sign in to comment.