diff options
| author | Joonsoo Kim <js1304@gmail.com> | 2012-10-21 01:30:06 +0900 |
|---|---|---|
| committer | Tejun Heo <tj@kernel.org> | 2012-12-01 16:45:40 -0800 |
| commit | 999767beb1b4a10eabf90e6017e496536cf4db0b (patch) | |
| tree | 52025d802a2d350e42b0857502cb1ac7b473f31b /kernel/workqueue.c | |
| parent | 8852aac25e79e38cc6529f20298eed154f60b574 (diff) | |
| download | lwn-999767beb1b4a10eabf90e6017e496536cf4db0b.tar.gz lwn-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); |