diff options
| author | Tejun Heo <tj@kernel.org> | 2014-10-27 12:00:01 -0400 |
|---|---|---|
| committer | Tejun Heo <tj@kernel.org> | 2014-10-27 12:00:01 -0400 |
| commit | 03e83cbd34a4602bf0d750e5ff4bf8b7d0e066b2 (patch) | |
| tree | 1991876407bc3a09d4d3f9acc5c1699c99c11304 /drivers | |
| parent | aa1cf25887099bba68f1f3879c0d394e08b8779f (diff) | |
Revert "AHCI: Do not acquire ata_host::lock from single IRQ handler"
This reverts commit 33fb0d01ce60fe4c0c12c4f0c134c5cdb818ac5a.
18dcf433f3de ("AHCI: Optimize single IRQ interrupt processing") is
scheduled to be reverted. This is an optimization dependent on the
mentioned commit. Revert it first.
Signed-off-by: Tejun Heo <tj@kernel.org>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/ata/libahci.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/drivers/ata/libahci.c b/drivers/ata/libahci.c index 5eb61c9e63da..3ce3d23e4f97 100644 --- a/drivers/ata/libahci.c +++ b/drivers/ata/libahci.c | |||
| @@ -1875,6 +1875,8 @@ static irqreturn_t ahci_single_irq_intr(int irq, void *dev_instance) | |||
| 1875 | 1875 | ||
| 1876 | irq_masked = irq_stat & hpriv->port_map; | 1876 | irq_masked = irq_stat & hpriv->port_map; |
| 1877 | 1877 | ||
| 1878 | spin_lock(&host->lock); | ||
| 1879 | |||
| 1878 | for (i = 0; i < host->n_ports; i++) { | 1880 | for (i = 0; i < host->n_ports; i++) { |
| 1879 | struct ata_port *ap; | 1881 | struct ata_port *ap; |
| 1880 | 1882 | ||
| @@ -1906,6 +1908,8 @@ static irqreturn_t ahci_single_irq_intr(int irq, void *dev_instance) | |||
| 1906 | */ | 1908 | */ |
| 1907 | writel(irq_stat, mmio + HOST_IRQ_STAT); | 1909 | writel(irq_stat, mmio + HOST_IRQ_STAT); |
| 1908 | 1910 | ||
| 1911 | spin_unlock(&host->lock); | ||
| 1912 | |||
| 1909 | VPRINTK("EXIT\n"); | 1913 | VPRINTK("EXIT\n"); |
| 1910 | 1914 | ||
| 1911 | return handled ? IRQ_WAKE_THREAD : IRQ_NONE; | 1915 | return handled ? IRQ_WAKE_THREAD : IRQ_NONE; |
