From: Christoph Lameter Add a few counters to be able to see how the softirq affects the scheduler. This patch should not go into mainline and maybe also not into mm. (akpm: heh. Bad boy) New counters in /proc/vmstat sched_tick_balance Ticks that required invokation of the softirq sched_tick_no_balance Ticks that did not require the softirq sched_load_balance # invocations of load_balance(). sched_balance_softirq # of softirq executions sched_balance_running # of times that serialization prevented load balancing of a sched_domain. Signed-off-by: Christoph Lameter Cc: Peter Williams Cc: Nick Piggin Cc: Christoph Lameter Cc: "Siddha, Suresh B" Cc: "Chen, Kenneth W" Cc: Ingo Molnar Cc: KAMEZAWA Hiroyuki Signed-off-by: Andrew Morton --- include/linux/vmstat.h | 2 ++ kernel/sched.c | 12 ++++++++++-- mm/vmstat.c | 5 +++++ 3 files changed, 17 insertions(+), 2 deletions(-) diff -puN include/linux/vmstat.h~mm-only-sched-add-a-few-scheduler-event-counters include/linux/vmstat.h --- a/include/linux/vmstat.h~mm-only-sched-add-a-few-scheduler-event-counters +++ a/include/linux/vmstat.h @@ -37,6 +37,8 @@ enum vm_event_item { PGPGIN, PGPGOUT, PS FOR_ALL_ZONES(PGSCAN_DIRECT), PGINODESTEAL, SLABS_SCANNED, KSWAPD_STEAL, KSWAPD_INODESTEAL, PAGEOUTRUN, ALLOCSTALL, PGROTATED, + SCHED_TICK_NO_BALANCE, SCHED_TICK_BALANCE, SCHED_BALANCE_SOFTIRQ, + SCHED_LOAD_BALANCE, SCHED_BALANCE_RUNNING, NR_VM_EVENT_ITEMS }; diff -puN kernel/sched.c~mm-only-sched-add-a-few-scheduler-event-counters kernel/sched.c --- a/kernel/sched.c~mm-only-sched-add-a-few-scheduler-event-counters +++ a/kernel/sched.c @@ -2953,6 +2953,8 @@ static void run_rebalance_domains(struct /* Earliest time when we have to call run_rebalance_domains again */ unsigned long next_balance = jiffies + 60*HZ; + __count_vm_event(SCHED_BALANCE_SOFTIRQ); + for_each_domain(this_cpu, sd) { if (!(sd->flags & SD_LOAD_BALANCE)) continue; @@ -2967,11 +2969,14 @@ static void run_rebalance_domains(struct interval = 1; if (sd->flags & SD_SERIALIZE) { - if (!spin_trylock(&balancing)) + if (!spin_trylock(&balancing)) { + __count_vm_event(SCHED_BALANCE_RUNNING); goto out; + } } if (time_after_eq(jiffies, sd->last_balance + interval)) { + __count_vm_event(SCHED_LOAD_BALANCE); if (load_balance(this_cpu, this_rq, sd, idle, &balance)) { /* * We've pulled tasks over so either we're no @@ -3247,8 +3252,11 @@ void scheduler_tick(void) task_running_tick(rq, p); #ifdef CONFIG_SMP update_load(rq); - if (time_after_eq(jiffies, rq->next_balance)) + if (time_after_eq(jiffies, rq->next_balance)) { + __count_vm_event(SCHED_TICK_BALANCE); raise_softirq(SCHED_SOFTIRQ); + } + __count_vm_event(SCHED_TICK_NO_BALANCE); #endif } diff -puN mm/vmstat.c~mm-only-sched-add-a-few-scheduler-event-counters mm/vmstat.c --- a/mm/vmstat.c~mm-only-sched-add-a-few-scheduler-event-counters +++ a/mm/vmstat.c @@ -484,6 +484,11 @@ static const char * const vmstat_text[] "allocstall", "pgrotated", + "sched_tick_no_balance", + "sched_tick_balance", + "sched_balance_softirq", + "sched_load_balance", + "sched_balance_running", #endif }; _