From: Konrad Rzeszutek <konrad@virtualiron.com>
To: Andrew Vasquez <andrew.vasquez@qlogic.com>
Cc: James Bottomley <James.Bottomley@HansenPartnership.com>,
Giridhar Malavali <giridhar.malavali@qlogic.com>,
Linux SCSI Mailing List <linux-scsi@vger.kernel.org>,
Joe Carnuccio <joe.carnuccio@qlogic.com>
Subject: Re: [PATCH 10/15] qla2xxx: Perform an implicit login to the Management Server.
Date: Tue, 7 Apr 2009 11:02:55 -0400 [thread overview]
Message-ID: <20090407150255.GA29910@mars.virtualiron.com> (raw)
In-Reply-To: <1239082431-19496-11-git-send-email-andrew.vasquez@qlogic.com>
On Mon, Apr 06, 2009 at 10:33:46PM -0700, Andrew Vasquez wrote:
> From: Joe Carnuccio <joe.carnuccio@qlogic.com>
>
> Set the conditional plogi option bit whenever logging in the
> fabric management server (if it is already logged in, it does not
> need an explicit login; an implicit login suffices).
How does one find out if it already has logged in? Obviously this routine
is used during the initialization for the driver, which implies that
the firmware has done the job of logging already, right? Wouldn't the
management_server_logged_in flag be set somewhere by the login code, say
by the asynchronous updates?
>
> Signed-off-by: Andrew Vasquez <andrew.vasquez@qlogic.com>
> ---
> drivers/scsi/qla2xxx/qla_gs.c | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/drivers/scsi/qla2xxx/qla_gs.c b/drivers/scsi/qla2xxx/qla_gs.c
> index 557f58d..3dbb9e7 100644
> --- a/drivers/scsi/qla2xxx/qla_gs.c
> +++ b/drivers/scsi/qla2xxx/qla_gs.c
> @@ -1107,7 +1107,7 @@ qla2x00_mgmt_svr_login(scsi_qla_host_t *vha)
> return ret;
>
> ha->isp_ops->fabric_login(vha, vha->mgmt_svr_loop_id, 0xff, 0xff, 0xfa,
> - mb, BIT_1);
> + mb, BIT_1|BIT_0);
> if (mb[0] != MBS_COMMAND_COMPLETE) {
> DEBUG2_13(printk("%s(%ld): Failed MANAGEMENT_SERVER login: "
> "loop_id=%x mb[0]=%x mb[1]=%x mb[2]=%x mb[6]=%x mb[7]=%x\n",
> --
> 1.6.2.2.404.ge96f3
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
next prev parent reply other threads:[~2009-04-07 15:05 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-04-07 5:33 [PATCH 00/15] qla2xxx: updates for 2.6.30 [8.03.01-k2] Andrew Vasquez
2009-04-07 5:33 ` [PATCH 01/15] qla2xxx: Correct bus-reset behaviour with recent ISPs Andrew Vasquez
2009-04-07 5:33 ` [PATCH 02/15] qla2xxx: Export additional FCoE attributes for application support Andrew Vasquez
2009-04-11 7:19 ` Joe Eykholt
2009-04-07 5:33 ` [PATCH 03/15] qla2xxx: Correct compilation failures when DEBUG'n' options are enabled Andrew Vasquez
2009-04-07 5:33 ` [PATCH 04/15] qla2xxx: Add QoS support Andrew Vasquez
2009-04-07 5:33 ` [PATCH 05/15] qla2xxx: Add CPU affinity support Andrew Vasquez
2009-04-07 5:33 ` [PATCH 06/15] qla2xxx: Remove reference to request queue from scsi request block Andrew Vasquez
2009-04-07 5:33 ` [PATCH 07/15] qla2xxx: Correct typo in read_nvram() callback Andrew Vasquez
2009-04-07 5:33 ` [PATCH 08/15] qla2xxx: Correct hard-coded address of a second-port's NVRAM Andrew Vasquez
2009-04-07 5:33 ` [PATCH 09/15] qla2xxx: Restrict model-name/description device-table usage Andrew Vasquez
2009-04-07 5:33 ` [PATCH 10/15] qla2xxx: Perform an implicit login to the Management Server Andrew Vasquez
2009-04-07 15:02 ` Konrad Rzeszutek [this message]
2009-04-07 15:24 ` Joe Carnuccio
2009-04-07 5:33 ` [PATCH 11/15] qla2xxx: Conditionally disable automatic queue full tracking Andrew Vasquez
2009-04-07 5:33 ` [PATCH 12/15] qla2xxx: Don't try to 'stop' firmware if already in ROM code Andrew Vasquez
2009-04-07 5:33 ` [PATCH 13/15] qla2xxx: Add an override option to specify ISP firmware load semantics Andrew Vasquez
2009-04-07 5:33 ` [PATCH 14/15] qla2xxx: Use port number to compute nvram/vpd parameter offsets Andrew Vasquez
2009-04-07 5:33 ` [PATCH 15/15] qla2xxx: Update version number to 8.03.01-k2 Andrew Vasquez
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=20090407150255.GA29910@mars.virtualiron.com \
--to=konrad@virtualiron.com \
--cc=James.Bottomley@HansenPartnership.com \
--cc=andrew.vasquez@qlogic.com \
--cc=giridhar.malavali@qlogic.com \
--cc=joe.carnuccio@qlogic.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.