Re: [PATCH 2/2] sched: add statistic for rq->max_idle_balance_cost

From: Jason Low
Date: Wed Jan 22 2014 - 13:10:22 EST


On Wed, 2014-01-22 at 17:09 +0100, Peter Zijlstra wrote:
> On Wed, Jan 22, 2014 at 04:24:13PM +0800, Alex Shi wrote:
> > From: Alex Shi <alex.shi@xxxxxxxxxx>
> > 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.shi@xxxxxxxxxx>
> > ---
> > 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);
>
> Not also the per-sd value in sd_alloc_ctl_domain_table() ?

Yeah, tracking the sd->max_newidle_lb_cost can also be useful.


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/