Re: [PATCH] hugetlbfs: remove meaningless variable avoid_reserve

2021-01-19 Thread Miaohe Lin
Hi: On 2021/1/20 2:41, Mike Kravetz wrote: > Please CC Andrew on hugetlb patches as they need to go through his tree. > > On 1/16/21 1:26 AM, Miaohe Lin wrote: >> The variable avoid_reserve is meaningless because we never changed its >> value and just passed it to alloc_huge_page(). So remove it

Re: [PATCH] hugetlbfs: remove meaningless variable avoid_reserve

2021-01-19 Thread Mike Kravetz
Please CC Andrew on hugetlb patches as they need to go through his tree. On 1/16/21 1:26 AM, Miaohe Lin wrote: > The variable avoid_reserve is meaningless because we never changed its > value and just passed it to alloc_huge_page(). So remove it to make code > more clear that in

Re: [PATCH] hugetlbfs: remove meaningless variable avoid_reserve

2021-01-18 Thread David Hildenbrand
On 16.01.21 10:26, Miaohe Lin wrote: > The variable avoid_reserve is meaningless because we never changed its > value and just passed it to alloc_huge_page(). So remove it to make code > more clear that in hugetlbfs_fallocate, we never avoid reserve when alloc > hugepage yet. > > Signed-off-by:

Re: [PATCH] hugetlbfs: remove meaningless variable avoid_reserve

2021-01-16 Thread Souptick Joarder
On Sat, Jan 16, 2021 at 2:57 PM Miaohe Lin wrote: > > The variable avoid_reserve is meaningless because we never changed its > value and just passed it to alloc_huge_page(). So remove it to make code > more clear that in hugetlbfs_fallocate, we never avoid reserve when alloc > hugepage yet. > >

[PATCH] hugetlbfs: remove meaningless variable avoid_reserve

2021-01-16 Thread Miaohe Lin
The variable avoid_reserve is meaningless because we never changed its value and just passed it to alloc_huge_page(). So remove it to make code more clear that in hugetlbfs_fallocate, we never avoid reserve when alloc hugepage yet. Signed-off-by: Miaohe Lin --- fs/hugetlbfs/inode.c | 3 +-- 1