All of lore.kernel.org
 help / color / mirror / Atom feed
From: Dan Carpenter <dan.carpenter@oracle.com>
To: alexei@purestorage.com
Cc: linux-scsi@vger.kernel.org
Subject: re: qla2xxx: Delete session if initiator is gone from FW
Date: Tue, 22 Dec 2015 22:46:17 +0300	[thread overview]
Message-ID: <20151222194617.GA31313@mwanda> (raw)

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 & 0xFFFF);
  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

             reply	other threads:[~2015-12-22 19:46 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-12-22 19:46 Dan Carpenter [this message]
2015-12-24  8:11 ` qla2xxx: Delete session if initiator is gone from FW Alexei Potashnik
2016-01-04 20:31   ` Himanshu Madhani

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20151222194617.GA31313@mwanda \
    --to=dan.carpenter@oracle.com \
    --cc=alexei@purestorage.com \
    --cc=linux-scsi@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.