Re: [PATCH v2 0/4] mm/hugetlb: Small cleanup and improvement

2020-08-24 Thread Mike Kravetz
Hello Andrew,

Unless someone objects, can you add patches 1-3 of this series to your tree.
They have been reviewed and are fairly simple cleanups.
-- 
Mike Kravetz

On 7/22/20 8:22 PM, Baoquan He wrote:
> v1 is here:
> https://lore.kernel.org/linux-mm/20200720062623.13135-1-...@redhat.com/
> 
> Patch 1~3 are small clean up.
> 
> Patch 4 is adding warning message when the number of persistent huge
> pages is not changed to the exact value written to the sysfs or proc
> nr_hugepages file.
> 
> v1->v2:
>   Drop the old patch 1/5 in v1 post, which was thought as typo, while
>   actually another kind of abbreviation.
> 
>   Updated patch log of patch 4 which is rephrased by Mike. And move the
>   added message logging code after the hugetlb_lock dropping, this is
>   suggested by Mike.
> 
> 
> Baoquan He (4):
>   mm/hugetlb.c: make is_hugetlb_entry_hwpoisoned return bool
>   mm/hugetlb.c: Remove the unnecessary non_swap_entry()
>   doc/vm: fix typo in the hugetlb admin documentation
>   mm/hugetl.c: warn out if expected count of huge pages adjustment is
> not achieved
> 
>  Documentation/admin-guide/mm/hugetlbpage.rst |  2 +-
>  mm/hugetlb.c | 27 +++-
>  2 files changed, 21 insertions(+), 8 deletions(-)
> 


[PATCH v2 0/4] mm/hugetlb: Small cleanup and improvement

2020-07-22 Thread Baoquan He
v1 is here:
https://lore.kernel.org/linux-mm/20200720062623.13135-1-...@redhat.com/

Patch 1~3 are small clean up.

Patch 4 is adding warning message when the number of persistent huge
pages is not changed to the exact value written to the sysfs or proc
nr_hugepages file.

v1->v2:
  Drop the old patch 1/5 in v1 post, which was thought as typo, while
  actually another kind of abbreviation.

  Updated patch log of patch 4 which is rephrased by Mike. And move the
  added message logging code after the hugetlb_lock dropping, this is
  suggested by Mike.


Baoquan He (4):
  mm/hugetlb.c: make is_hugetlb_entry_hwpoisoned return bool
  mm/hugetlb.c: Remove the unnecessary non_swap_entry()
  doc/vm: fix typo in the hugetlb admin documentation
  mm/hugetl.c: warn out if expected count of huge pages adjustment is
not achieved

 Documentation/admin-guide/mm/hugetlbpage.rst |  2 +-
 mm/hugetlb.c | 27 +++-
 2 files changed, 21 insertions(+), 8 deletions(-)

-- 
2.17.2