Hello Mike,

The following snippet from the 39d4ceb04f051c208ae7509d268a3871ffa194c5
commit is preventing bnx2i from being able to offload when connecting
through the iscsi_sysfs_get_hostinfo_by_host_no code path where the
session argument == NULL.  The initiatorname sysfs inquiry failed but
this is expected since it does not exist in the iscsi_host for bnx2i.

Perhaps the error return code should only be propagated upward when
session != NULL specifically for hbas like qla4xxx?

Thanks,
Eddie

----- 8< ----- 8< -----
@@ -527,7 +528,10 @@ static int iscsi_sysfs_read_iface(struct iface_rec
*iface, int host_no,
                                          iface_str(iface));
                }
        }
-       return ret;
+       if (ret)
+               return ISCSI_ERR_SYSFS_LOOKUP;
+       else
+               return 0;
 }
----- 8< ----- 8< -----


Author: Mike Christie <[email protected]>
Date:   Sun Jan 30 22:49:56 2011 -0600

    iscsi tools: fix iscsiadm exit codes
    
    iscsiadm/iscsistart return a mix of type of returns codes.
    Sometimes -1 and sometimes a EXYZ type of error code. This
    patch has them return a ISCSI_ERR value.
    
    See iscsiadm man page EXIT STATUS section for error
    code definitions.

commit 39d4ceb04f051c208ae7509d268a3871ffa194c5



-- 
You received this message because you are subscribed to the Google Groups 
"open-iscsi" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/open-iscsi?hl=en.

Reply via email to