lkml.org 
[lkml]   [2007]   [May]   [24]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
Date
From
Subject[BUG] 2.6.21 hang in cancel_rearming_delayed_workqueue()
There is a problem with the calling cancel_rearming_delayed_work if the 
timer was not yet active.

I see this problem when netpoll_cleanup() is called without having done
any work because it had not processed any packets yet. The problem
appears to be a result of the loop check
while(!cancel_delayed_work(dwork)). This endlessly loops because
del_timer_sync() can return 0 or 1 for success which is passed back as a
result to the final invariant check for the loop. In this particular
case zero will always be returned because the timer is not active.

It is possible that the problem exists else where, but I thought I would
ask if this is expected?

#0 del_timer_sync (timer=0xc7ed90f8) at kernel/timer.c:530
#1 0xc012f08e in cancel_rearming_delayed_workqueue (wq=0xc7fee800,
dwork=0xc7ed90e8) at include/linux/workqueue.h:201
#2 0xc012f0af in cancel_rearming_delayed_work (dwork=0x20)
at kernel/workqueue.c:680
#3 0xc0312f78 in netpoll_cleanup (np=0xc880bf40) at net/core/netpoll.c:784

Possible fix.

Signed-off-by: Jason Wessel <jason.wessel@windriver.com>

Index: linux-2.6.21/kernel/workqueue.c
===================================================================
--- linux-2.6.21.orig/kernel/workqueue.c
+++ linux-2.6.21/kernel/workqueue.c
@@ -666,7 +666,7 @@ EXPORT_SYMBOL(flush_scheduled_work);
void cancel_rearming_delayed_workqueue(struct workqueue_struct *wq,
struct delayed_work *dwork)
{
- while (!cancel_delayed_work(dwork))
+ while (cancel_delayed_work(dwork) > 0)
flush_workqueue(wq);
}
EXPORT_SYMBOL(cancel_rearming_delayed_workqueue);




Thanks,
Jason.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
\
 
 \ /
  Last update: 2007-05-25 05:25    [W:0.085 / U:0.032 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site