On Wed, Jul 03, 2013 at 03:21:36PM +0100, Mel Gorman wrote: > static bool migrate_improves_locality(struct task_struct *p, struct lb_env *env) > { > + if (p->numa_faults[task_faults_idx(dst_nid, 1)] > > + p->numa_faults[task_faults_idx(src_nid, 1)]) > + return true; > +} > +static bool migrate_degrades_locality(struct task_struct *p, struct lb_env *env) > +{ > + if (p->numa_faults[src_nid] > p->numa_faults[dst_nid]) > return true; I bet you wanted to use task_faults_idx() there too ;-) -- 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>