Re: [PATCH] thaw FC local port

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]


On Thu, 2010-12-16 at 08:07 -0800, Hillf Danton wrote:
> The state machine of local port is freezed if it could not get ready,
> so thawing is necessary.
> 
Please start putting a prefix "fcoe: ", "libfc: " or "libfcoe: " on your
patch titles. Also, please provide more meaningful commit messages. I
believe that I've pointed out these two issues with your patches before.

> Signed-off-by: Hillf Danton <dhillf@xxxxxxxxx>
> ---
> 
> --- a/drivers/scsi/libfc/fc_lport.c	2010-11-01 19:54:12.000000000 +0800
> +++ b/drivers/scsi/libfc/fc_lport.c	2010-12-17 00:08:06.000000000 +0800
> @@ -1514,11 +1514,11 @@ void fc_lport_enter_flogi(struct fc_lpor
> 
>  	fc_lport_state_enter(lport, LPORT_ST_FLOGI);
> 
> -	if (lport->point_to_multipoint) {
> -		if (lport->port_id)
> +	if (lport->point_to_multipoint)
> +		if (lport->port_id) {
>  			fc_lport_enter_ready(lport);
> -		return;
> -	}
> +			return;
> +		}
> 
>  	fp = fc_frame_alloc(lport, sizeof(struct fc_els_flogi));
>  	if (!fp)

We do not want to proceed with sending a FLOGI here when we're in
point-to-multipoint (VN2VN) mode. VN2VN is a mode where there is no
fabric and we FLOGI directly into the remote port. So for VN2VN the
FLOGI happens in the rport layer.

//Rob

--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[SCSI Target Devel]     [Linux SCSI Target Infrastructure]     [Kernel Newbies]     [Share Photos]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Photos]     [Yosemite]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Linux IIO]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]

Add to Google Powered by Linux