From c132f692085ac624d7c8123df781846c8dcb3166 Mon Sep 17 00:00:00 2001 From: Dan Williams Date: Tue, 3 Jan 2012 23:26:08 -0800 Subject: [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 [dmilburn@redhat.com: fix i686 compile error] Signed-off-by: Dan Williams Signed-off-by: James Bottomley --- drivers/scsi/isci/port.h | 2 -- 1 file changed, 2 deletions(-) (limited to 'drivers/scsi/isci/port.h') diff --git a/drivers/scsi/isci/port.h b/drivers/scsi/isci/port.h index 08116090eb7..6b42e84b695 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; -- cgit v1.2.3-70-g09d2