mm/memory.c: make tlb_next_batch() return bool
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>
This commit is contained in:

committed by
Linus Torvalds

parent
d9e7e37b4d
commit
ca1d6c7d9d
10
mm/memory.c
10
mm/memory.c
@@ -181,22 +181,22 @@ static void check_sync_rss_stat(struct task_struct *task)
|
|||||||
|
|
||||||
#ifdef HAVE_GENERIC_MMU_GATHER
|
#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;
|
struct mmu_gather_batch *batch;
|
||||||
|
|
||||||
batch = tlb->active;
|
batch = tlb->active;
|
||||||
if (batch->next) {
|
if (batch->next) {
|
||||||
tlb->active = batch->next;
|
tlb->active = batch->next;
|
||||||
return 1;
|
return true;
|
||||||
}
|
}
|
||||||
|
|
||||||
if (tlb->batch_count == MAX_GATHER_BATCH_COUNT)
|
if (tlb->batch_count == MAX_GATHER_BATCH_COUNT)
|
||||||
return 0;
|
return false;
|
||||||
|
|
||||||
batch = (void *)__get_free_pages(GFP_NOWAIT | __GFP_NOWARN, 0);
|
batch = (void *)__get_free_pages(GFP_NOWAIT | __GFP_NOWARN, 0);
|
||||||
if (!batch)
|
if (!batch)
|
||||||
return 0;
|
return false;
|
||||||
|
|
||||||
tlb->batch_count++;
|
tlb->batch_count++;
|
||||||
batch->next = NULL;
|
batch->next = NULL;
|
||||||
@@ -206,7 +206,7 @@ static int tlb_next_batch(struct mmu_gather *tlb)
|
|||||||
tlb->active->next = batch;
|
tlb->active->next = batch;
|
||||||
tlb->active = batch;
|
tlb->active = batch;
|
||||||
|
|
||||||
return 1;
|
return true;
|
||||||
}
|
}
|
||||||
|
|
||||||
/* tlb_gather_mmu
|
/* tlb_gather_mmu
|
||||||
|
Reference in New Issue
Block a user