diff options
| author | Andrew Vasquez <andrew.vasquez@qlogic.com> | 2006-02-14 17:22:14 -0500 |
|---|---|---|
| committer | James Bottomley <jejb@mulgrave.il.steeleye.com> | 2006-02-27 22:25:40 -0500 |
| commit | 8b097a67264ba3e10620b268979de3be6fe5e3cd (patch) | |
| tree | a014abfb3e58ddf7b71e1c37dbecdac44020a312 /drivers/scsi/scsi_transport_fc.c | |
| parent | eca7be5e1899626db01ae42b0123458d6fb34930 (diff) | |
[SCSI] fc_transport: stop creating duplicate rport entries.
Current fc_transport consumers initially register rports
with an UNKNOWN role-state and follow-up with a call to
fc_remote_port_rolechg(). Modify code in
fc_remote_port_add() to scan the fc_host_rport_bindings()
array for consistent bindings regardless of role-type.
Original code would only scan bindings array for targets,
causing duplicate fc_remote_ports/rport-X:Y-Z entries to be
created for the yet-to-be-role-changed rports.
Signed-off-by: Andrew Vasquez <andrew.vasquez@qlogic.com>
Signed-off-by: James Bottomley <James.Bottomley@SteelEye.com>
Diffstat (limited to 'drivers/scsi/scsi_transport_fc.c')
| -rw-r--r-- | drivers/scsi/scsi_transport_fc.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/scsi/scsi_transport_fc.c b/drivers/scsi/scsi_transport_fc.c index f2c9acf11bd0..929032e370db 100644 --- a/drivers/scsi/scsi_transport_fc.c +++ b/drivers/scsi/scsi_transport_fc.c | |||
| @@ -1498,8 +1498,7 @@ fc_remote_port_add(struct Scsi_Host *shost, int channel, | |||
| 1498 | } | 1498 | } |
| 1499 | 1499 | ||
| 1500 | /* Search the bindings array */ | 1500 | /* Search the bindings array */ |
| 1501 | if (likely((ids->roles & FC_RPORT_ROLE_FCP_TARGET) && | 1501 | if (fc_host_tgtid_bind_type(shost) != FC_TGTID_BIND_NONE) { |
| 1502 | (fc_host_tgtid_bind_type(shost) != FC_TGTID_BIND_NONE))) { | ||
| 1503 | 1502 | ||
| 1504 | /* search for a matching consistent binding */ | 1503 | /* search for a matching consistent binding */ |
| 1505 | 1504 | ||