Re: qla2xxx: Delete session if initiator is gone from FW
Hi Alexie/Dan,
I will send follow up patch to fix this.
Thanks,
-Himanshu
On 12/24/15, 12:11 AM, "Alexei Potashnik" <ale...@purestorage.com> wrote:
>I my original patch line 3583 was:
>
>+ bool logged_out = (status & 0x) ==
>CTIO_PORT_LOGGED_OUT;
>
>and line 3587 was:
>
>+ logged_out ? "PORT LOGGED OUT" : "PORT
>UNAVAILABLE",
>
>Forwarding to Quinn and Himanshu who must have massaged it.
>
>Alexei
>
>> -Original Message-
>> From: Dan Carpenter [mailto:dan.carpen...@oracle.com]
>> Sent: Tuesday, December 22, 2015 11:46 AM
>> To: ale...@purestorage.com
>> Cc: linux-scsi@vger.kernel.org
>> Subject: re: qla2xxx: Delete session if initiator is gone from FW
>>
>> Hello Alexei Potashnik,
>>
>> The patch b93bb8ecc389: "qla2xxx: Delete session if initiator is gone
>from
>> FW" from Dec 17, 2015, leads to the following static checker
>> warning:
>>
>> drivers/scsi/qla2xxx/qla_target.c:3587 qlt_do_ctio_completion()
>> warn: impossible condition '(logged_out == 41) => (0-1 == 41)'
>>
>> drivers/scsi/qla2xxx/qla_target.c
>> 3580 case CTIO_PORT_LOGGED_OUT:
>> 3581 case CTIO_PORT_UNAVAILABLE:
>> 3582 {
>> 3583 bool logged_out = (status & 0x);
>> 3584 ql_dbg(ql_dbg_tgt_mgt, vha, 0xf059,
>> 3585 "qla_target(%d): CTIO with %s status
>%x "
>> 3586 "received (state %x, se_cmd %p)\n",
>vha->vp_idx,
>> 3587 (logged_out == CTIO_PORT_LOGGED_OUT)
>?
>> ^^
>> Bool cannot equal 0x26.
>>
>> 3588 "PORT LOGGED OUT" : "PORT
>UNAVAILABLE",
>> 3589 status, cmd->state, se_cmd);
>> 3590
>> 3591 if (logged_out && cmd->sess) {
>> ^^
>> == CTIO_PORT_LOGGED_OUT here?
>>
>> 3592 /*
>> 3593 * Session is already logged
>out, but we need
>> 3594 * to notify initiator, who's
>not aware of this
>> 3595 */
>> 3596 cmd->sess->logout_on_delete = 0;
>> 3597 cmd->sess->send_els_logo = 1;
>> 3598
>qlt_schedule_sess_for_deletion(cmd->sess, true);
>> 3599 }
>> 3600 break;
>> 3601 }
>>
>> regards,
>> dan carpenter
<>
RE: qla2xxx: Delete session if initiator is gone from FW
I my original patch line 3583 was:
+ bool logged_out = (status & 0x) ==
CTIO_PORT_LOGGED_OUT;
and line 3587 was:
+ logged_out ? "PORT LOGGED OUT" : "PORT
UNAVAILABLE",
Forwarding to Quinn and Himanshu who must have massaged it.
Alexei
> -Original Message-
> From: Dan Carpenter [mailto:dan.carpen...@oracle.com]
> Sent: Tuesday, December 22, 2015 11:46 AM
> To: ale...@purestorage.com
> Cc: linux-scsi@vger.kernel.org
> Subject: re: qla2xxx: Delete session if initiator is gone from FW
>
> Hello Alexei Potashnik,
>
> The patch b93bb8ecc389: "qla2xxx: Delete session if initiator is gone
from
> FW" from Dec 17, 2015, leads to the following static checker
> warning:
>
> drivers/scsi/qla2xxx/qla_target.c:3587 qlt_do_ctio_completion()
> warn: impossible condition '(logged_out == 41) => (0-1 == 41)'
>
> drivers/scsi/qla2xxx/qla_target.c
> 3580 case CTIO_PORT_LOGGED_OUT:
> 3581 case CTIO_PORT_UNAVAILABLE:
> 3582 {
> 3583 bool logged_out = (status & 0x);
> 3584 ql_dbg(ql_dbg_tgt_mgt, vha, 0xf059,
> 3585 "qla_target(%d): CTIO with %s status
%x "
> 3586 "received (state %x, se_cmd %p)\n",
vha->vp_idx,
> 3587 (logged_out == CTIO_PORT_LOGGED_OUT)
?
> ^^
> Bool cannot equal 0x26.
>
> 3588 "PORT LOGGED OUT" : "PORT
UNAVAILABLE",
> 3589 status, cmd->state, se_cmd);
> 3590
> 3591 if (logged_out && cmd->sess) {
> ^^
> == CTIO_PORT_LOGGED_OUT here?
>
> 3592 /*
> 3593 * Session is already logged
out, but we need
> 3594 * to notify initiator, who's
not aware of this
> 3595 */
> 3596 cmd->sess->logout_on_delete = 0;
> 3597 cmd->sess->send_els_logo = 1;
> 3598
qlt_schedule_sess_for_deletion(cmd->sess, true);
> 3599 }
> 3600 break;
> 3601 }
>
> regards,
> dan carpenter
--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
re: qla2xxx: Delete session if initiator is gone from FW
Hello Alexei Potashnik,
The patch b93bb8ecc389: "qla2xxx: Delete session if initiator is gone
from FW" from Dec 17, 2015, leads to the following static checker
warning:
drivers/scsi/qla2xxx/qla_target.c:3587 qlt_do_ctio_completion()
warn: impossible condition '(logged_out == 41) => (0-1 == 41)'
drivers/scsi/qla2xxx/qla_target.c
3580 case CTIO_PORT_LOGGED_OUT:
3581 case CTIO_PORT_UNAVAILABLE:
3582 {
3583 bool logged_out = (status & 0x);
3584 ql_dbg(ql_dbg_tgt_mgt, vha, 0xf059,
3585 "qla_target(%d): CTIO with %s status %x "
3586 "received (state %x, se_cmd %p)\n",
vha->vp_idx,
3587 (logged_out == CTIO_PORT_LOGGED_OUT) ?
^^
Bool cannot equal 0x26.
3588 "PORT LOGGED OUT" : "PORT UNAVAILABLE",
3589 status, cmd->state, se_cmd);
3590
3591 if (logged_out && cmd->sess) {
^^
== CTIO_PORT_LOGGED_OUT here?
3592 /*
3593 * Session is already logged out, but
we need
3594 * to notify initiator, who's not aware
of this
3595 */
3596 cmd->sess->logout_on_delete = 0;
3597 cmd->sess->send_els_logo = 1;
3598
qlt_schedule_sess_for_deletion(cmd->sess, true);
3599 }
3600 break;
3601 }
regards,
dan carpenter
--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
