diff options
author | Dan Williams <dan.j.williams@intel.com> | 2012-01-03 23:26:08 -0800 |
---|---|---|
committer | James Bottomley <JBottomley@Parallels.com> | 2012-02-29 15:05:05 -0600 |
commit | c132f692085ac624d7c8123df781846c8dcb3166 (patch) | |
tree | c9009fcb9112812544f54efc6c134021dffb6545 /drivers/scsi/isci/port.h | |
parent | cb48d672bfeb4147f3e20d5b16fa2eb6377231e2 (diff) | |
download | op-kernel-dev-c132f692085ac624d7c8123df781846c8dcb3166.zip op-kernel-dev-c132f692085ac624d7c8123df781846c8dcb3166.tar.gz |
[SCSI] isci: kill iphy->isci_port lookups
This field is a holdover from the OS abstraction conversion. The stable
phy to port lookups are done via iphy->ownining_port under scic_lock.
After this conversion to use port->lldd_port the only volatile lookup is
the initial lookup in isci_port_formed(). After that point any lookup
via a successfully notified domain_device is guaranteed to be valid
until the domain_device is destroyed.
Delete ->start_complete as it is only set once and is set as a
consequence of the port going link up, by definition of getting a port
formed event the port is "ready".
While we are correcting port lookups also move the asd_sas_port table
out from under the isci_port. This is to preclude any temptation to use
container_of() to convert an asd_sas_port to an isci_port, the
association is dynamic and under libsas control.
Tested-by: Maciej Trela <maciej.trela@intel.com>
[dmilburn@redhat.com: fix i686 compile error]
Signed-off-by: Dan Williams <dan.j.williams@intel.com>
Signed-off-by: James Bottomley <JBottomley@Parallels.com>
Diffstat (limited to 'drivers/scsi/isci/port.h')
-rw-r--r-- | drivers/scsi/isci/port.h | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/drivers/scsi/isci/port.h b/drivers/scsi/isci/port.h index 0811609..6b42e84 100644 --- a/drivers/scsi/isci/port.h +++ b/drivers/scsi/isci/port.h @@ -97,11 +97,9 @@ enum isci_status { struct isci_port { enum isci_status status; struct isci_host *isci_host; - struct asd_sas_port sas_port; struct list_head remote_dev_list; spinlock_t state_lock; struct list_head domain_dev_list; - struct completion start_complete; struct completion hard_reset_complete; enum sci_status hard_reset_status; struct sci_base_state_machine sm; |