On Thu, May 13, 2010 at 12:25:36PM +0900, KOSAKI Motohiro wrote: > > @@ -1789,7 +1787,7 @@ static unsigned long do_try_to_free_page > > sc->nr_scanned = 0; > > if (!priority) > > disable_swap_token(); > > - shrink_zones(priority, zonelist, sc); > > + ret = shrink_zones(priority, zonelist, sc); > > Please use more good name instead 'ret' ;) Guess you are right, let me send a follow-up. > otherwise looks good. > Reviewed-by: KOSAKI Motohiro <kosaki.motohiro@xxxxxxxxxxxxxx> Thank you. Hannes -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@xxxxxxxxxx For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: <a href=mailto:"dont@xxxxxxxxx";> email@xxxxxxxxx </a>