diff options
| author | Baoquan He <bhe@redhat.com> | 2022-10-24 10:14:32 +0200 |
|---|---|---|
| committer | Dennis Zhou <dennis@kernel.org> | 2022-11-08 07:59:15 +0100 |
| commit | 83d261fc9e5fb03e8c32e365ca4ee53952611a2b (patch) | |
| tree | 784cb37cf995c8e061eb2d5da9fb0b028989411b /mm/percpu.c | |
| parent | mm/percpu: add comment to state the empty populated pages accounting (diff) | |
| download | linux-83d261fc9e5fb03e8c32e365ca4ee53952611a2b.tar.xz linux-83d261fc9e5fb03e8c32e365ca4ee53952611a2b.zip | |
mm/percpu: replace the goto with break
In function pcpu_reclaim_populated(), the line of goto jumping is
unnecessary since the label 'end_chunk' is near the end of the for
loop, use break instead.
Signed-off-by: Baoquan He <bhe@redhat.com>
Signed-off-by: Dennis Zhou <dennis@kernel.org>
Diffstat (limited to 'mm/percpu.c')
| -rw-r--r-- | mm/percpu.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/mm/percpu.c b/mm/percpu.c index 2a7313b56254..a223b3120d33 100644 --- a/mm/percpu.c +++ b/mm/percpu.c @@ -2167,7 +2167,7 @@ static void pcpu_reclaim_populated(void) /* reintegrate chunk to prevent atomic alloc failures */ if (pcpu_nr_empty_pop_pages < PCPU_EMPTY_POP_PAGES_HIGH) { reintegrate = true; - goto end_chunk; + break; } /* @@ -2203,7 +2203,6 @@ static void pcpu_reclaim_populated(void) end = -1; } -end_chunk: /* batch tlb flush per chunk to amortize cost */ if (freed_page_start < freed_page_end) { spin_unlock_irq(&pcpu_lock); |