diff options
| author |   Sebastian Ott <sebott@linux.ibm.com> | 2018-10-30 14:04:46 +0100 |
|---|---|---|
| committer |   Martin Schwidefsky <schwidefsky@de.ibm.com> | 2018-12-13 10:42:25 +0100 |
| commit | 9594ca6b87d9f11e9f14ac31581e0e5d79a8e839 (patch) | |
| tree | a62624e2f1bfb99351c775f04dd1db04f2c59b32 /arch/s390 | |
| parent | s390/drivers: fix proc/debugfs file permissions (diff) | |
| download | linux-dev-9594ca6b87d9f11e9f14ac31581e0e5d79a8e839.tar.xz linux-dev-9594ca6b87d9f11e9f14ac31581e0e5d79a8e839.zip | |
s390/pci: remove bit_lock usage in interrupt handler
The interrupt handler uses bit_spin_lock around a call to retrieve
per irq data (the irq number). However this per irq data is only
set during irq setup time and never changed until the irq is freed.
Thus it's safe to remove the lock usage.
Signed-off-by: Sebastian Ott <sebott@linux.ibm.com>
Signed-off-by: Martin Schwidefsky <schwidefsky@de.ibm.com>
Diffstat (limited to 'arch/s390')
| -rw-r--r-- | arch/s390/pci/pci.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/arch/s390/pci/pci.c b/arch/s390/pci/pci.c index 9f6f392a4461..6df622fb406d 100644 --- a/arch/s390/pci/pci.c +++ b/arch/s390/pci/pci.c @@ -382,9 +382,7 @@ static void zpci_irq_handler(struct airq_struct *airq) if (ai == -1UL) break; inc_irq_stat(IRQIO_MSI); - airq_iv_lock(aibv, ai); generic_handle_irq(airq_iv_get_data(aibv, ai)); - airq_iv_unlock(aibv, ai); } } } @@ -410,7 +408,7 @@ int arch_setup_msi_irqs(struct pci_dev *pdev, int nvec, int type) zdev->aisb = aisb; /* Create adapter interrupt vector */ - zdev->aibv = airq_iv_create(msi_vecs, AIRQ_IV_DATA | AIRQ_IV_BITLOCK); + zdev->aibv = airq_iv_create(msi_vecs, AIRQ_IV_DATA); if (!zdev->aibv) return -ENOMEM; |