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/nodemgr.c | |
parent | ieee1394: cycle timer read extension for raw1394 (diff) | |
download | linux-a65421ea3f8a0e78b4e3b858bdc5ada5ad653807.tar.xz linux-a65421ea3f8a0e78b4e3b858bdc5ada5ad653807.zip |
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/nodemgr.c')
-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 ba9faeff4793..c5ace190bfe6 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); |