diff options
author | 2025-01-20 09:43:29 -0800 | |
---|---|---|
committer | 2025-01-20 09:47:17 -0800 | |
commit | 3f3c2f0cf669ff28b995b3d6b820ab870c2aa9d9 (patch) | |
tree | 9b3a227d026d6b2eb96fe789b792099353f03310 /net/lapb/lapb_timer.c | |
parent | Merge branch 'free-htab-element-out-of-bucket-lock' (diff) | |
parent | selftests/bpf: Add some tests related to 'may_goto 0' insns (diff) | |
download | wireguard-linux-3f3c2f0cf669ff28b995b3d6b820ab870c2aa9d9.tar.xz wireguard-linux-3f3c2f0cf669ff28b995b3d6b820ab870c2aa9d9.zip |
Merge branch 'bpf-allow-may_goto-0-instruction'
Yonghong Song says:
====================
Emil Tsalapatis from Meta reported such a case where 'may_goto 0' insn is
generated by clang-19 compiler and this caused verification failure
since 'may_goto 0' is rejected by verifier.
In fact, 'may_goto 0' insn is actually a no-op and it won't hurt
verification. The only side effect is that the verifier will convert
the insn to a sequence of codes like
/* r10 - 8 stores the implicit loop count */
r11 = *(u64 *)(r10 -8)
if r11 == 0x0 goto pc+2
r11 -= 1
*(u64 *)(r10 -8) = r11
With this patch set 'may_goto 0' insns are allowed in verification which
also removes those insns.
Changelogs:
v1 -> v2:
- Instead of a separate function, removing 'may_goto 0' in existing
func opt_remove_nops().
====================
Link: https://patch.msgid.link/20250118192019.2123689-1-yonghong.song@linux.dev
Signed-off-by: Alexei Starovoitov <ast@kernel.org>
Diffstat (limited to 'net/lapb/lapb_timer.c')
0 files changed, 0 insertions, 0 deletions