scsi: bnx2fc: Stop using the SCSI pointer

Set .cmd_size in the SCSI host template instead of using the SCSI pointer
from struct scsi_cmnd. Remove the CMD_SCSI_STATUS() assignment because the
assigned value is not used.

This patch prepares for removal of the SCSI pointer from struct scsi_cmnd.

Link: https://lore.kernel.org/r/20220218195117.25689-29-bvanassche@acm.org
Reviewed-by: Hannes Reinecke <hare@suse.de>
Signed-off-by: Bart Van Assche <bvanassche@acm.org>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>

diff --git a/drivers/scsi/bnx2fc/bnx2fc.h b/drivers/scsi/bnx2fc/bnx2fc.h
index b4cea8b06ea1d130181ef1a83b46431887547dab..046247420cfa5a67eb9c301c48cf8c94102577ca 100644 (file)
--- a/drivers/scsi/bnx2fc/bnx2fc.h
+++ b/drivers/scsi/bnx2fc/bnx2fc.h
@@ -137,8 +137,6 @@
 #define BNX2FC_FW_TIMEOUT              (3 * HZ)
 #define PORT_MAX                       2
 
-#define CMD_SCSI_STATUS(Cmnd)          ((Cmnd)->SCp.Status)
-
 /* FC FCP Status */
 #define        FC_GOOD                         0
 
@@ -493,7 +491,14 @@ struct bnx2fc_unsol_els {
        struct work_struct unsol_els_work;
 };
 
+struct bnx2fc_priv {
+       struct bnx2fc_cmd *io_req;
+};
 
+static inline struct bnx2fc_priv *bnx2fc_priv(struct scsi_cmnd *cmd)
+{
+       return scsi_cmd_priv(cmd);
+}
 
 struct bnx2fc_cmd *bnx2fc_cmd_alloc(struct bnx2fc_rport *tgt);
 struct bnx2fc_cmd *bnx2fc_elstm_alloc(struct bnx2fc_rport *tgt, int type);

diff --git a/drivers/scsi/bnx2fc/bnx2fc_fcoe.c b/drivers/scsi/bnx2fc/bnx2fc_fcoe.c
index 05f5afc304d4d8e04d95447a30482bfa7e837b8f..d295867a9b46545103e7e668afb9b498fcb910d7 100644 (file)
--- a/drivers/scsi/bnx2fc/bnx2fc_fcoe.c
+++ b/drivers/scsi/bnx2fc/bnx2fc_fcoe.c
@@ -2974,6 +2974,7 @@ static struct scsi_host_template bnx2fc_shost_template = {
        .track_queue_depth      = 1,
        .slave_configure        = bnx2fc_slave_configure,
        .shost_groups           = bnx2fc_host_groups,
+       .cmd_size               = sizeof(struct bnx2fc_priv),
 };
 
 static struct libfc_function_template bnx2fc_libfc_fcn_templ = {

diff --git a/drivers/scsi/bnx2fc/bnx2fc_io.c b/drivers/scsi/bnx2fc/bnx2fc_io.c
index b9114113ee73b8c2290a3b1c8471ddf9bacf8c01..962454f2e2b16c9d88318aed9d7a9735983b338b 100644 (file)
--- a/drivers/scsi/bnx2fc/bnx2fc_io.c
+++ b/drivers/scsi/bnx2fc/bnx2fc_io.c
@@ -204,7 +204,7 @@ static void bnx2fc_scsi_done(struct bnx2fc_cmd *io_req, int err_code)
                sc_cmd, host_byte(sc_cmd->result), sc_cmd->retries,
                sc_cmd->allowed);
        scsi_set_resid(sc_cmd, scsi_bufflen(sc_cmd));
-       sc_cmd->SCp.ptr = NULL;
+       bnx2fc_priv(sc_cmd)->io_req = NULL;
        scsi_done(sc_cmd);
 }
 
@@ -765,7 +765,7 @@ retry_tmf:
        task = &(task_page[index]);
        bnx2fc_init_mp_task(io_req, task);
 
-       sc_cmd->SCp.ptr = (char *)io_req;
+       bnx2fc_priv(sc_cmd)->io_req = io_req;
 
        /* Obtain free SQ entry */
        spin_lock_bh(&tgt->tgt_lock);
@@ -1147,7 +1147,7 @@ int bnx2fc_eh_abort(struct scsi_cmnd *sc_cmd)
        BNX2FC_TGT_DBG(tgt, "Entered bnx2fc_eh_abort\n");
 
        spin_lock_bh(&tgt->tgt_lock);
-       io_req = (struct bnx2fc_cmd *)sc_cmd->SCp.ptr;
+       io_req = bnx2fc_priv(sc_cmd)->io_req;
        if (!io_req) {
                /* Command might have just completed */
                printk(KERN_ERR PFX "eh_abort: io_req is NULL\n");
@@ -1572,8 +1572,8 @@ void bnx2fc_process_tm_compl(struct bnx2fc_cmd *io_req,
                printk(KERN_ERR PFX "tmf's fc_hdr r_ctl = 0x%x\n",
                        fc_hdr->fh_r_ctl);
        }
-       if (!sc_cmd->SCp.ptr) {
-               printk(KERN_ERR PFX "tm_compl: SCp.ptr is NULL\n");
+       if (!bnx2fc_priv(sc_cmd)->io_req) {
+               printk(KERN_ERR PFX "tm_compl: io_req is NULL\n");
                return;
        }
        switch (io_req->fcp_status) {
@@ -1609,7 +1609,7 @@ void bnx2fc_process_tm_compl(struct bnx2fc_cmd *io_req,
                return;
        }
 
-       sc_cmd->SCp.ptr = NULL;
+       bnx2fc_priv(sc_cmd)->io_req = NULL;
        scsi_done(sc_cmd);
 
        kref_put(&io_req->refcount, bnx2fc_cmd_release);
@@ -1773,8 +1773,7 @@ static void bnx2fc_parse_fcp_rsp(struct bnx2fc_cmd *io_req,
                io_req->fcp_resid = fcp_rsp->fcp_resid;
 
        io_req->scsi_comp_flags = rsp_flags;
-       CMD_SCSI_STATUS(sc_cmd) = io_req->cdb_status =
-                               fcp_rsp->scsi_status_code;
+       io_req->cdb_status = fcp_rsp->scsi_status_code;
 
        /* Fetch fcp_rsp_info and fcp_sns_info if available */
        if (num_rq) {
@@ -1946,8 +1945,8 @@ void bnx2fc_process_scsi_cmd_compl(struct bnx2fc_cmd *io_req,
        /* parse fcp_rsp and obtain sense data from RQ if available */
        bnx2fc_parse_fcp_rsp(io_req, fcp_rsp, num_rq, rq_data);
 
-       if (!sc_cmd->SCp.ptr) {
-               printk(KERN_ERR PFX "SCp.ptr is NULL\n");
+       if (!bnx2fc_priv(sc_cmd)->io_req) {
+               printk(KERN_ERR PFX "io_req is NULL\n");
                return;
        }
 
@@ -2018,7 +2017,7 @@ void bnx2fc_process_scsi_cmd_compl(struct bnx2fc_cmd *io_req,
                        io_req->fcp_status);
                break;
        }
-       sc_cmd->SCp.ptr = NULL;
+       bnx2fc_priv(sc_cmd)->io_req = NULL;
        scsi_done(sc_cmd);
        kref_put(&io_req->refcount, bnx2fc_cmd_release);
 }
@@ -2044,7 +2043,7 @@ int bnx2fc_post_io_req(struct bnx2fc_rport *tgt,
        io_req->port = port;
        io_req->tgt = tgt;
        io_req->data_xfer_len = scsi_bufflen(sc_cmd);
-       sc_cmd->SCp.ptr = (char *)io_req;
+       bnx2fc_priv(sc_cmd)->io_req = io_req;
 
        stats = per_cpu_ptr(lport->stats, get_cpu());
        if (sc_cmd->sc_data_direction == DMA_FROM_DEVICE) {