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_sx4.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_sx4.c')
-rw-r--r-- | drivers/scsi/sata_sx4.c | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/drivers/scsi/sata_sx4.c b/drivers/scsi/sata_sx4.c index e2db499f22dd..a20d4285090a 100644 --- a/drivers/scsi/sata_sx4.c +++ b/drivers/scsi/sata_sx4.c | |||
@@ -848,10 +848,14 @@ static irqreturn_t pdc20621_interrupt (int irq, void *dev_instance, struct pt_re | |||
848 | static void pdc_eng_timeout(struct ata_port *ap) | 848 | static void pdc_eng_timeout(struct ata_port *ap) |
849 | { | 849 | { |
850 | u8 drv_stat; | 850 | u8 drv_stat; |
851 | struct ata_host_set *host_set = ap->host_set; | ||
851 | struct ata_queued_cmd *qc; | 852 | struct ata_queued_cmd *qc; |
853 | unsigned long flags; | ||
852 | 854 | ||
853 | DPRINTK("ENTER\n"); | 855 | DPRINTK("ENTER\n"); |
854 | 856 | ||
857 | spin_lock_irqsave(&host_set->lock, flags); | ||
858 | |||
855 | qc = ata_qc_from_tag(ap, ap->active_tag); | 859 | qc = ata_qc_from_tag(ap, ap->active_tag); |
856 | if (!qc) { | 860 | if (!qc) { |
857 | printk(KERN_ERR "ata%u: BUG: timeout without command\n", | 861 | printk(KERN_ERR "ata%u: BUG: timeout without command\n", |
@@ -885,6 +889,7 @@ static void pdc_eng_timeout(struct ata_port *ap) | |||
885 | } | 889 | } |
886 | 890 | ||
887 | out: | 891 | out: |
892 | spin_unlock_irqrestore(&host_set->lock, flags); | ||
888 | DPRINTK("EXIT\n"); | 893 | DPRINTK("EXIT\n"); |
889 | } | 894 | } |
890 | 895 | ||