diff options
| author | Stefan Richter <stefanr@s5r6.in-berlin.de> | 2008-02-03 23:10:47 +0100 |
|---|---|---|
| committer | Stefan Richter <stefanr@s5r6.in-berlin.de> | 2008-02-16 15:40:35 +0100 |
| commit | 0fa6dfdb0a2768541e998a5dab10b368de56c60a (patch) | |
| tree | 3fac0c712c15107247c029f1802673e6553e57b7 /drivers/macintosh/macio-adb.c | |
| parent | 48f18c761c001a66ef1928b42799c717368b1d64 (diff) | |
firewire: fw-sbp2: don't add scsi_device twice
When a reconnect failed but re-login succeeded, __scsi_add_device was
called again.
In those cases, __scsi_add_device succeeded and returned the pointer to
the existing scsi_device. fw-sbp2 then continued orderly, except that
it missed to call sbp2_cancel_orbs. SCSI core would call fw-sbp2's
eh_abort_handler eventually if there had been an outstanding command.
This patch avoids the needless lookups and temporary allocations in SCSI
core and I/O stall and timeout until eh_abort_handler hits.
Also, __scsi_add_device tolerating calls for devices which already exist
is undocumented behavior on which we shouldn't rely.
Signed-off-by: Stefan Richter <stefanr@s5r6.in-berlin.de>
Signed-off-by: Jarod Wilson <jwilson@redhat.com>
Diffstat (limited to 'drivers/macintosh/macio-adb.c')
0 files changed, 0 insertions, 0 deletions