diff options
| author | Eric Dumazet <edumazet@google.com> | 2017-12-12 14:22:39 -0800 |
|---|---|---|
| committer | Alexei Starovoitov <ast@kernel.org> | 2017-12-12 15:27:22 -0800 |
| commit | 9147efcbe0b7cc96b18eb64b1a3f0d4bba81443c (patch) | |
| tree | e96ef5a6e898fd45ca4d48f467f71fa505ff807e /kernel | |
| parent | 9dbd2d948e7345595e5b90e44795c1e8ca3e5728 (diff) | |
| download | linux-0-day-9147efcbe0b7cc96b18eb64b1a3f0d4bba81443c.tar.gz linux-0-day-9147efcbe0b7cc96b18eb64b1a3f0d4bba81443c.tar.xz | |
bpf: add schedule points to map alloc/free
While using large percpu maps, htab_map_alloc() can hold
cpu for hundreds of ms.
This patch adds cond_resched() calls to percpu alloc/free
call sites, all running in process context.
Signed-off-by: Eric Dumazet <edumazet@google.com>
Signed-off-by: Alexei Starovoitov <ast@kernel.org>
Diffstat (limited to 'kernel')
| -rw-r--r-- | kernel/bpf/hashtab.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/kernel/bpf/hashtab.c b/kernel/bpf/hashtab.c index e469e05c8e83b..3905d4bc5b80d 100644 --- a/kernel/bpf/hashtab.c +++ b/kernel/bpf/hashtab.c @@ -114,6 +114,7 @@ static void htab_free_elems(struct bpf_htab *htab) pptr = htab_elem_get_ptr(get_htab_elem(htab, i), htab->map.key_size); free_percpu(pptr); + cond_resched(); } free_elems: bpf_map_area_free(htab->elems); @@ -159,6 +160,7 @@ static int prealloc_init(struct bpf_htab *htab) goto free_elems; htab_elem_set_ptr(get_htab_elem(htab, i), htab->map.key_size, pptr); + cond_resched(); } skip_percpu_elems: |