diff options
author | Joonsoo Kim <js1304@gmail.com> | 2012-10-20 12:30:06 -0400 |
---|---|---|
committer | Tejun Heo <tj@kernel.org> | 2012-12-01 19:45:40 -0500 |
commit | 999767beb1b4a10eabf90e6017e496536cf4db0b (patch) | |
tree | 52025d802a2d350e42b0857502cb1ac7b473f31b /kernel/workqueue.c | |
parent | 8852aac25e79e38cc6529f20298eed154f60b574 (diff) |
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) | |||
3485 | unsigned int ret = 0; | 3485 | unsigned int ret = 0; |
3486 | 3486 | ||
3487 | if (!gcwq) | 3487 | if (!gcwq) |
3488 | return false; | 3488 | return 0; |
3489 | 3489 | ||
3490 | spin_lock_irqsave(&gcwq->lock, flags); | 3490 | spin_lock_irqsave(&gcwq->lock, flags); |
3491 | 3491 | ||