diff options
author | Lai Jiangshan <laijs@cn.fujitsu.com> | 2014-07-10 16:11:13 (GMT) |
---|---|---|
committer | Tejun Heo <tj@kernel.org> | 2014-07-11 15:14:50 (GMT) |
commit | 9c34a7042e6fe79b900bd44db03dde8a4ecdf6f6 (patch) | |
tree | 1ff3a9f8b3fdea4fd8e7a2e2bbcb43b91e0aa96b | |
parent | 85327af61d4cb1700d4c71e8080f7c7df9bccafb (diff) | |
download | linux-9c34a7042e6fe79b900bd44db03dde8a4ecdf6f6.tar.xz |
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>
-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 f344334..338d418 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); |