On Tue, Dec 10, 2013 at 05:19:31PM +0800, Wanpeng Li wrote: > Use wrapper function task_faults_idx to calculate index in group_faults. yes, it syncs group_faults() with task_faults(). > Signed-off-by: Wanpeng Li <liwanp@xxxxxxxxxxxxxxxxxx> Reviewed-by: Naoya Horiguchi <n-horiguchi@xxxxxxxxxxxxx> > --- > kernel/sched/fair.c | 3 ++- > 1 files changed, 2 insertions(+), 1 deletions(-) > > diff --git a/kernel/sched/fair.c b/kernel/sched/fair.c > index e0b1063..7073c76 100644 > --- a/kernel/sched/fair.c > +++ b/kernel/sched/fair.c > @@ -935,7 +935,8 @@ static inline unsigned long group_faults(struct task_struct *p, int nid) > if (!p->numa_group) > return 0; > > - return p->numa_group->faults[2*nid] + p->numa_group->faults[2*nid+1]; > + return p->numa_group->faults[task_faults_idx(nid, 0)] + > + p->numa_group->faults[task_faults_idx(nid, 1)]; > } > > /* > -- > 1.7.7.6 > > -- > To unsubscribe, send a message with 'unsubscribe linux-mm' in > the body to majordomo@xxxxxxxxx. For more info on Linux MM, > see: http://www.linux-mm.org/ . > Don't email: <a href=mailto:"dont@xxxxxxxxx";> email@xxxxxxxxx </a> > -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@xxxxxxxxx. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: <a href=mailto:"dont@xxxxxxxxx";> email@xxxxxxxxx </a>