On Tue, Sep 11, 2012 at 04:29:46PM +0800, Shaohua Li wrote: > On Tue, Sep 11, 2012 at 10:45:55AM +0900, Minchan Kim wrote: > > On Mon, Sep 10, 2012 at 09:18:30AM +0800, Shaohua Li wrote: > > > isolate_migratepages_range() might isolate none pages, for example, when > > > zone->lru_lock is contended and compaction is async. In this case, we should > > > abort compaction, otherwise, compact_zone will run a useless loop and make > > > zone->lru_lock is even contended. > > > > > > > As I read old thread, you have the scenario and number. > > Please include them in this description. > > that is just to show how the contention is, not performance data. I thought > explaining the contention is enough. How did you think? If you measured it with perf, you had data which is percentage of sampling of the lock. It's enough. I meant it, NOT performance data. > > -- > 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> -- Kind regards, Minchan Kim -- 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>