| From | Vincent Guittot <> | Date | Thu, 10 Jul 2014 15:59:37 +0200 | Subject | Re: [PATCH v3 08/12] sched: move cfs task on a CPU with higher capacity |
| |
On 10 July 2014 13:24, Peter Zijlstra <peterz@infradead.org> wrote: > On Mon, Jun 30, 2014 at 06:05:39PM +0200, Vincent Guittot wrote: >> + if ((sd->flags & SD_SHARE_PKG_RESOURCES) >> + && ((capacity_orig_of(src_cpu) * 100) > (capacity_of(src_cpu) * >> + sd->imbalance_pct))) > > >> + if ((rq->cfs.h_nr_running >= 1) >> + && ((rq->cpu_capacity * sd->imbalance_pct) < >> + (rq->cpu_capacity_orig * 100))) { > > That's just weird indentation and operators should be at the end not at > the beginning of a line-break.
Ok, i'm going to fix it
|