selftests/bpf: test_progs avoid minus shell exit codes
authorJesper Dangaard Brouer <brouer@redhat.com>
Tue, 7 Jul 2020 07:12:25 +0000 (09:12 +0200)
committerDaniel Borkmann <daniel@iogearbox.net>
Wed, 8 Jul 2020 22:35:33 +0000 (00:35 +0200)
commitb8c50df0cb3eb9008f8372e4ff0317eee993b8d1
treefc555b91bf4d84ab7bbac23fe87ef543fe1111b0
parent3220fb667842a9725cbb71656f406eadb03c094b
selftests/bpf: test_progs avoid minus shell exit codes

There are a number of places in test_progs that use minus-1 as the argument
to exit(). This is confusing as a process exit status is masked to be a
number between 0 and 255 as defined in man exit(3). Thus, users will see
status 255 instead of minus-1.

This patch use positive exit code 3 instead of minus-1. These cases are put
in the same group of infrastructure setup errors.

Fixes: fd27b1835e70 ("selftests/bpf: Reset process and thread affinity after each test/sub-test")
Fixes: 811d7e375d08 ("bpf: selftests: Restore netns after each test")
Signed-off-by: Jesper Dangaard Brouer <brouer@redhat.com>
Signed-off-by: Daniel Borkmann <daniel@iogearbox.net>
Acked-by: Andrii Nakryiko <andriin@fb.com>
Link: https://lore.kernel.org/bpf/159410594499.1093222.11080787853132708654.stgit@firesoul
tools/testing/selftests/bpf/test_progs.c