Anil Veerabhadrappa wrote:
>     * When using iface, bnx2i was unable to offload further connections after
>       all active sessions are logged out. bnx2i will unregister the device 
> from
>       cnic when the last connection is torn down. Next call to ep_connect()
>       will fail because the device is not registered. This issue is not seen
>       if shost == NULL is passed to ep_connect() call because in that case 
> bnx2i
>       will registers all known devices with cnic before doing a route look-up.
>       When shost != NULL, bnx2i knows the device on which to offload the
>       connection and has to register this device before attempting to offload
>       the connection
> 
> Signed-off-by: Anil Veerabhadrappa <ani...@broadcom.com>
> Reviewed-by: Michael Chan <mc...@broadcom.com>
> ---
>  drivers/scsi/bnx2i/bnx2i_init.c  |    7 +++++--
>  drivers/scsi/bnx2i/bnx2i_iscsi.c |    7 +++++--
>  2 files changed, 10 insertions(+), 4 deletions(-)
> 
> diff --git a/drivers/scsi/bnx2i/bnx2i_init.c b/drivers/scsi/bnx2i/bnx2i_init.c
> index fd78540..0c4210d 100644
> --- a/drivers/scsi/bnx2i/bnx2i_init.c
> +++ b/drivers/scsi/bnx2i/bnx2i_init.c
> @@ -185,14 +185,17 @@ void bnx2i_stop(void *handle)
>   */
>  void bnx2i_register_device(struct bnx2i_hba *hba)
>  {
> +     int rc;
> +
>       if (test_bit(ADAPTER_STATE_GOING_DOWN, &hba->adapter_state) ||
>           test_bit(BNX2I_CNIC_REGISTERED, &hba->reg_with_cnic)) {
>               return;
>       }
>  
> -     hba->cnic->register_device(hba->cnic, CNIC_ULP_ISCSI, hba);
> +     rc = hba->cnic->register_device(hba->cnic, CNIC_ULP_ISCSI, hba);
>  
> -     set_bit(BNX2I_CNIC_REGISTERED, &hba->reg_with_cnic);
> +     if (!rc)
> +             set_bit(BNX2I_CNIC_REGISTERED, &hba->reg_with_cnic);
>  }
>  
>  
> diff --git a/drivers/scsi/bnx2i/bnx2i_iscsi.c 
> b/drivers/scsi/bnx2i/bnx2i_iscsi.c
> index f741219..98148f3 100644
> --- a/drivers/scsi/bnx2i/bnx2i_iscsi.c
> +++ b/drivers/scsi/bnx2i/bnx2i_iscsi.c
> @@ -1653,15 +1653,18 @@ static struct iscsi_endpoint *bnx2i_ep_connect(struct 
> Scsi_Host *shost,
>       struct iscsi_endpoint *ep;
>       int rc = 0;
>  
> -     if (shost)
> +     if (shost) {
>               /* driver is given scsi host to work with */
>               hba = iscsi_host_priv(shost);
> -     else
> +             /* Register the device with cnic if not already done so */
> +             bnx2i_register_device(hba);
> +     } else
>               /*
>                * check if the given destination can be reached through
>                * a iscsi capable NetXtreme2 device
>                */
>               hba = bnx2i_check_route(dst_addr);
> +
>       if (!hba) {
>               rc = -ENOMEM;
>               goto check_busy;

Do you want to do the test_bit(BNX2I_CNIC_REGISTERED, 
&hba->reg_with_cnic) test here instead of down below. If it failed then 
you might try to do a cm_create on a cnic that is not properly registered.

--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups 
"open-iscsi" group.
To post to this group, send email to open-iscsi@googlegroups.com
To unsubscribe from this group, send email to 
open-iscsi+unsubscr...@googlegroups.com
For more options, visit this group at http://groups.google.com/group/open-iscsi
-~----------~----~----~----~------~----~------~--~---

Reply via email to