Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751956AbWI1R10 (ORCPT ); Thu, 28 Sep 2006 13:27:26 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1751957AbWI1R1Z (ORCPT ); Thu, 28 Sep 2006 13:27:25 -0400 Received: from e2.ny.us.ibm.com ([32.97.182.142]:37860 "EHLO e2.ny.us.ibm.com") by vger.kernel.org with ESMTP id S1751956AbWI1R1Y (ORCPT ); Thu, 28 Sep 2006 13:27:24 -0400 Date: Thu, 28 Sep 2006 22:57:02 +0530 From: Srivatsa Vaddagiri To: Ingo Molnar , Nick Piggin Cc: linux-kernel@vger.kernel.org, Kirill Korotaev , Mike Galbraith , Balbir Singh , sekharan@us.ibm.com, Andrew Morton , nagar@watson.ibm.com, matthltc@us.ibm.com, dipankar@in.ibm.com, ckrm-tech@lists.sourceforge.net Subject: [RFC, PATCH 1/9] CPU Controller V2 - Split runqueue Message-ID: <20060928172702.GB8746@in.ibm.com> Reply-To: vatsa@in.ibm.com References: <20060928172520.GA8746@in.ibm.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20060928172520.GA8746@in.ibm.com> User-Agent: Mutt/1.5.11 Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 14619 Lines: 491 This patch introduces a new runqueue structure (struct task_grp_rq) which is used to hold "related" tasks together. The structure is quite similar to how the runqueue structure looks today. The scheme is every task group will have its own per-cpu runqueues (which is of type 'struct task_grp_rq'), to hold its tasks together on each CPU. This is in addition to a per-cpu runqueue which holds runnable task-groups themselves. The patch also converts over several references of 'struct rq' to 'struct task_grp_rq' (since tasks now reside in task_grp_rq). Signed-off-by : Srivatsa Vaddagiri --- linux-2.6.18-root/init/Kconfig | 8 + linux-2.6.18-root/kernel/sched.c | 204 +++++++++++++++++++++++++++++++-------- 2 files changed, 174 insertions(+), 38 deletions(-) diff -puN kernel/sched.c~base_cpu_ctlr kernel/sched.c --- linux-2.6.18/kernel/sched.c~base_cpu_ctlr 2006-09-27 12:48:23.000000000 +0530 +++ linux-2.6.18-root/kernel/sched.c 2006-09-28 17:23:22.133456408 +0530 @@ -191,11 +191,48 @@ static inline unsigned int task_timeslic struct prio_array { unsigned int nr_active; + int best_static_prio; DECLARE_BITMAP(bitmap, MAX_PRIO+1); /* include 1 bit for delimiter */ struct list_head queue[MAX_PRIO]; }; /* + * Each task belong to some task-group. Task-groups and what tasks they contain + * are defined by the administrator using some suitable interface. + * Administrator can also define the CPU bandwidth provided to each task-group. + * + * Task-groups are given a certain priority to run on every CPU. Currently + * task-group priority on a CPU is defined to be the same as that of + * highest-priority runnable task it has on that CPU. Task-groups also + * get their own runqueue on every CPU. The main runqueue on each CPU is + * used to hold task-groups, rather than tasks. + * + * Scheduling decision on a CPU is now two-step : first pick highest priority + * task-group from the main runqueue and next pick highest priority task from + * the runqueue of that group. Both decisions are of O(1) complexity. + */ + +/* runqueue used for every task-group */ +struct task_grp_rq { + struct prio_array arrays[2]; + struct prio_array *active, *expired, *rq_array; + unsigned long expired_timestamp; + int prio; /* Priority of the task-group */ + struct list_head list; + struct task_grp *tg; +}; + +static DEFINE_PER_CPU(struct task_grp_rq, init_tg_rq); + +/* task-group object - maintains information about each task-group */ +struct task_grp { + struct task_grp_rq *rq[NR_CPUS]; /* runqueue pointer for every cpu */ +}; + +/* The "default" task-group */ +struct task_grp init_task_grp; + +/* * This is the main, per-CPU runqueue data structure. * * Locking rule: those places that want to lock multiple runqueues @@ -224,12 +261,13 @@ struct rq { */ unsigned long nr_uninterruptible; - unsigned long expired_timestamp; unsigned long long timestamp_last_tick; struct task_struct *curr, *idle; struct mm_struct *prev_mm; + /* these arrays hold task-groups. + * xxx: Avoid this for !CONFIG_CPUMETER? + */ struct prio_array *active, *expired, arrays[2]; - int best_expired_prio; atomic_t nr_iowait; #ifdef CONFIG_SMP @@ -244,6 +282,7 @@ struct rq { #endif #ifdef CONFIG_SCHEDSTATS + /* xxx: move these to task-group runqueue where necessary */ /* latency stats */ struct sched_info rq_sched_info; @@ -267,6 +306,13 @@ struct rq { static DEFINE_PER_CPU(struct rq, runqueues); +#define switch_array(array1, array2) \ + { \ + struct prio_array *tmp = array2; \ + array2 = array1; \ + array1 = tmp; \ + } + /* * The domain tree (rq->sd) is protected by RCU's quiescent state transition. * See detach_destroy_domains: synchronize_sched for details. @@ -280,6 +326,7 @@ static DEFINE_PER_CPU(struct rq, runqueu #define cpu_rq(cpu) (&per_cpu(runqueues, (cpu))) #define this_rq() (&__get_cpu_var(runqueues)) #define task_rq(p) cpu_rq(task_cpu(p)) +#define task_grp_rq(p) (task_grp(p)->rq[task_cpu(p)]) #define cpu_curr(cpu) (cpu_rq(cpu)->curr) #ifndef prepare_arch_switch @@ -359,6 +406,12 @@ static inline void finish_lock_switch(st } #endif /* __ARCH_WANT_UNLOCKED_CTXSW */ +/* return the task-group to which a task belongs */ +static inline struct task_grp *task_grp(struct task_struct *p) +{ + return &init_task_grp; +} + /* * __task_rq_lock - lock the runqueue a given task resides on. * Must be called interrupts disabled. @@ -831,10 +884,11 @@ static int effective_prio(struct task_st */ static void __activate_task(struct task_struct *p, struct rq *rq) { - struct prio_array *target = rq->active; + struct task_grp_rq *tgrq = task_grp_rq(p); + struct prio_array *target = tgrq->active; if (batch_task(p)) - target = rq->expired; + target = tgrq->expired; enqueue_task(p, target); inc_nr_running(p, rq); } @@ -844,7 +898,10 @@ static void __activate_task(struct task_ */ static inline void __activate_idle_task(struct task_struct *p, struct rq *rq) { - enqueue_task_head(p, rq->active); + struct task_grp_rq *tgrq = task_grp_rq(p); + struct prio_array *target = tgrq->active; + + enqueue_task_head(p, target); inc_nr_running(p, rq); } @@ -2077,7 +2134,7 @@ int can_migrate_task(struct task_struct return 1; } -#define rq_best_prio(rq) min((rq)->curr->prio, (rq)->best_expired_prio) +#define rq_best_prio(rq) min((rq)->curr->prio, (rq)->expired->best_static_prio) /* * move_tasks tries to move up to max_nr_move tasks and max_load_move weighted @@ -2884,6 +2941,8 @@ unsigned long long current_sched_time(co return ns; } +#define nr_tasks(tgrq) (tgrq->active->nr_active + tgrq->expired->nr_active) + /* * We place interactive tasks back into the active array, if possible. * @@ -2894,13 +2953,13 @@ unsigned long long current_sched_time(co * increasing number of running tasks. We also ignore the interactivity * if a better static_prio task has expired: */ -static inline int expired_starving(struct rq *rq) +static inline int expired_starving(struct task_grp_rq *rq) { - if (rq->curr->static_prio > rq->best_expired_prio) + if (current->static_prio > rq->expired->best_static_prio) return 1; if (!STARVATION_LIMIT || !rq->expired_timestamp) return 0; - if (jiffies - rq->expired_timestamp > STARVATION_LIMIT * rq->nr_running) + if (jiffies - rq->expired_timestamp > STARVATION_LIMIT * nr_tasks(rq)) return 1; return 0; } @@ -2991,6 +3050,7 @@ void scheduler_tick(void) struct task_struct *p = current; int cpu = smp_processor_id(); struct rq *rq = cpu_rq(cpu); + struct task_grp_rq *tgrq = task_grp_rq(p); update_cpu_clock(p, rq, now); @@ -3004,7 +3064,7 @@ void scheduler_tick(void) } /* Task might have expired already, but not scheduled off yet */ - if (p->array != rq->active) { + if (p->array != tgrq->active) { set_tsk_need_resched(p); goto out; } @@ -3027,25 +3087,26 @@ void scheduler_tick(void) set_tsk_need_resched(p); /* put it at the end of the queue: */ - requeue_task(p, rq->active); + requeue_task(p, tgrq->active); } goto out_unlock; } if (!--p->time_slice) { - dequeue_task(p, rq->active); + dequeue_task(p, tgrq->active); set_tsk_need_resched(p); p->prio = effective_prio(p); p->time_slice = task_timeslice(p); p->first_time_slice = 0; - if (!rq->expired_timestamp) - rq->expired_timestamp = jiffies; - if (!TASK_INTERACTIVE(p) || expired_starving(rq)) { - enqueue_task(p, rq->expired); - if (p->static_prio < rq->best_expired_prio) - rq->best_expired_prio = p->static_prio; + if (!tgrq->expired_timestamp) + tgrq->expired_timestamp = jiffies; + if (!TASK_INTERACTIVE(p) || expired_starving(tgrq)) { + enqueue_task(p, tgrq->expired); + if (p->static_prio < tgrq->expired->best_static_prio) + tgrq->expired->best_static_prio = + p->static_prio; } else - enqueue_task(p, rq->active); + enqueue_task(p, tgrq->active); } else { /* * Prevent a too long timeslice allowing a task to monopolize @@ -3066,12 +3127,13 @@ void scheduler_tick(void) if (TASK_INTERACTIVE(p) && !((task_timeslice(p) - p->time_slice) % TIMESLICE_GRANULARITY(p)) && (p->time_slice >= TIMESLICE_GRANULARITY(p)) && - (p->array == rq->active)) { + (p->array == tgrq->active)) { - requeue_task(p, rq->active); + requeue_task(p, tgrq->active); set_tsk_need_resched(p); } } + out_unlock: spin_unlock(&rq->lock); out: @@ -3264,6 +3326,7 @@ asmlinkage void __sched schedule(void) int cpu, idx, new_prio; long *switch_count; struct rq *rq; + struct task_grp_rq *next_grp; /* * Test if we are atomic. Since do_exit() needs to call into @@ -3332,23 +3395,36 @@ need_resched_nonpreemptible: idle_balance(cpu, rq); if (!rq->nr_running) { next = rq->idle; - rq->expired_timestamp = 0; wake_sleeping_dependent(cpu); goto switch_tasks; } } + /* Pick a task group first */ +#ifdef CONFIG_CPUMETER array = rq->active; if (unlikely(!array->nr_active)) { + switch_array(rq->active, rq->expired); + array = rq->active; + } + idx = sched_find_first_bit(array->bitmap); + queue = array->queue + idx; + next_grp = list_entry(queue->next, struct task_grp_rq, list); +#else + next_grp = init_task_grp.rq[cpu]; +#endif + + /* Pick a task within that group next */ + array = next_grp->active; + if (unlikely(!array->nr_active)) { /* * Switch the active and expired arrays. */ - schedstat_inc(rq, sched_switch); - rq->active = rq->expired; - rq->expired = array; - array = rq->active; - rq->expired_timestamp = 0; - rq->best_expired_prio = MAX_PRIO; + schedstat_inc(next_grp, sched_switch); + switch_array(next_grp->active, next_grp->expired); + array = next_grp->active; + next_grp->expired_timestamp = 0; + next_grp->expired->best_static_prio = MAX_PRIO; } idx = sched_find_first_bit(array->bitmap); @@ -3826,11 +3902,13 @@ void rt_mutex_setprio(struct task_struct struct prio_array *array; unsigned long flags; struct rq *rq; + struct task_grp_rq *tgrq; int oldprio; BUG_ON(prio < 0 || prio > MAX_PRIO); rq = task_rq_lock(p, &flags); + tgrq = task_grp_rq(p); oldprio = p->prio; array = p->array; @@ -3844,7 +3922,7 @@ void rt_mutex_setprio(struct task_struct * in the active array! */ if (rt_task(p)) - array = rq->active; + array = tgrq->active; enqueue_task(p, array); /* * Reschedule if we are currently running on this runqueue and @@ -4411,7 +4489,8 @@ asmlinkage long sys_sched_getaffinity(pi asmlinkage long sys_sched_yield(void) { struct rq *rq = this_rq_lock(); - struct prio_array *array = current->array, *target = rq->expired; + struct task_grp_rq *tgrq = task_grp_rq(current); + struct prio_array *array = current->array, *target = tgrq->expired; schedstat_inc(rq, yld_cnt); /* @@ -4422,13 +4501,13 @@ asmlinkage long sys_sched_yield(void) * array.) */ if (rt_task(current)) - target = rq->active; + target = tgrq->active; if (array->nr_active == 1) { schedstat_inc(rq, yld_act_empty); - if (!rq->expired->nr_active) + if (!tgrq->expired->nr_active) schedstat_inc(rq, yld_both_empty); - } else if (!rq->expired->nr_active) + } else if (!tgrq->expired->nr_active) schedstat_inc(rq, yld_exp_empty); if (array != target) { @@ -5131,9 +5210,8 @@ static void migrate_dead(unsigned int de } /* release_task() removes task from tasklist, so we won't find dead tasks. */ -static void migrate_dead_tasks(unsigned int dead_cpu) +static void migrate_dead_tasks_from_grp(struct task_grp_rq *rq) { - struct rq *rq = cpu_rq(dead_cpu); unsigned int arr, i; for (arr = 0; arr < 2; arr++) { @@ -5146,6 +5224,26 @@ static void migrate_dead_tasks(unsigned } } } + +static void migrate_dead_tasks(unsigned int dead_cpu) +{ + struct rq *rq = cpu_rq(dead_cpu); + unsigned int arr, i; + + for (arr = 0; arr < 2; arr++) { + for (i = 0; i < MAX_PRIO; i++) { + struct list_head *list = &rq->arrays[arr].queue[i]; + + while (!list_empty(list)) { + struct task_grp_rq *tgrq = + list_entry(list->next, + struct task_grp_rq, list); + + migrate_dead_tasks_from_grp(tgrq); + } + } + } +} #endif /* CONFIG_HOTPLUG_CPU */ /* @@ -6725,21 +6823,48 @@ int in_sched_functions(unsigned long add && addr < (unsigned long)__sched_text_end); } +static void task_grp_rq_init(struct task_grp_rq *tgrq, struct task_grp *tg) +{ + int j, k; + + tgrq->active = tgrq->arrays; + tgrq->expired = tgrq->arrays + 1; + tgrq->rq_array = NULL; + tgrq->expired->best_static_prio = MAX_PRIO; + tgrq->active->best_static_prio = MAX_PRIO; + tgrq->prio = MAX_PRIO; + tgrq->tg = tg; + INIT_LIST_HEAD(&tgrq->list); + + for (j = 0; j < 2; j++) { + struct prio_array *array; + + array = tgrq->arrays + j; + for (k = 0; k < MAX_PRIO; k++) { + INIT_LIST_HEAD(array->queue + k); + __clear_bit(k, array->bitmap); + } + // delimiter for bitsearch + __set_bit(MAX_PRIO, array->bitmap); + } +} + void __init sched_init(void) { - int i, j, k; + int i, j; for_each_possible_cpu(i) { - struct prio_array *array; struct rq *rq; + struct task_grp_rq *tgrq; rq = cpu_rq(i); + tgrq = init_task_grp.rq[i] = &per_cpu(init_tg_rq, i); spin_lock_init(&rq->lock); + task_grp_rq_init(tgrq, &init_task_grp); lockdep_set_class(&rq->lock, &rq->rq_lock_key); rq->nr_running = 0; rq->active = rq->arrays; rq->expired = rq->arrays + 1; - rq->best_expired_prio = MAX_PRIO; #ifdef CONFIG_SMP rq->sd = NULL; @@ -6753,6 +6878,9 @@ void __init sched_init(void) atomic_set(&rq->nr_iowait, 0); for (j = 0; j < 2; j++) { + struct prio_array *array; + int k; + array = rq->arrays + j; for (k = 0; k < MAX_PRIO; k++) { INIT_LIST_HEAD(array->queue + k); diff -puN init/Kconfig~base_cpu_ctlr init/Kconfig --- linux-2.6.18/init/Kconfig~base_cpu_ctlr 2006-09-27 12:48:23.000000000 +0530 +++ linux-2.6.18-root/init/Kconfig 2006-09-27 12:48:24.000000000 +0530 @@ -231,6 +231,14 @@ config CPUSETS Say N if unsure. +config CPUMETER + bool "CPU resource control" + depends on CPUSETS && EXPERIMENTAL + help + This options lets you create cpu resource partitions within + cpusets. Each resource partition can be given a different quota + of CPU usage. + config RELAY bool "Kernel->user space relay support (formerly relayfs)" help _ -- Regards, vatsa - 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/