On Thu, Dec 27, 2012 at 11:42 PM, Zlatko Calusic <zlatko.calusic@xxxxxxxx> wrote: > On 21.12.2012 12:51, Hillf Danton wrote: >> >> On Thu, Dec 20, 2012 at 7:25 AM, Zlatko Calusic <zlatko.calusic@xxxxxxxx> >> wrote: >>> >>> static unsigned long balance_pgdat(pg_data_t *pgdat, int order, >>> int >>> *classzone_idx) >>> { >>> - int all_zones_ok; >>> + struct zone *unbalanced_zone; >> >> >> nit: less hunks if not erase that mark >> >> Hillf > > > This one left unanswered and forgotten because I didn't understand what you > meant. Could you elaborate? > Sure, the patch looks simpler(and nicer) if we dont erase all_zones_ok. Hillf -- 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>