Re: [PATCH v2] mm/hugetlb: remove redundant reservation check condition in alloc_huge_page()

2021-03-02 Thread Mike Kravetz
On 3/1/21 2:47 AM, Miaohe Lin wrote: > vma_resv_map(vma) checks if a reserve map is associated with the vma. The > routine vma_needs_reservation() will check vma_resv_map(vma) and return 1 > if no reserv map is present. map_chg is set to the return value of > vma_needs_reservation(). Therefore,

[PATCH v2] mm/hugetlb: remove redundant reservation check condition in alloc_huge_page()

2021-03-01 Thread Miaohe Lin
vma_resv_map(vma) checks if a reserve map is associated with the vma. The routine vma_needs_reservation() will check vma_resv_map(vma) and return 1 if no reserv map is present. map_chg is set to the return value of vma_needs_reservation(). Therefore, !vma_resv_map(vma) is redundant in the