On 01/21/2014 03:43 PM, Jason Low wrote: > On Mon, Jan 20, 2014 at 9:33 PM, Alex Shi <alex....@linaro.org> wrote: >> It's useful to track this value in debug mode. >> >> Signed-off-by: Alex Shi <alex....@linaro.org> >> --- >> kernel/sched/debug.c | 1 + >> 1 file changed, 1 insertion(+) >> >> diff --git a/kernel/sched/debug.c b/kernel/sched/debug.c >> index 1e43e70..f5c529a 100644 >> --- a/kernel/sched/debug.c >> +++ b/kernel/sched/debug.c >> @@ -315,6 +315,7 @@ do { >> \ >> P(sched_goidle); >> #ifdef CONFIG_SMP >> P64(avg_idle); >> + p64(max_idle_balance_cost); > > Hi Alex, > > Does this need to be P64(max_idle_balance_cost)? >
Ops, this letter lay in my Drafts box. Yes, thanks for fix! === >From 173417f3646ac1c6ac83809b51d0471591cbc8f0 Mon Sep 17 00:00:00 2001 From: Alex Shi <alex....@linaro.org> Date: Tue, 21 Jan 2014 13:28:55 +0800 Subject: [RFC PATCH] sched: add statistic for rq->max_idle_balance_cost It's useful to track this value in debug mode. Signed-off-by: Alex Shi <alex....@linaro.org> --- kernel/sched/debug.c | 1 + 1 file changed, 1 insertion(+) diff --git a/kernel/sched/debug.c b/kernel/sched/debug.c index 1e43e70..f5c529a 100644 --- a/kernel/sched/debug.c +++ b/kernel/sched/debug.c @@ -315,6 +315,7 @@ do { \ P(sched_goidle); #ifdef CONFIG_SMP P64(avg_idle); + P64(max_idle_balance_cost); #endif P(ttwu_count); -- 1.8.1.2 -- Thanks Alex -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/