On Wednesday, January 18, 2017 6:16 AM Vlastimil Babka wrote: 
> 
> @@ -3802,13 +3811,8 @@ __alloc_pages_nodemask(gfp_t gfp_mask, unsigned int 
> order,
>        * Also recalculate the starting point for the zonelist iterator or
>        * we could end up iterating over non-eligible zones endlessly.
>        */
Is the newly added comment still needed?

> -     if (unlikely(ac.nodemask != nodemask)) {
> -no_zone:
> +     if (unlikely(ac.nodemask != nodemask))
>               ac.nodemask = nodemask;
> -             ac.preferred_zoneref = first_zones_zonelist(ac.zonelist,
> -                                             ac.high_zoneidx, ac.nodemask);
> -             /* If we have NULL preferred zone, slowpath wll handle that */
> -     }
> 
>       page = __alloc_pages_slowpath(alloc_mask, order, &ac);
> 
> --
> 2.11.0

Reply via email to