On 06/11/2014 04:12 AM, Minchan Kim wrote:
>@@ -314,6 +315,9 @@ static unsigned long isolate_freepages_block(struct compact_control *cc,
> int isolated, i;
> struct page *page = cursor;
>
>+ /* Record how far we have got within the block */
>+ *start_pfn = blockpfn;
>+
Couldn't we move this out of the loop for just one store?
You mean using a local variable inside the loop, and assigning once, for
performance reasons (register vs memory access)?
--
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>