diff options
author | Jeff Garzik <jgarzik@pobox.com> | 2005-08-25 22:01:20 -0400 |
---|---|---|
committer | Jeff Garzik <jgarzik@pobox.com> | 2005-08-25 22:01:20 -0400 |
commit | b8f6153ee421014f42b620238f4203a4106db309 (patch) | |
tree | ebc9eac4517325be0a91a4e2b2cb38f08d36812e /drivers/scsi/sata_promise.c | |
parent | 617e44fdfd7ee3d53388ab295d9411b826437ce9 (diff) |
libata: fix EH locking
Wrap ata_qc_complete() calls in EH context in spinlocks, to prevent
races (mainly in ATAPI code paths).
Diffstat (limited to 'drivers/scsi/sata_promise.c')
-rw-r--r-- | drivers/scsi/sata_promise.c | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/drivers/scsi/sata_promise.c b/drivers/scsi/sata_promise.c index defcc1fb3f16..b8dc49fed769 100644 --- a/drivers/scsi/sata_promise.c +++ b/drivers/scsi/sata_promise.c | |||
@@ -325,11 +325,15 @@ static void pdc_qc_prep(struct ata_queued_cmd *qc) | |||
325 | 325 | ||
326 | static void pdc_eng_timeout(struct ata_port *ap) | 326 | static void pdc_eng_timeout(struct ata_port *ap) |
327 | { | 327 | { |
328 | struct ata_host_set *host_set = ap->host_set; | ||
328 | u8 drv_stat; | 329 | u8 drv_stat; |
329 | struct ata_queued_cmd *qc; | 330 | struct ata_queued_cmd *qc; |
331 | unsigned long flags; | ||
330 | 332 | ||
331 | DPRINTK("ENTER\n"); | 333 | DPRINTK("ENTER\n"); |
332 | 334 | ||
335 | spin_lock_irqsave(&host_set->lock, flags); | ||
336 | |||
333 | qc = ata_qc_from_tag(ap, ap->active_tag); | 337 | qc = ata_qc_from_tag(ap, ap->active_tag); |
334 | if (!qc) { | 338 | if (!qc) { |
335 | printk(KERN_ERR "ata%u: BUG: timeout without command\n", | 339 | printk(KERN_ERR "ata%u: BUG: timeout without command\n", |
@@ -363,6 +367,7 @@ static void pdc_eng_timeout(struct ata_port *ap) | |||
363 | } | 367 | } |
364 | 368 | ||
365 | out: | 369 | out: |
370 | spin_unlock_irqrestore(&host_set->lock, flags); | ||
366 | DPRINTK("EXIT\n"); | 371 | DPRINTK("EXIT\n"); |
367 | } | 372 | } |
368 | 373 | ||