diff options
author | Phil Sutter <phil@nwl.cc> | 2010-05-29 13:23:34 +0000 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2010-06-02 06:12:18 -0700 |
commit | ceb3d2394532540a52ce34f71e67c8d008913f79 (patch) | |
tree | 8bd8cb8d9af745ed0aa2e55a1b0e8ef51b74020a /drivers/firewire/core-transaction.c | |
parent | 2df4a0fa1540c460ec69788ab2a901cc72a75644 (diff) |
korina: fix deadlock on RX FIFO overrun
By calling korina_restart(), the IRQ handler tries to disable the
interrupt it's currently serving. This leads to a deadlock since
disable_irq() waits for any running IRQ handlers to finish before
returning. This patch addresses the issue by turning korina_restart()
into a workqueue task, which is then scheduled when needed.
Reproducing the deadlock is easily done using e.g. GNU netcat to send
large amounts of UDP data to the host running this driver.
Note that the same problem (and fix) applies to TX FIFO underruns, but
apparently these are less easy to trigger.
Signed-off-by: Phil Sutter <phil@nwl.cc>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/firewire/core-transaction.c')
0 files changed, 0 insertions, 0 deletions