On Fri, Jun 28, 2013 at 02:45:35PM +0100, Mel Gorman wrote: > > Also, until I just actually _read_ that function; I assumed it would > > compare p->numa_faults[src_nid] and p->numa_faults[dst_nid]. Because > > even when the dst_nid isn't the preferred nid; it might still have more > > pages than where we currently are. > > > > I tested something like this and also tested it when only taking shared > accesses into account but it performed badly in some cases. I've included > the last patch I tested below for reference but dropped it until I figured > out why it performed badly. I guessed it was due to increased bouncing > due to shared faults but didn't prove it. Oh, interesting. Yeah it would be good to figure out why that gave funnies. -- 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>