diff options
author | Joonsoo Kim <js1304@gmail.com> | 2012-10-20 18:30:06 +0200 |
---|---|---|
committer | Tejun Heo <tj@kernel.org> | 2012-12-02 01:45:40 +0100 |
commit | 999767beb1b4a10eabf90e6017e496536cf4db0b (patch) | |
tree | 52025d802a2d350e42b0857502cb1ac7b473f31b /kernel/workqueue.c | |
parent | workqueue: mod_delayed_work_on() shouldn't queue timer on 0 delay (diff) | |
download | linux-999767beb1b4a10eabf90e6017e496536cf4db0b.tar.xz linux-999767beb1b4a10eabf90e6017e496536cf4db0b.zip |
workqueue: trivial fix for return statement in work_busy()
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>
Diffstat (limited to 'kernel/workqueue.c')
-rw-r--r-- | kernel/workqueue.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/kernel/workqueue.c b/kernel/workqueue.c index 084aa47bac82..26f5d16aef65 100644 --- a/kernel/workqueue.c +++ b/kernel/workqueue.c @@ -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); |