bsg-lib: introduce a timeout field in struct bsg_job
authorChristoph Hellwig <hch@lst.de>
Tue, 13 Mar 2018 16:28:39 +0000 (17:28 +0100)
committerJens Axboe <axboe@kernel.dk>
Tue, 13 Mar 2018 17:40:21 +0000 (11:40 -0600)
The zfcp driver wants to know the timeout for a bsg job, so add a field
to struct bsg_job for it in preparation of not exposing the request
to the bsg-lib users.

Signed-off-by: Christoph Hellwig <hch@lst.de>
Reviewed-by: Benjamin Block <bblock@linux.vnet.ibm.com>
Reviewed-by: Hannes Reinecke <hare@suse.com>
Reviewed-by: Johannes Thumshirn <jthumshirn@suse.de>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
block/bsg-lib.c
drivers/s390/scsi/zfcp_fc.c
include/linux/bsg-lib.h

index b4fe1a48f111f981e9b3c5fbc921284a9c04a363..fb509779a0900ea55cffbe7a642b7a2a965890df 100644 (file)
@@ -132,6 +132,7 @@ static int bsg_prepare_job(struct device *dev, struct request *req)
        struct bsg_job *job = blk_mq_rq_to_pdu(req);
        int ret;
 
+       job->timeout = req->timeout;
        job->request = rq->cmd;
        job->request_len = rq->cmd_len;
 
index ca218c82321f882228b11be13aed307aee208b60..6162cf57a20af5226b9a6abfd82b1f329b1026d6 100644 (file)
@@ -961,7 +961,7 @@ static int zfcp_fc_exec_els_job(struct bsg_job *job,
                d_id = ntoh24(bsg_request->rqst_data.h_els.port_id);
 
        els->handler = zfcp_fc_ct_els_job_handler;
-       return zfcp_fsf_send_els(adapter, d_id, els, job->req->timeout / HZ);
+       return zfcp_fsf_send_els(adapter, d_id, els, job->timeout / HZ);
 }
 
 static int zfcp_fc_exec_ct_job(struct bsg_job *job,
@@ -980,7 +980,7 @@ static int zfcp_fc_exec_ct_job(struct bsg_job *job,
                return ret;
 
        ct->handler = zfcp_fc_ct_job_handler;
-       ret = zfcp_fsf_send_ct(wka_port, ct, NULL, job->req->timeout / HZ);
+       ret = zfcp_fsf_send_ct(wka_port, ct, NULL, job->timeout / HZ);
        if (ret)
                zfcp_fc_wka_port_put(wka_port);
 
index b1be0233ce353d93241bf1d1776d901637e9b95a..402223c95ce19a5be1cce90fd0238ad83e1d1b7a 100644 (file)
@@ -44,6 +44,8 @@ struct bsg_job {
 
        struct kref kref;
 
+       unsigned int timeout;
+
        /* Transport/driver specific request/reply structs */
        void *request;
        void *reply;