On Friday, August 25, 2006 12:48 PM, James Bottomley wrote: > -----Original Message----- > From: James Bottomley [mailto:James.Bottomley@xxxxxxxxxxxx] > Sent: Friday, August 25, 2006 12:48 PM > To: linux-scsi > Cc: Moore, Eric > Subject: [PATCH] scsi_transport_sas: remove local_attached flag > > This flag denotes local attachment of the phy. There are two problems > with it: > > 1) It's actually redundant ... you can get the same information simply > by seeing whether a host is the phys parent > 2) we condition a lot of phy parameters on it on the false assumption > that we can only control local phys. I'm wiring up phy resets in the > aic94xx now, and it will be able to reset non-local phys as well. > > I fixed 2) by moving the local check into the reset and stats function > of the mptsas, since that seems to be the only HBA that can't > (currently) control non-local phys. > This makes sense to remove this flag. Perhaps this is time to decide on how to handle this for expanders. At the time this was being developed in transport via Christophs help, we couldn't decide if this should be done by userspace, smp_passthru, or fusion internal implementation. Currently you can get link errors or phy/link reset via Doug Gilberts smputils. For instance you can get link errors using smp_rep_phy_err_log, and send link/phy reset via smp_phy_control. If the tools are fine with you. Then nothing needs to be done. I could add support in fusion for sending internal smp passthru, or we can export a smp portal for you. The smp portal should still be in the future plans for the bsg driver. Eric - 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
