On Sun, Jan 26, 2014 at 07:52:24PM -0800, Davidlohr Bueso wrote: > From: Joonsoo Kim <iamjoonsoo.kim@xxxxxxx> > > 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 vma_has_reserves(), so remove vma_has_reserves(). I'm concerned that this patch doesn't work when VM_NORESERVE is set. vma_needs_reservation() doesn't check VM_NORESERVE and this patch changes dequeue_huge_page_vma() not to check it. So no one seems to check it any more. It would be nice if you add some comment about the justification in changelog. Does the testcase attached in commit af0ed73e699b still pass with this patch? (I'm not sure it's covered in libhugetlbfs test suite.) Thanks, Naoya Horiguchi -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@xxxxxxxxx. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: <a href=mailto:"dont@xxxxxxxxx";> email@xxxxxxxxx </a>