mm/hugetlb.c: make vma_has_reserves() return bool
authorNicholas Krause <xerofoify@gmail.com>
Fri, 4 Sep 2015 22:48:27 +0000 (15:48 -0700)
committerLinus Torvalds <torvalds@linux-foundation.org>
Fri, 4 Sep 2015 23:54:41 +0000 (16:54 -0700)
This makes vma_has_reserves() return bool due to this particular function
only returning either one or zero as its return value.

Signed-off-by: Nicholas Krause <xerofoify@gmail.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
mm/hugetlb.c

index 5d1d84ca967493b50f09dccac64fafafcdc2a1fe..51ae41d0fbc0d8ba3556e2f272c90442d90b2ab2 100644 (file)
@@ -616,7 +616,7 @@ void reset_vma_resv_huge_pages(struct vm_area_struct *vma)
 }
 
 /* Returns true if the VMA has associated reserve pages */
-static int vma_has_reserves(struct vm_area_struct *vma, long chg)
+static bool vma_has_reserves(struct vm_area_struct *vma, long chg)
 {
        if (vma->vm_flags & VM_NORESERVE) {
                /*
@@ -629,23 +629,23 @@ static int vma_has_reserves(struct vm_area_struct *vma, long chg)
                 * properly, so add work-around here.
                 */
                if (vma->vm_flags & VM_MAYSHARE && chg == 0)
-                       return 1;
+                       return true;
                else
-                       return 0;
+                       return false;
        }
 
        /* Shared mappings always use reserves */
        if (vma->vm_flags & VM_MAYSHARE)
-               return 1;
+               return true;
 
        /*
         * Only the process that called mmap() has reserves for
         * private mappings.
         */
        if (is_vma_resv_set(vma, HPAGE_RESV_OWNER))
-               return 1;
+               return true;
 
-       return 0;
+       return false;
 }
 
 static void enqueue_huge_page(struct hstate *h, struct page *page)