Skip to content

mm/hugetlb: Small cleanup and improvement

Baoquan He linux-mm lkml 2 versions – In review

Get series with b4
b4 am [email protected]

Versions newest first · linked by author and title

  1. v2 Latest4 patches 15 repliesIn review 2 of 4 applied
  2. v15 patches 12 repliesSuperseded

Changes in v2 from the cover letter

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

Patches in v2 2 of 4 applied

#Subject and filesSizeRepliesApplied
1/4mm/hugetlb.c: make is_hugetlb_entry_hwpoisoned return boolmm/hugetlb.c+4 -41 reply
2/4mm/hugetlb.c: Remove the unnecessary non_swap_entry()mm/hugetlb.c+2 -24 repliesApplied
3/4doc/vm: fix typo in the hugetlb admin documentationDocumentation/admin-guide/mm/hugetlbpage.rst+1 -12 repliesApplied
4/4mm/hugetl.c: warn out if expected count of huge pages adjustment is not achievedmm/hugetlb.c+14 -18 replies

Reviews trailers per patch

3 of 4 patches have a Reviewed-by or Acked-by.

PatchMike KravetzDavid HildenbrandAnshuman Khandual
1/4R-b R-b R-b
2/4R-b R-b R-b R-b R-b R-b
3/4R-b R-b
4/4

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes. Scroll the table sideways for all 3 reviewers.

Read the v2 thread

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

v1 is here:
https://lore.kernel.org/linux-mm/[email protected]/

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(-)
Signature · 1 line
2.17.2