| From | KOSAKI Motohiro <> | Subject | Re: [PATCH 04/25] Check only once if the zonelist is suitable for the allocation | Date | Tue, 21 Apr 2009 12:03:31 +0900 (JST) |
| |
> -restart: > - z = zonelist->_zonerefs; /* the list of zones suitable for gfp_mask */ > - > + /* the list of zones suitable for gfp_mask */ > + z = zonelist->_zonerefs; > if (unlikely(!z->zone)) { > /* > * Happens if we have an empty zonelist as a result of > @@ -1497,6 +1496,7 @@ restart: > return NULL; > } > > +restart: > page = get_page_from_freelist(gfp_mask|__GFP_HARDWALL, nodemask, order, > zonelist, high_zoneidx, ALLOC_WMARK_LOW|ALLOC_CPUSET); > if (page)
looks good.
Reviewed-by: KOSAKI Motohiro <kosaki.motohiro@jp.fujitsu.com>
|