On 6/4/15, 3:57 PM, "Bart Van Assche" <[email protected]> wrote:
>Comment out the code that is never reached in qla83xx_idc_unlock() >and also in qlt_set_data_offset() to avoid that static source code >analysis tools report warnings for this code. > >Signed-off-by: Bart Van Assche <[email protected]> >Cc: Himanshu Madhani <[email protected]> >Cc: Quinn Tran <[email protected]> >Cc: Saurav Kashyap <[email protected]> >--- > drivers/scsi/qla2xxx/qla_os.c | 7 ++++++- > drivers/scsi/qla2xxx/qla_target.c | 16 +++++++++------- > 2 files changed, 15 insertions(+), 8 deletions(-) > >diff --git a/drivers/scsi/qla2xxx/qla_os.c b/drivers/scsi/qla2xxx/qla_os.c >index 0ababd5..5a5166b 100644 >--- a/drivers/scsi/qla2xxx/qla_os.c >+++ b/drivers/scsi/qla2xxx/qla_os.c >@@ -4417,7 +4417,10 @@ retry_lock2: > void > qla83xx_idc_unlock(scsi_qla_host_t *base_vha, uint16_t requester_id) > { >- uint16_t options = (requester_id << 15) | BIT_7, retry; >+#if 0 >+ uint16_t options = (requester_id << 15) | BIT_7; >+#endif >+ uint16_t retry; > uint32_t data; > struct qla_hw_data *ha = base_vha->hw; > >@@ -4453,6 +4456,7 @@ retry_unlock: > > return; > >+#if 0 > /* XXX: IDC-unlock implementation using access-control mbx */ > retry = 0; > retry_unlock2: >@@ -4468,6 +4472,7 @@ retry_unlock2: > } > > return; >+#endif > } > > int >diff --git a/drivers/scsi/qla2xxx/qla_target.c >b/drivers/scsi/qla2xxx/qla_target.c >index 59a0897..22a91a3 100644 >--- a/drivers/scsi/qla2xxx/qla_target.c >+++ b/drivers/scsi/qla2xxx/qla_target.c >@@ -3699,6 +3699,14 @@ static int qlt_24xx_handle_els(struct >scsi_qla_host *vha, > > static int qlt_set_data_offset(struct qla_tgt_cmd *cmd, uint32_t offset) > { >+#if 1 >+ /* >+ * FIXME: Reject non zero SRR relative offset until we can test >+ * this code properly. >+ */ >+ pr_debug("Rejecting non zero SRR rel_offs: %u\n", offset); >+ return -1; >+#else > struct scatterlist *sg, *sgp, *sg_srr, *sg_srr_start = NULL; > size_t first_offset = 0, rem_offset = offset, tmp = 0; > int i, sg_srr_cnt, bufflen = 0; >@@ -3708,13 +3716,6 @@ static int qlt_set_data_offset(struct qla_tgt_cmd >*cmd, uint32_t offset) > "cmd->sg_cnt: %u, direction: %d\n", > cmd, cmd->sg, cmd->sg_cnt, cmd->dma_data_direction); > >- /* >- * FIXME: Reject non zero SRR relative offset until we can test >- * this code properly. >- */ >- pr_debug("Rejecting non zero SRR rel_offs: %u\n", offset); >- return -1; >- > if (!cmd->sg || !cmd->sg_cnt) { > ql_dbg(ql_dbg_tgt, cmd->vha, 0xe055, > "Missing cmd->sg or zero cmd->sg_cnt in" >@@ -3797,6 +3798,7 @@ static int qlt_set_data_offset(struct qla_tgt_cmd >*cmd, uint32_t offset) > BUG(); > > return 0; >+#endif > } > > static inline int qlt_srr_adjust_data(struct qla_tgt_cmd *cmd, >-- >2.1.4 > Looks good. Acked-by: Himanshu Madhani <[email protected]> >
<<attachment: winmail.dat>>
