Re: [PATCH 04/25] Check only once if the zonelist is suitable for the allocation

KOSAKI Motohiro
2009-04-21T03:03:45Z

> -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>