bpf: Add BPF_F_ANY_ALIGNMENT.

Often we want to write tests cases that check things like bad context
offset accesses.  And one way to do this is to use an odd offset on,
for example, a 32-bit load.

This unfortunately triggers the alignment checks first on platforms
that do not set CONFIG_EFFICIENT_UNALIGNED_ACCESS.  So the test
case see the alignment failure rather than what it was testing for.

It is often not completely possible to respect the original intention
of the test, or even test the same exact thing, while solving the
alignment issue.

Another option could have been to check the alignment after the
context and other validations are performed by the verifier, but
that is a non-trivial change to the verifier.

Signed-off-by: David S. Miller <davem@davemloft.net>
Signed-off-by: Alexei Starovoitov <ast@kernel.org>

2 files changed, 5 insertions, 5 deletions

diff --git a/tools/lib/bpf/bpf.c b/tools/lib/bpf/bpf.c
index ce1822194590..c19226cccf39 100644
--- a/tools/lib/bpf/bpf.c
+++ b/tools/lib/bpf/bpf.c
@@ -279,9 +279,9 @@ int bpf_load_program(enum bpf_prog_type type, const struct bpf_insn *insns,
 }
 
 int bpf_verify_program(enum bpf_prog_type type, const struct bpf_insn *insns,
-                       size_t insns_cnt, int strict_alignment,
-                       const char *license, __u32 kern_version,
-                       char *log_buf, size_t log_buf_sz, int log_level)
+                       size_t insns_cnt, __u32 prog_flags, const char *license,
+                       __u32 kern_version, char *log_buf, size_t log_buf_sz,
+                       int log_level)
 {
         union bpf_attr attr;
 
@@ -295,7 +295,7 @@ int bpf_verify_program(enum bpf_prog_type type, const struct bpf_insn *insns,
         attr.log_level = log_level;
         log_buf[0] = 0;
         attr.kern_version = kern_version;
-        attr.prog_flags = strict_alignment ? BPF_F_STRICT_ALIGNMENT : 0;
+        attr.prog_flags = prog_flags;
 
         return sys_bpf(BPF_PROG_LOAD, &attr, sizeof(attr));
 }
diff --git a/tools/lib/bpf/bpf.h b/tools/lib/bpf/bpf.h
index 09e8bbe111d4..60392b70587c 100644
--- a/tools/lib/bpf/bpf.h
+++ b/tools/lib/bpf/bpf.h
@@ -98,7 +98,7 @@ LIBBPF_API int bpf_load_program(enum bpf_prog_type type,
                                 char *log_buf, size_t log_buf_sz);
 LIBBPF_API int bpf_verify_program(enum bpf_prog_type type,
                                   const struct bpf_insn *insns,
-                                  size_t insns_cnt, int strict_alignment,
+                                  size_t insns_cnt, __u32 prog_flags,
                                   const char *license, __u32 kern_version,
                                   char *log_buf, size_t log_buf_sz,
                                   int log_level);