On 09/17/22 09:15, Xin Hao wrote:Ok, i will remove the changelog in the next patch.
In hugetlb.c file, there are several places to compare the values ofThanks, I think the code looks a little better with this change.
'h->free_huge_pages' and 'h->resv_huge_pages', it looks a bit messy, so
there add a new available_huge_pages() func to do these.
Signed-off-by: Xin Hao <xhao@xxxxxxxxxxxxxxxxx>I believe the above 'Changes from v2' information should be below the marker
Changes from v2
https://lore.kernel.org/linux-mm/20220916064127.1904-1-xhao@xxxxxxxxxxxxxxxxx/
- Convert is_resv_equal_free() to available_huge_pages()
'---' as it will not be included in the changelog.
---The changes look fine,
mm/hugetlb.c | 14 +++++++++-----
1 file changed, 9 insertions(+), 5 deletions(-)
Reviewed-by: Mike Kravetz <mike.kravetz@xxxxxxxxxx>