diff options
| author | Mark Brown <broonie@kernel.org> | 2021-09-21 15:23:50 +0100 |
|---|---|---|
| committer | Mark Brown <broonie@kernel.org> | 2021-09-21 15:23:50 +0100 |
| commit | 66ae258ccf400e5275b8cb4ecbfe7d25f2cb2e56 (patch) | |
| tree | 386a7b596562b7dd99163e1c80255335711e7afc /kernel/bpf/stackmap.c | |
| parent | 09134c5322df9f105d9ed324051872d5d0e162aa (diff) | |
| parent | ffb1e76f4f32d2b8ea4189df0484980370476395 (diff) | |
| download | cachepc-linux-66ae258ccf400e5275b8cb4ecbfe7d25f2cb2e56.tar.gz cachepc-linux-66ae258ccf400e5275b8cb4ecbfe7d25f2cb2e56.zip | |
Merge branch 'spi-5.15' into spi-5.16
Diffstat (limited to 'kernel/bpf/stackmap.c')
| -rw-r--r-- | kernel/bpf/stackmap.c | 10 |
1 files changed, 8 insertions, 2 deletions
diff --git a/kernel/bpf/stackmap.c b/kernel/bpf/stackmap.c index e8eefdf8cf3e..09a3fd97d329 100644 --- a/kernel/bpf/stackmap.c +++ b/kernel/bpf/stackmap.c @@ -179,7 +179,7 @@ static void stack_map_get_build_id_offset(struct bpf_stack_build_id *id_offs, * with build_id. */ if (!user || !current || !current->mm || irq_work_busy || - !mmap_read_trylock_non_owner(current->mm)) { + !mmap_read_trylock(current->mm)) { /* cannot access current->mm, fall back to ips */ for (i = 0; i < trace_nr; i++) { id_offs[i].status = BPF_STACK_BUILD_ID_IP; @@ -204,9 +204,15 @@ static void stack_map_get_build_id_offset(struct bpf_stack_build_id *id_offs, } if (!work) { - mmap_read_unlock_non_owner(current->mm); + mmap_read_unlock(current->mm); } else { work->mm = current->mm; + + /* The lock will be released once we're out of interrupt + * context. Tell lockdep that we've released it now so + * it doesn't complain that we forgot to release it. + */ + rwsem_release(¤t->mm->mmap_lock.dep_map, _RET_IP_); irq_work_queue(&work->irq_work); } } |
