On 06/04/2015 06:04 AM, Xishi Qiu wrote: > spin_lock(&zone->lock); > for (i = 0; i < count; ++i) { > - struct page *page = __rmqueue(zone, order, migratetype); > + struct page *page; > + > + if (is_migrate_mirror(migratetype)) > + page = __rmqueue_smallest(zone, order, migratetype); > + else > + page = __rmqueue(zone, order, migratetype); > if (unlikely(page == NULL)) > break; Why is this necessary/helpful? The changelog doesn't tell me either. :( Why was this code modified in stead of putting the changes in __rmqueue() itself (like CMA did)? -- 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>