diff options
| author | Eric Dumazet <edumazet@google.com> | 2018-02-23 17:12:42 +0100 |
|---|---|---|
| committer | Tejun Heo <tj@kernel.org> | 2018-02-23 17:52:34 +0100 |
| commit | accd4f36a7d11c2d54544007eb65e10604dcf2f5 (patch) | |
| tree | 49498cc42444c9e2295a895d518cee0597513806 /mm/percpu.c | |
| parent | percpu: allow select gfp to be passed to underlying allocators (diff) | |
| download | linux-accd4f36a7d11c2d54544007eb65e10604dcf2f5.tar.xz linux-accd4f36a7d11c2d54544007eb65e10604dcf2f5.zip | |
percpu: add a schedule point in pcpu_balance_workfn()
When a large BPF percpu map is destroyed, I have seen
pcpu_balance_workfn() holding cpu for hundreds of milliseconds.
On KASAN config and 112 hyperthreads, average time to destroy a chunk
is ~4 ms.
[ 2489.841376] destroy chunk 1 in 4148689 ns
...
[ 2490.093428] destroy chunk 32 in 4072718 ns
Signed-off-by: Eric Dumazet <edumazet@google.com>
Signed-off-by: Tejun Heo <tj@kernel.org>
Diffstat (limited to 'mm/percpu.c')
| -rw-r--r-- | mm/percpu.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/mm/percpu.c b/mm/percpu.c index fa3f854634a1..36e7b65ba6cf 100644 --- a/mm/percpu.c +++ b/mm/percpu.c @@ -1610,6 +1610,7 @@ static void pcpu_balance_workfn(struct work_struct *work) spin_unlock_irq(&pcpu_lock); } pcpu_destroy_chunk(chunk); + cond_resched(); } /* |