aboutsummaryrefslogtreecommitdiffstats
path: root/drivers/ieee1394/sbp2.c
diff options
context:
space:
mode:
Diffstat (limited to 'drivers/ieee1394/sbp2.c')
-rw-r--r--drivers/ieee1394/sbp2.c21
1 files changed, 11 insertions, 10 deletions
diff --git a/drivers/ieee1394/sbp2.c b/drivers/ieee1394/sbp2.c
index 3cb6b479b2ef..017259cc34f3 100644
--- a/drivers/ieee1394/sbp2.c
+++ b/drivers/ieee1394/sbp2.c
@@ -478,7 +478,7 @@ static void sbp2util_notify_fetch_agent(struct scsi_id_instance_data *scsi_id,
478 * There is a small window after a bus reset within which the node 478 * There is a small window after a bus reset within which the node
479 * entry's generation is current but the reconnect wasn't completed. 479 * entry's generation is current but the reconnect wasn't completed.
480 */ 480 */
481 if (atomic_read(&scsi_id->unfinished_reset)) 481 if (unlikely(atomic_read(&scsi_id->state) == SBP2LU_STATE_IN_RESET))
482 return; 482 return;
483 483
484 if (hpsb_node_write(scsi_id->ne, 484 if (hpsb_node_write(scsi_id->ne,
@@ -489,7 +489,7 @@ static void sbp2util_notify_fetch_agent(struct scsi_id_instance_data *scsi_id,
489 * Now accept new SCSI commands, unless a bus reset happended during 489 * Now accept new SCSI commands, unless a bus reset happended during
490 * hpsb_node_write. 490 * hpsb_node_write.
491 */ 491 */
492 if (!atomic_read(&scsi_id->unfinished_reset)) 492 if (likely(atomic_read(&scsi_id->state) != SBP2LU_STATE_IN_RESET))
493 scsi_unblock_requests(scsi_id->scsi_host); 493 scsi_unblock_requests(scsi_id->scsi_host);
494} 494}
495 495
@@ -756,7 +756,7 @@ static int sbp2_remove(struct device *dev)
756 sbp2scsi_complete_all_commands(scsi_id, DID_NO_CONNECT); 756 sbp2scsi_complete_all_commands(scsi_id, DID_NO_CONNECT);
757 /* scsi_remove_device() will trigger shutdown functions of SCSI 757 /* scsi_remove_device() will trigger shutdown functions of SCSI
758 * highlevel drivers which would deadlock if blocked. */ 758 * highlevel drivers which would deadlock if blocked. */
759 atomic_set(&scsi_id->unfinished_reset, 0); 759 atomic_set(&scsi_id->state, SBP2LU_STATE_IN_SHUTDOWN);
760 scsi_unblock_requests(scsi_id->scsi_host); 760 scsi_unblock_requests(scsi_id->scsi_host);
761 } 761 }
762 sdev = scsi_id->sdev; 762 sdev = scsi_id->sdev;
@@ -811,7 +811,7 @@ static int sbp2_update(struct unit_directory *ud)
811 /* Accept new commands unless there was another bus reset in the 811 /* Accept new commands unless there was another bus reset in the
812 * meantime. */ 812 * meantime. */
813 if (hpsb_node_entry_valid(scsi_id->ne)) { 813 if (hpsb_node_entry_valid(scsi_id->ne)) {
814 atomic_set(&scsi_id->unfinished_reset, 0); 814 atomic_set(&scsi_id->state, SBP2LU_STATE_RUNNING);
815 scsi_unblock_requests(scsi_id->scsi_host); 815 scsi_unblock_requests(scsi_id->scsi_host);
816 } 816 }
817 return 0; 817 return 0;
@@ -842,7 +842,7 @@ static struct scsi_id_instance_data *sbp2_alloc_device(struct unit_directory *ud
842 INIT_LIST_HEAD(&scsi_id->sbp2_command_orb_completed); 842 INIT_LIST_HEAD(&scsi_id->sbp2_command_orb_completed);
843 INIT_LIST_HEAD(&scsi_id->scsi_list); 843 INIT_LIST_HEAD(&scsi_id->scsi_list);
844 spin_lock_init(&scsi_id->sbp2_command_orb_lock); 844 spin_lock_init(&scsi_id->sbp2_command_orb_lock);
845 atomic_set(&scsi_id->unfinished_reset, 0); 845 atomic_set(&scsi_id->state, SBP2LU_STATE_RUNNING);
846 INIT_WORK(&scsi_id->protocol_work, NULL, NULL); 846 INIT_WORK(&scsi_id->protocol_work, NULL, NULL);
847 847
848 ud->device.driver_data = scsi_id; 848 ud->device.driver_data = scsi_id;
@@ -926,13 +926,14 @@ static void sbp2_host_reset(struct hpsb_host *host)
926 struct scsi_id_instance_data *scsi_id; 926 struct scsi_id_instance_data *scsi_id;
927 927
928 hi = hpsb_get_hostinfo(&sbp2_highlevel, host); 928 hi = hpsb_get_hostinfo(&sbp2_highlevel, host);
929 929 if (!hi)
930 if (hi) { 930 return;
931 list_for_each_entry(scsi_id, &hi->scsi_ids, scsi_list) { 931 list_for_each_entry(scsi_id, &hi->scsi_ids, scsi_list)
932 atomic_set(&scsi_id->unfinished_reset, 1); 932 if (likely(atomic_read(&scsi_id->state) !=
933 SBP2LU_STATE_IN_SHUTDOWN)) {
934 atomic_set(&scsi_id->state, SBP2LU_STATE_IN_RESET);
933 scsi_block_requests(scsi_id->scsi_host); 935 scsi_block_requests(scsi_id->scsi_host);
934 } 936 }
935 }
936} 937}
937 938
938/* 939/*