Add simple unit-style tests for rte_bpf_load, and fix some minor
discovered bugs.

v4:
* Add Fixes: and Cc: [email protected] tags to all patches except 3. Try
  to make sure [email protected] does not receive patch 3.
* Ensure no (possibly hallucinated) trailing whitespace after Acked-by.
* Improve subject line for the "conditional jump first" patch.
* Update REGISTER_FAST_TEST() calls in app/test/test_bpf.c to use new
  constants (NOHUGE_OK, ASAN_OK) instead of old boolean values.

v3:
* Fixed unfortunate typos in the last patch commit message.
* Marked all ARM opcodes with UINT32_C for clarity as suggested by
  Morten Brørup, in a separate commit.

v2:
* Moved new tests back into test_bpf.c
* Changed library macros RTE_SHIFT_VALxx to handle variable first
  arguments instead of introducing new macros.
* Added another test and fix, for conditional jump as first instruction.

Marat Khalili (6):
  eal: variable first arguments of RTE_SHIFT_VALxx
  bpf: fix signed shift overflows in ARM JIT
  bpf: mark ARM opcodes with UINT32_C
  bpf: disallow empty program
  bpf: make add/subtract one program validate
  bpf: fix validation when conditional jump is first instruction

 app/test/test_bpf.c          | 254 +++++++++++++++++++++++++++++++++++
 lib/bpf/bpf_jit_arm64.c      | 184 ++++++++++++-------------
 lib/bpf/bpf_load.c           |   2 +-
 lib/bpf/bpf_validate.c       |  48 ++++---
 lib/eal/include/rte_bitops.h |   4 +-
 5 files changed, 375 insertions(+), 117 deletions(-)

-- 
2.34.1

Reply via email to