diff options
author | Lai Jiangshan <laijs@cn.fujitsu.com> | 2014-07-11 00:11:13 +0800 |
---|---|---|
committer | Tejun Heo <tj@kernel.org> | 2014-07-11 11:14:50 -0400 |
commit | 9c34a7042e6fe79b900bd44db03dde8a4ecdf6f6 (patch) | |
tree | 1ff3a9f8b3fdea4fd8e7a2e2bbcb43b91e0aa96b /kernel/workqueue.c | |
parent | 85327af61d4cb1700d4c71e8080f7c7df9bccafb (diff) | |
download | lwn-9c34a7042e6fe79b900bd44db03dde8a4ecdf6f6.tar.gz lwn-9c34a7042e6fe79b900bd44db03dde8a4ecdf6f6.zip |
workqueue: reuse the already calculated pwq in try_to_grab_pending()
try_to_grab_pending() was re-calculating the associated pwq using
get_work_pwq() when it already has it cached in a local varible and
the association can't change. Reuse the local variable instead.
This doesn't introduce any functional changes.
tj: Updated description.
Signed-off-by: Lai Jiangshan <laijs@cn.fujitsu.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 f344334f5690..338d418ba6aa 100644 --- a/kernel/workqueue.c +++ b/kernel/workqueue.c @@ -1225,7 +1225,7 @@ static int try_to_grab_pending(struct work_struct *work, bool is_dwork, pwq_activate_delayed_work(work); list_del_init(&work->entry); - pwq_dec_nr_in_flight(get_work_pwq(work), get_work_color(work)); + pwq_dec_nr_in_flight(pwq, get_work_color(work)); /* work->data points to pwq iff queued, point to pool */ set_work_pool_and_keep_pending(work, pool->id); |