diff options
| author | Peter Zijlstra <a.p.zijlstra@chello.nl> | 2009-03-02 10:45:53 +0100 |
|---|---|---|
| committer | Helge Deller <deller@gmx.de> | 2013-02-20 22:50:26 +0100 |
| commit | b54cb2332e387f29c65f19f3620e5c812c89a328 (patch) | |
| tree | f583188f34b154f3a17ce5e2a0747b553faea4a1 /drivers/parisc | |
| parent | cca8e9026041544c0103b3037d8f03c1d2f4ae02 (diff) | |
| download | lwn-b54cb2332e387f29c65f19f3620e5c812c89a328.tar.gz lwn-b54cb2332e387f29c65f19f3620e5c812c89a328.zip | |
parisc: remove IRQF_DISABLED
People are playing odd games with IRQF_DISABLED, remove it.
Its not reliable, since shared interrupt lines could disable it for you,
and its possible and allowed for archs to disable IRQs to limit IRQ nesting.
Therefore, simply mandate that _ALL_ IRQ handlers are run with IRQs disabled.
[ This _should_ not break anything, since we've mandated that IRQ handlers
_must_ be able to deal with this for a _long_ time ]
IRQ handlers should be fast, no if buts and any other exceptions. We also have
plenty instrumentation to find any offending IRQ latency sources.
Signed-off-by: Peter Zijlstra <a.p.zijlstra@chello.nl>
Acked-by: Ingo Molnar <mingo@elte.hu>
Signed-off-by: Helge Deller <deller@gmx.de>
Diffstat (limited to 'drivers/parisc')
| -rw-r--r-- | drivers/parisc/superio.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/parisc/superio.c b/drivers/parisc/superio.c index 5003458980d3..ac6e8e7a02df 100644 --- a/drivers/parisc/superio.c +++ b/drivers/parisc/superio.c @@ -274,7 +274,7 @@ superio_init(struct pci_dev *pcidev) else printk(KERN_ERR PFX "USB regulator not initialized!\n"); - if (request_irq(pdev->irq, superio_interrupt, IRQF_DISABLED, + if (request_irq(pdev->irq, superio_interrupt, 0, SUPERIO, (void *)sio)) { printk(KERN_ERR PFX "could not get irq\n"); |