Re: [PATCH v2 12/20] mm, hugetlb: remove vma_has_reserves()

2013-08-22 Thread Joonsoo Kim
On Thu, Aug 22, 2013 at 04:34:22PM +0530, Aneesh Kumar K.V wrote: > Joonsoo Kim writes: > > > On Thu, Aug 22, 2013 at 02:14:38PM +0530, Aneesh Kumar K.V wrote: > >> Joonsoo Kim writes: > >> > >> > vma_has_reserves() can be substituted by using return value of > >> > vma_needs_reservation(). If

Re: [PATCH v2 12/20] mm, hugetlb: remove vma_has_reserves()

2013-08-22 Thread Aneesh Kumar K.V
Joonsoo Kim writes: > On Thu, Aug 22, 2013 at 02:14:38PM +0530, Aneesh Kumar K.V wrote: >> Joonsoo Kim writes: >> >> > vma_has_reserves() can be substituted by using return value of >> > vma_needs_reservation(). If chg returned by vma_needs_reservation() >> > is 0, it means that vma has reserve

Re: [PATCH v2 12/20] mm, hugetlb: remove vma_has_reserves()

2013-08-22 Thread Joonsoo Kim
On Thu, Aug 22, 2013 at 02:14:38PM +0530, Aneesh Kumar K.V wrote: > Joonsoo Kim writes: > > > vma_has_reserves() can be substituted by using return value of > > vma_needs_reservation(). If chg returned by vma_needs_reservation() > > is 0, it means that vma has reserves. Otherwise, it means that v

Re: [PATCH v2 12/20] mm, hugetlb: remove vma_has_reserves()

2013-08-22 Thread Aneesh Kumar K.V
Joonsoo Kim writes: > vma_has_reserves() can be substituted by using return value of > vma_needs_reservation(). If chg returned by vma_needs_reservation() > is 0, it means that vma has reserves. Otherwise, it means that vma don't > have reserves and need a hugepage outside of reserve pool. This d

[PATCH v2 12/20] mm, hugetlb: remove vma_has_reserves()

2013-08-09 Thread Joonsoo Kim
vma_has_reserves() can be substituted by using return value of vma_needs_reservation(). If chg returned by vma_needs_reservation() is 0, it means that vma has reserves. Otherwise, it means that vma don't have reserves and need a hugepage outside of reserve pool. This definition is perfectly same as