This is the mail archive of the
elfutils-devel@sourceware.org
mailing list for the elfutils project.
Re: [PATCH] Fix BPF check for kernels < 3.20
- From: Mark Wielaard <mjw at redhat dot com>
- To: elfutils-devel at lists dot fedorahosted dot org
- Date: Wed, 31 Aug 2016 16:21:11 +0200
- Subject: Re: [PATCH] Fix BPF check for kernels < 3.20
On Tue, 2016-08-30 at 23:38 +0200, Julian Ospald wrote:
> -dnl Check if we have <linux/bpf.h> for EM_BPF disassembly.
> -AC_CHECK_HEADERS(linux/bpf.h)
> -AM_CONDITIONAL(HAVE_LINUX_BPF_H, [test "x$ac_cv_header_linux_bpf_h" = "xyes"])
> +dnl Check if we have <linux/bpf.h> for EM_BPF disassembly and it has the required defines
> +AC_COMPILE_IFELSE(
> + [AC_LANG_PROGRAM(
> + [#include <linux/bpf.h>],
> + [int foo=BPF_PSEUDO_MAP_FD; return 0;]
> + )],
> + [have_bpf="true"],
> + [have_bpf="false"]
> +)
> +
> +if test "$have_bpf" = "true" ; then
> + AC_DEFINE([HAVE_LINUX_BPF_H], [1], [if we have <linux/bpf.h> for EM_BPF disassembly])
> +fi
> +AM_CONDITIONAL(HAVE_LINUX_BPF_H, [test "$have_bpf" = "true"])
Thanks, that does look like the correct configure check.
But it seems that BPF_PSEUDO_MAP_FD is the only missing constant
introduced since 3.18 (when linux/bpf.h was added) that we use.
So maybe we could simply define it ourselves when not found. So we can
build against any linux/bpf.h out there.
What do people think of the following?
diff --git a/libcpu/bpf_disasm.c b/libcpu/bpf_disasm.c
index 153dba9..e4bbae4 100644
--- a/libcpu/bpf_disasm.c
+++ b/libcpu/bpf_disasm.c
@@ -40,6 +40,10 @@
#include "../libelf/common.h"
#include "../libebl/libeblP.h"
+/* BPF_PSEUDO_MAP_FD was only introduced in linux 3.20. */
+#ifndef BPF_PSEUDO_MAP_FD
+ #define BPF_PSEUDO_MAP_FD 1
+#endif
static const char class_string[8][8] = {
[BPF_LD] = "ld",
Thanks,
Mark