diff options
author | Dan Williams <dan.j.williams@intel.com> | 2012-01-12 14:47:24 -0500 |
---|---|---|
committer | James Bottomley <JBottomley@Parallels.com> | 2012-02-29 16:27:09 -0500 |
commit | 5d7f6d1071eadd020edb2cf366d358e0f6d0a0f9 (patch) | |
tree | 4c3d4c7a22ee37a61edb12ab76f3a37ba685e8b6 | |
parent | ab5266335ba1a43461443f9823276a2b44dd1ba7 (diff) |
[SCSI] libsas: fix sas_unregister_ports vs sas_drain_work
We need to hold drain_mutex across the unregistration as port down events
queue device removal as chained events, so we need to make sure no other
drainers are active.
[ 1118.673968] WARNING: at kernel/workqueue.c:996 __queue_work+0x11a/0x326()
[ 1118.681982] Hardware name: S2600CP
[ 1118.686193] Modules linked in: isci(-) libsas scsi_transport_sas nls_utf8
ipv6 uinput sg iTCO_wdt iTCO_vendor_support i2c_i801 i2c_core ioatdma dca
sd_mod sr_mod cdrom ahci libahci libata [last unloaded: scsi_transport_sas]
[ 1118.709893] Pid: 6831, comm: rmmod Not tainted 3.2.0-isci+ #1
[ 1118.716727] Call Trace:
[ 1118.719867] [<ffffffff8103e9f5>] warn_slowpath_common+0x85/0x9d
[ 1118.727000] [<ffffffff8103ea27>] warn_slowpath_null+0x1a/0x1c
[ 1118.733942] [<ffffffff81056d44>] __queue_work+0x11a/0x326
[ 1118.740481] [<ffffffff81056f99>] queue_work_on+0x1b/0x22
[ 1118.746925] [<ffffffff81057106>] queue_work+0x37/0x3e
[ 1118.753105] [<ffffffffa0120e05>] ? sas_discover_event+0x55/0x82 [libsas]
[ 1118.761094] [<ffffffff813217c3>] scsi_queue_work+0x42/0x44
[ 1118.767717] [<ffffffffa0120e19>] sas_discover_event+0x69/0x82 [libsas]
[ 1118.775509] [<ffffffffa0120f5b>] sas_unregister_dev+0xc3/0xcc [libsas]
[ 1118.783319] [<ffffffffa0120fae>] sas_unregister_domain_devices+0x4a/0xc8 [libsas]
[ 1118.792731] [<ffffffffa0120071>] sas_deform_port+0x60/0x1a6 [libsas]
[ 1118.800339] [<ffffffffa01201ea>] sas_unregister_ports+0x33/0x44 [libsas]
[ 1118.808342] [<ffffffffa011f7e5>] sas_unregister_ha+0x41/0x6b [libsas]
[ 1118.816055] [<ffffffffa0134055>] isci_unregister+0x22/0x4d [isci]
[ 1118.823384] [<ffffffffa0143040>] isci_pci_remove+0x2e/0x60 [isci]
Reported-by: Jacek Danecki <jacek.danecki@intel.com>
Signed-off-by: Dan Williams <dan.j.williams@intel.com>
Signed-off-by: James Bottomley <JBottomley@Parallels.com>
-rw-r--r-- | drivers/scsi/libsas/sas_event.c | 19 | ||||
-rw-r--r-- | drivers/scsi/libsas/sas_init.c | 18 | ||||
-rw-r--r-- | drivers/scsi/libsas/sas_internal.h | 1 |
3 files changed, 25 insertions, 13 deletions
diff --git a/drivers/scsi/libsas/sas_event.c b/drivers/scsi/libsas/sas_event.c index dbfaceeea0f7..16639bbae629 100644 --- a/drivers/scsi/libsas/sas_event.c +++ b/drivers/scsi/libsas/sas_event.c | |||
@@ -51,15 +51,11 @@ static void sas_queue_event(int event, unsigned long *pending, | |||
51 | } | 51 | } |
52 | } | 52 | } |
53 | 53 | ||
54 | int sas_drain_work(struct sas_ha_struct *ha) | 54 | |
55 | void __sas_drain_work(struct sas_ha_struct *ha) | ||
55 | { | 56 | { |
56 | struct workqueue_struct *wq = ha->core.shost->work_q; | 57 | struct workqueue_struct *wq = ha->core.shost->work_q; |
57 | struct work_struct *w, *_w; | 58 | struct work_struct *w, *_w; |
58 | int err; | ||
59 | |||
60 | err = mutex_lock_interruptible(&ha->drain_mutex); | ||
61 | if (err) | ||
62 | return err; | ||
63 | 59 | ||
64 | set_bit(SAS_HA_DRAINING, &ha->state); | 60 | set_bit(SAS_HA_DRAINING, &ha->state); |
65 | /* flush submitters */ | 61 | /* flush submitters */ |
@@ -75,6 +71,17 @@ int sas_drain_work(struct sas_ha_struct *ha) | |||
75 | sas_queue_work(ha, w); | 71 | sas_queue_work(ha, w); |
76 | } | 72 | } |
77 | spin_unlock_irq(&ha->state_lock); | 73 | spin_unlock_irq(&ha->state_lock); |
74 | } | ||
75 | |||
76 | int sas_drain_work(struct sas_ha_struct *ha) | ||
77 | { | ||
78 | int err; | ||
79 | |||
80 | err = mutex_lock_interruptible(&ha->drain_mutex); | ||
81 | if (err) | ||
82 | return err; | ||
83 | if (test_bit(SAS_HA_REGISTERED, &ha->state)) | ||
84 | __sas_drain_work(ha); | ||
78 | mutex_unlock(&ha->drain_mutex); | 85 | mutex_unlock(&ha->drain_mutex); |
79 | 86 | ||
80 | return 0; | 87 | return 0; |
diff --git a/drivers/scsi/libsas/sas_init.c b/drivers/scsi/libsas/sas_init.c index dc93e1181469..09c14ca3fbd5 100644 --- a/drivers/scsi/libsas/sas_init.c +++ b/drivers/scsi/libsas/sas_init.c | |||
@@ -160,18 +160,22 @@ Undo_phys: | |||
160 | 160 | ||
161 | int sas_unregister_ha(struct sas_ha_struct *sas_ha) | 161 | int sas_unregister_ha(struct sas_ha_struct *sas_ha) |
162 | { | 162 | { |
163 | unsigned long flags; | ||
164 | |||
165 | /* Set the state to unregistered to avoid further unchained | 163 | /* Set the state to unregistered to avoid further unchained |
166 | * events to be queued | 164 | * events to be queued, and flush any in-progress drainers |
167 | */ | 165 | */ |
168 | spin_lock_irqsave(&sas_ha->state_lock, flags); | 166 | mutex_lock(&sas_ha->drain_mutex); |
167 | spin_lock_irq(&sas_ha->state_lock); | ||
169 | clear_bit(SAS_HA_REGISTERED, &sas_ha->state); | 168 | clear_bit(SAS_HA_REGISTERED, &sas_ha->state); |
170 | spin_unlock_irqrestore(&sas_ha->state_lock, flags); | 169 | spin_unlock_irq(&sas_ha->state_lock); |
171 | sas_drain_work(sas_ha); | 170 | __sas_drain_work(sas_ha); |
171 | mutex_unlock(&sas_ha->drain_mutex); | ||
172 | 172 | ||
173 | sas_unregister_ports(sas_ha); | 173 | sas_unregister_ports(sas_ha); |
174 | sas_drain_work(sas_ha); | 174 | |
175 | /* flush unregistration work */ | ||
176 | mutex_lock(&sas_ha->drain_mutex); | ||
177 | __sas_drain_work(sas_ha); | ||
178 | mutex_unlock(&sas_ha->drain_mutex); | ||
175 | 179 | ||
176 | if (sas_ha->lldd_max_execute_num > 1) { | 180 | if (sas_ha->lldd_max_execute_num > 1) { |
177 | sas_shutdown_queue(sas_ha); | 181 | sas_shutdown_queue(sas_ha); |
diff --git a/drivers/scsi/libsas/sas_internal.h b/drivers/scsi/libsas/sas_internal.h index 4157f6e1eda2..7818c4673c3a 100644 --- a/drivers/scsi/libsas/sas_internal.h +++ b/drivers/scsi/libsas/sas_internal.h | |||
@@ -70,6 +70,7 @@ int sas_init_events(struct sas_ha_struct *sas_ha); | |||
70 | void sas_shutdown_queue(struct sas_ha_struct *sas_ha); | 70 | void sas_shutdown_queue(struct sas_ha_struct *sas_ha); |
71 | void sas_disable_revalidation(struct sas_ha_struct *ha); | 71 | void sas_disable_revalidation(struct sas_ha_struct *ha); |
72 | void sas_enable_revalidation(struct sas_ha_struct *ha); | 72 | void sas_enable_revalidation(struct sas_ha_struct *ha); |
73 | void __sas_drain_work(struct sas_ha_struct *ha); | ||
73 | 74 | ||
74 | void sas_deform_port(struct asd_sas_phy *phy, int gone); | 75 | void sas_deform_port(struct asd_sas_phy *phy, int gone); |
75 | 76 | ||