On 7/9/15, 7:33 AM, "Bart Van Assche" <bart.vanassche@xxxxxxxxxxx> wrote:
>Detected these by building with W=1. This patch does not change
>any functionality.
>
>Signed-off-by: Bart Van Assche <bart.vanassche@xxxxxxxxxxx>
>Cc: Himanshu Madhani <himanshu.madhani@xxxxxxxxxx>
>Cc: Quinn Tran <quinn.tran@xxxxxxxxxx>
>Cc: Saurav Kashyap <saurav.kashyap@xxxxxxxxxx>
>---
> drivers/scsi/qla2xxx/tcm_qla2xxx.c | 6 ------
> 1 file changed, 6 deletions(-)
>
>diff --git a/drivers/scsi/qla2xxx/tcm_qla2xxx.c
>b/drivers/scsi/qla2xxx/tcm_qla2xxx.c
>index d9a8c60..474d0c0 100644
>--- a/drivers/scsi/qla2xxx/tcm_qla2xxx.c
>+++ b/drivers/scsi/qla2xxx/tcm_qla2xxx.c
>@@ -1367,9 +1367,7 @@ static void tcm_qla2xxx_free_session(struct
>qla_tgt_sess *sess)
> struct qla_hw_data *ha = tgt->ha;
> scsi_qla_host_t *vha = pci_get_drvdata(ha->pdev);
> struct se_session *se_sess;
>- struct se_node_acl *se_nacl;
> struct tcm_qla2xxx_lport *lport;
>- struct tcm_qla2xxx_nacl *nacl;
>
> BUG_ON(in_interrupt());
>
>@@ -1379,8 +1377,6 @@ static void tcm_qla2xxx_free_session(struct
>qla_tgt_sess *sess)
> dump_stack();
> return;
> }
>- se_nacl = se_sess->se_node_acl;
>- nacl = container_of(se_nacl, struct tcm_qla2xxx_nacl, se_node_acl);
>
> lport = vha->vha_tgt.target_lport_ptr;
> if (!lport) {
>@@ -1684,7 +1680,6 @@ static int
>tcm_qla2xxx_lport_register_npiv_cb(struct scsi_qla_host *base_vha,
> (struct tcm_qla2xxx_lport *)target_lport_ptr;
> struct tcm_qla2xxx_lport *base_lport =
> (struct tcm_qla2xxx_lport *)base_vha->vha_tgt.target_lport_ptr;
>- struct tcm_qla2xxx_tpg *base_tpg;
> struct fc_vport_identifiers vport_id;
>
> if (!qla_tgt_mode_enabled(base_vha)) {
>@@ -1697,7 +1692,6 @@ static int
>tcm_qla2xxx_lport_register_npiv_cb(struct scsi_qla_host *base_vha,
> pr_err("qla2xxx base_lport or tpg_1 not available\n");
> return -EPERM;
> }
>- base_tpg = base_lport->tpg_1;
>
> memset(&vport_id, 0, sizeof(vport_id));
> vport_id.port_name = npiv_wwpn;
>--
>2.1.4
>
Looks Good.
Acked-by: Himanshu Madhani <himanshu.madhani@xxxxxxxxxx>
>
--
To unsubscribe from this list: send the line "unsubscribe target-devel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
