A-Team_Digital_Solutions/android_kernel_motorola_sm6375
Watch
2
0
Fork
You've already forked android_kernel_motorola_sm6375
 0
mirror of https://github.com/BobTheBlinker/android_kernel_motorola_sm6375.git synced 2026-10-10 22:40:54 -04:00
Code Issues Projects Releases Packages Wiki Activity

UPSTREAM: bpf: Enforce preallocation for instrumentation programs on RT

Browse source 
Aside of the general unsafety of run-time map allocation for
instrumentation type programs RT enabled kernels have another constraint:

The instrumentation programs are invoked with preemption disabled, but the
memory allocator spinlocks cannot be acquired in atomic context because
they are converted to 'sleeping' spinlocks on RT.

Therefore enforce map preallocation for these programs types when RT is
enabled.

Change-Id: I6eb71f8165d313c13679f751a8091b1a4e8690e5
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Signed-off-by: Alexei Starovoitov <ast@kernel.org>
Link: https://lore.kernel.org/bpf/20200224145642.648784007@linutronix.de
This commit is contained in:
Thomas Gleixner 2020-02-24 15:01:33 +01:00 • committed by basamaryan
commit c59465dd5d
No known key found for this signature in database
GPG key ID: 210D5384FB2A5885
1 changed files with 9 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

13
kernel/bpf/verifier.c
View file

@ -8422,16 +8422,21 @@ static int check_map_prog_compatibility(struct bpf_verifier_env *env,
* of the memory allocator or at a place where a recursion into the
* memory allocator would see inconsistent state.
*
* For now running such programs is allowed for backwards
* compatibility reasons, but warnings are emitted so developers are
* made aware of the unsafety and can fix their programs before this
* is enforced.
* On RT enabled kernels run-time allocation of all trace type
* programs is strictly prohibited due to lock type constraints. On
* !RT kernels it is allowed for backwards compatibility reasons for
* now, but warnings are emitted so developers are made aware of
* the unsafety and can fix their programs before this is enforced.
*/
if (is_tracing_prog_type(prog->type) && !is_preallocated_map(map)) {
if (prog->type == BPF_PROG_TYPE_PERF_EVENT) {
verbose(env, "perf_event programs can only use preallocated hash map\n");
return -EINVAL;
}
if (IS_ENABLED(CONFIG_PREEMPT_RT)) {
verbose(env, "trace type programs can only use preallocated hash map\n");
return -EINVAL;
}
WARN_ONCE(1, "trace type BPF program uses run-time allocation\n");
verbose(env, "trace type programs with run-time allocated hash maps are unsafe. Switch to preallocated hash maps.\n");
}