On Fri, Dec 02, 2016 at 02:56:43PM +0100, Michal Hocko wrote:

On Mon 14-11-16 15:07:36, Huang Shijie wrote:

quoted

This patch changes the return type to "struct page*" for
alloc_fresh_gigantic_page().

OK, this makes somme sense. Other hugetlb allocation function (and page
allocator in general) return struct page as well. Besides that int would
make sense if we wanted to convey an error code but 0 vs. 1 just doesn't
make any sense.

But if you are changing that then alloc_fresh_huge_page should be
changed as well.

Okay.

quoted

This patch makes preparation for later patch.

Signed-off-by: Huang Shijie <redacted>

Acked-by: Michal Hocko <mhocko@suse.com>

Thanks a lot.

Thanks
Huang Shijie