mm/memory.c: make tlb_next_batch() return bool
authorNicholas Krause <xerofoify@gmail.com>
Fri, 4 Sep 2015 22:48:22 +0000 (15:48 -0700)
committerLinus Torvalds <torvalds@linux-foundation.org>
Fri, 4 Sep 2015 23:54:41 +0000 (16:54 -0700)
This makes the tlb_next_batch() bool due to this particular function only
ever 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/memory.c

index 2961fb654369e25b720e0d5044f5b75e881b9399..bb04d8f2f86c415c24a52dadd71623a2316454d1 100644 (file)
@@ -181,22 +181,22 @@ static void check_sync_rss_stat(struct task_struct *task)
 
 #ifdef HAVE_GENERIC_MMU_GATHER
 
-static int tlb_next_batch(struct mmu_gather *tlb)
+static bool tlb_next_batch(struct mmu_gather *tlb)
 {
        struct mmu_gather_batch *batch;
 
        batch = tlb->active;
        if (batch->next) {
                tlb->active = batch->next;
-               return 1;
+               return true;
        }
 
        if (tlb->batch_count == MAX_GATHER_BATCH_COUNT)
-               return 0;
+               return false;
 
        batch = (void *)__get_free_pages(GFP_NOWAIT | __GFP_NOWARN, 0);
        if (!batch)
-               return 0;
+               return false;
 
        tlb->batch_count++;
        batch->next = NULL;
@@ -206,7 +206,7 @@ static int tlb_next_batch(struct mmu_gather *tlb)
        tlb->active->next = batch;
        tlb->active = batch;
 
-       return 1;
+       return true;
 }
 
 /* tlb_gather_mmu