Re: [PATCH v3 2/3] mm: return zero_resv_unavail optimization

From: Oscar Salvador
Date: Wed Oct 10 2018 - 05:39:14 EST


On Tue, Oct 02, 2018 at 10:38:20AM -0400, Masayoshi Mizuma wrote:
> From: Pavel Tatashin <pavel.tatashin@xxxxxxxxxxxxx>
>
> When checking for valid pfns in zero_resv_unavail(), it is not necessary to
> verify that pfns within pageblock_nr_pages ranges are valid, only the first
> one needs to be checked. This is because memory for pages are allocated in
> contiguous chunks that contain pageblock_nr_pages struct pages.
>
> Signed-off-by: Pavel Tatashin <pavel.tatashin@xxxxxxxxxxxxx>
> Reviewed-by: Masayoshi Mizuma <m.mizuma@xxxxxxxxxxxxxx>
> Acked-by: Naoya Horiguchi <n-horiguchi@xxxxxxxxxxxxx>

Looks good to me

Reviewed-by: Oscar Salvador <osalvador@xxxxxxx>

Thanks
--
Oscar Salvador
SUSE L3