Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755525AbZCKOzu (ORCPT ); Wed, 11 Mar 2009 10:55:50 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1754503AbZCKOz3 (ORCPT ); Wed, 11 Mar 2009 10:55:29 -0400 Received: from hawking.rebel.net.au ([203.20.69.83]:42646 "EHLO hawking.rebel.net.au" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752811AbZCKOz2 (ORCPT ); Wed, 11 Mar 2009 10:55:28 -0400 Message-ID: <49B7D0D5.9070001@davidnewall.com> Date: Thu, 12 Mar 2009 01:25:17 +1030 From: David Newall User-Agent: Thunderbird 2.0.0.12 (X11/20080227) Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="=_hawking.rebel.net.au-9018-1236783324-0001-2" To: Linux Kernel Mailing List Subject: [PATCH] make dequeue_task less confusing Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2685 Lines: 93 This is a MIME-formatted message. If you see this text it means that your E-mail software does not support MIME-formatted messages. --=_hawking.rebel.net.au-9018-1236783324-0001-2 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit The dequeue_patch function in kernel/sched.c is complicated by including a sleep parameter. This parameter is always zero except in one instance. This patch clarifies the task of dequeue_patch by removing the sleep parameter and moving the code that handles non-zero sleep to that one place where it is needed. --=_hawking.rebel.net.au-9018-1236783324-0001-2 Content-Type: text/x-patch; name="sched.patch"; charset=iso-8859-1 Content-Transfer-Encoding: 7bit Content-Disposition: inline; filename="sched.patch" --- linux-2.6.27.19/kernel/sched.c 2009-02-21 09:09:34.000000000 +1030 +++ linux-2.6.27.19/kernel/sched.c.dn 2009-03-12 01:01:48.000000000 +1030 @@ -1645,16 +1645,10 @@ p->se.on_rq = 1; } -static void dequeue_task(struct rq *rq, struct task_struct *p, int sleep) +static void dequeue_task(struct rq *rq, struct task_struct *p) { - if (sleep && p->se.last_wakeup) { - update_avg(&p->se.avg_overlap, - p->se.sum_exec_runtime - p->se.last_wakeup); - p->se.last_wakeup = 0; - } - sched_info_dequeued(p); - p->sched_class->dequeue_task(rq, p, sleep); + p->sched_class->dequeue_task(rq, p, 0); p->se.on_rq = 0; } @@ -1724,7 +1718,17 @@ if (task_contributes_to_load(p)) rq->nr_uninterruptible++; - dequeue_task(rq, p, sleep); + if (sleep && p->se.last_wakeup) { + update_avg(&p->se.avg_overlap, + p->se.sum_exec_runtime - p->se.last_wakeup); + p->se.last_wakeup = 0; + } + + /*dequeue_task(rq, p, sleep);*/ + sched_info_dequeued(p); + p->sched_class->dequeue_task(rq, p, sleep); + p->se.on_rq = 0; + dec_nr_running(rq); } @@ -4848,7 +4852,7 @@ on_rq = p->se.on_rq; running = task_current(rq, p); if (on_rq) - dequeue_task(rq, p, 0); + dequeue_task(rq, p); if (running) p->sched_class->put_prev_task(rq, p); @@ -4897,7 +4901,7 @@ } on_rq = p->se.on_rq; if (on_rq) - dequeue_task(rq, p, 0); + dequeue_task(rq, p); p->static_prio = NICE_TO_PRIO(nice); set_load_weight(p); @@ -8637,7 +8641,7 @@ on_rq = tsk->se.on_rq; if (on_rq) - dequeue_task(rq, tsk, 0); + dequeue_task(rq, tsk); if (unlikely(running)) tsk->sched_class->put_prev_task(rq, tsk); --=_hawking.rebel.net.au-9018-1236783324-0001-2-- -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/