workqueue: trivial fix for return statement in work_busy()
authorJoonsoo Kim <js1304@gmail.com>
Sat, 20 Oct 2012 16:30:06 +0000 (01:30 +0900)
committerTejun Heo <tj@kernel.org>
Sun, 2 Dec 2012 00:45:40 +0000 (16:45 -0800)
Return type of work_busy() is unsigned int.
There is return statement returning boolean value, 'false' in work_busy().
It is not problem, because 'false' may be treated '0'.
However, fixing it would make code robust.

Signed-off-by: Joonsoo Kim <js1304@gmail.com>
Signed-off-by: Tejun Heo <tj@kernel.org>
kernel/workqueue.c

index 084aa47bac820498f932ae6ff54158414e48e889..26f5d16aef65264d138fb83d28cf6b15d881d70d 100644 (file)
@@ -3485,7 +3485,7 @@ unsigned int work_busy(struct work_struct *work)
        unsigned int ret = 0;
 
        if (!gcwq)
-               return false;
+               return 0;
 
        spin_lock_irqsave(&gcwq->lock, flags);