Michael Reed wrote: > > James Smart wrote: >>Good Catch! >> >>-- james s >> >>Andrew Vasquez wrote: >>>Current fc_transport consumers initially register rports >>>with an UNKNOWN role-state and follow-up with a call to >>>fc_remote_port_rolechg(). > > So, is the above sequence necessary? Why not just register > the rport with appropriate role and state? This is what > LSI fusion driver (mptfc) does. "roles" has target and > initiator set. (Is fusion broken?) WHAT AM I SAYING? roles is passed in with "target" set unless the target is also an initiator. Mike > > (Not saying the fix shouldn't be there.) > > Mike > > <...snip...> > > - > : 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 > > - : 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