diff options
| author | Stefan Richter <stefanr@s5r6.in-berlin.de> | 2007-02-10 22:06:18 +0100 |
|---|---|---|
| committer | Stefan Richter <stefanr@s5r6.in-berlin.de> | 2007-02-17 14:41:18 +0100 |
| commit | a65421ea3f8a0e78b4e3b858bdc5ada5ad653807 (patch) | |
| tree | 5efc5d9e1f57afef8457f9c40c0071da4a26d8a6 /drivers/ieee1394 | |
| parent | 3dc5ea9b31a8c83cffa338aba91adbc9bd387002 (diff) | |
| download | op-kernel-dev-a65421ea3f8a0e78b4e3b858bdc5ada5ad653807.zip op-kernel-dev-a65421ea3f8a0e78b4e3b858bdc5ada5ad653807.tar.gz | |
ieee1394: fix another deadlock in nodemgr
A "modprobe ohci1394; sleep 1.5; modprobe -r ohci1394" could get stuck
in uninterruptible state, especially if an external node was connected.
http://bugzilla.kernel.org/show_bug.cgi?id=7792
Signed-off-by: Stefan Richter <stefanr@s5r6.in-berlin.de>
Diffstat (limited to 'drivers/ieee1394')
| -rw-r--r-- | drivers/ieee1394/nodemgr.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/ieee1394/nodemgr.c b/drivers/ieee1394/nodemgr.c index ba9faef..c5ace19 100644 --- a/drivers/ieee1394/nodemgr.c +++ b/drivers/ieee1394/nodemgr.c @@ -1681,7 +1681,8 @@ static int nodemgr_host_thread(void *__hi) for (;;) { /* Sleep until next bus reset */ set_current_state(TASK_INTERRUPTIBLE); - if (get_hpsb_generation(host) == generation) + if (get_hpsb_generation(host) == generation && + !kthread_should_stop()) schedule(); __set_current_state(TASK_RUNNING); |
