Thread (7 messages) 7 messages, 2 authors, 2021-04-27

Re: [PATCH v2 bpf-next 0/5] CO-RE relocation selftests fixes

From: Alexei Starovoitov <hidden>
Date: 2021-04-27 01:43:28
Also in: bpf

On Mon, Apr 26, 2021 at 12:30 PM Andrii Nakryiko [off-list ref] wrote:
Lorenz Bauer noticed that core_reloc selftest has two inverted CHECK()
conditions, allowing failing tests to pass unnoticed. Fixing that opened up
few long-standing (field existence and direct memory bitfields) and one recent
failures (BTF_KIND_FLOAT relos).

This patch set fixes core_reloc selftest to capture such failures reliably in
the future. It also fixes all the newly failing tests. See individual patches
for details.

This patch set also completes a set of ASSERT_xxx() macros, so now there
should be a very little reason to use verbose and error-prone generic CHECK()
macro.

v1->v2:
  - updated bpf_core_fields_are_compat() comment to mention FLOAT (Lorenz).
Applied.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help