workqueue: cosmetic update in try_to_grab_pending()
authorTejun Heo <tj@kernel.org>
Thu, 7 Feb 2013 02:04:53 +0000 (18:04 -0800)
committerTejun Heo <tj@kernel.org>
Thu, 7 Feb 2013 02:04:53 +0000 (18:04 -0800)
With the recent is-work-queued-here test simplification, the nested
if() in try_to_grab_pending() can be collapsed.  Collapse it.

This patch is purely cosmetic.

Signed-off-by: Tejun Heo <tj@kernel.org>
Cc: Lai Jiangshan <laijs@cn.fujitsu.com>
kernel/workqueue.c

index 251f00914295036ae1f2e528a67e3d9851a433e7..e2dd61861fbd17a150f3be5f470c782327fc0073 100644 (file)
@@ -1107,31 +1107,27 @@ static int try_to_grab_pending(struct work_struct *work, bool is_dwork,
         * item is currently queued on that pool.
         */
        cwq = get_work_cwq(work);
-       if (cwq) {
-               if (cwq->pool == pool) {
-                       debug_work_deactivate(work);
+       if (cwq && cwq->pool == pool) {
+               debug_work_deactivate(work);
 
-                       /*
-                        * A delayed work item cannot be grabbed directly
-                        * because it might have linked NO_COLOR work items
-                        * which, if left on the delayed_list, will confuse
-                        * cwq->nr_active management later on and cause
-                        * stall.  Make sure the work item is activated
-                        * before grabbing.
-                        */
-                       if (*work_data_bits(work) & WORK_STRUCT_DELAYED)
-                               cwq_activate_delayed_work(work);
+               /*
+                * A delayed work item cannot be grabbed directly because
+                * it might have linked NO_COLOR work items which, if left
+                * on the delayed_list, will confuse cwq->nr_active
+                * management later on and cause stall.  Make sure the work
+                * item is activated before grabbing.
+                */
+               if (*work_data_bits(work) & WORK_STRUCT_DELAYED)
+                       cwq_activate_delayed_work(work);
 
-                       list_del_init(&work->entry);
-                       cwq_dec_nr_in_flight(get_work_cwq(work),
-                               get_work_color(work));
+               list_del_init(&work->entry);
+               cwq_dec_nr_in_flight(get_work_cwq(work), get_work_color(work));
 
-                       /* work->data points to cwq iff queued, point to pool */
-                       set_work_pool_and_keep_pending(work, pool->id);
+               /* work->data points to cwq iff queued, point to pool */
+               set_work_pool_and_keep_pending(work, pool->id);
 
-                       spin_unlock(&pool->lock);
-                       return 1;
-               }
+               spin_unlock(&pool->lock);
+               return 1;
        }
        spin_unlock(&pool->lock);
 fail: