Re: [PATCH v2 3/4] hugetlbfs: remove hugetlb_add_hstate() warning for existing hstate

From: Peter Xu
Date: Fri Apr 10 2020 - 15:34:28 EST


On Wed, Apr 01, 2020 at 11:38:18AM -0700, Mike Kravetz wrote:

[...]

> @@ -3255,7 +3254,6 @@ void __init hugetlb_add_hstate(unsigned int order)
> unsigned long i;
>
> if (size_to_hstate(PAGE_SIZE << order)) {
> - pr_warn("hugepagesz= specified twice, ignoring\n");
> return;
> }

Nitpick: I think the brackets need to be removed to follow linux
coding style. With that:

Reviewed-by: Peter Xu <peterx@xxxxxxxxxx>

--
Peter Xu