diff options
| author | Libin <huawei.libin@huawei.com> | 2013-08-21 08:50:40 +0800 |
|---|---|---|
| committer | Tejun Heo <tj@kernel.org> | 2013-08-21 10:32:09 -0400 |
| commit | 2d498db9814c6f3a79b708c8867c7ffcf7b5e2fc (patch) | |
| tree | af09d25871b0b8768c63411530e5c7d7c6e4c22b /kernel/workqueue.c | |
| parent | b11895c45899daff094610f6cdbf7611d74ae2a6 (diff) | |
workqueue: Fix manage_workers() RETURNS description
No functional change. The comment of function manage_workers()
RETURNS description is obvious wrong, same as the CONTEXT.
Fix it.
Signed-off-by: Libin <huawei.libin@huawei.com>
Signed-off-by: Tejun Heo <tj@kernel.org>
Diffstat (limited to 'kernel/workqueue.c')
| -rw-r--r-- | kernel/workqueue.c | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/kernel/workqueue.c b/kernel/workqueue.c index eebd9a66c04..10f655ec8de 100644 --- a/kernel/workqueue.c +++ b/kernel/workqueue.c @@ -2034,8 +2034,11 @@ static bool maybe_destroy_workers(struct worker_pool *pool) * multiple times. Does GFP_KERNEL allocations. * * RETURNS: - * spin_lock_irq(pool->lock) which may be released and regrabbed - * multiple times. Does GFP_KERNEL allocations. + * %false if the pool don't need management and the caller can safely start + * processing works, %true indicates that the function released pool->lock + * and reacquired it to perform some management function and that the + * conditions that the caller verified while holding the lock before + * calling the function might no longer be true. */ static bool manage_workers(struct worker *worker) { |