+                       if (zone->nr_mem_map > 1) {
+                               /*
+                                * there may be hole in zone's memmap &&
+                                * hole is not aligned in this order.
+                                * currently, I think CONFIG_VIRTUAL_MEM_MAP
+                                * case is only case to reach here.
+                                * Is there any other case ?
+                                */
+                               /*
+                                * Is there better call than pfn_valid ?
+                                */
+                               if (!pfn_valid(zone->zone_start_pfn
+                                              + (page_idx ^ (1 << order))))
+                                       break;
+                       }

Nice try.  How about putting the ia64 code in a macro or header function
that you can #ifdef out on all the other architectures?  We used to be
able to see that entire while loop on one screen.  That's a bit harder
now.  

-- Dave

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"aart@kvack.org"> aart@kvack.org </a>