perf tools: Fix PowerPC native building
authorWang Nan <wangnan0@huawei.com>
Mon, 11 Jan 2016 13:47:57 +0000 (13:47 +0000)
committerArnaldo Carvalho de Melo <acme@redhat.com>
Mon, 11 Jan 2016 22:22:20 +0000 (19:22 -0300)
Checks BPF syscall number, turn off libbpf building on platform doesn't
correctly support sys_bpf instead of blocking compiling.

Reported-and-Tested-by: Naveen N. Rao <naveen.n.rao@linux.vnet.ibm.com>
Signed-off-by: Wang Nan <wangnan0@huawei.com>
Acked-by: Jiri Olsa <jolsa@kernel.org>
Cc: Sukadev Bhattiprolu <sukadev@linux.vnet.ibm.com>
Cc: Zefan Li <lizefan@huawei.com>
Cc: pi3orama@163.com
Link: http://lkml.kernel.org/r/1452520124-2073-7-git-send-email-wangnan0@huawei.com
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
tools/build/feature/test-bpf.c
tools/lib/bpf/bpf.c

index 062bac811af962433fb75786cdffe708ad8c6bd6..b389026839b97c0e35738f52564e87036fb79735 100644 (file)
@@ -1,9 +1,23 @@
+#include <asm/unistd.h>
 #include <linux/bpf.h>
+#include <unistd.h>
+
+#ifndef __NR_bpf
+# if defined(__i386__)
+#  define __NR_bpf 357
+# elif defined(__x86_64__)
+#  define __NR_bpf 321
+# elif defined(__aarch64__)
+#  define __NR_bpf 280
+#  error __NR_bpf not defined. libbpf does not support your arch.
+# endif
+#endif
 
 int main(void)
 {
        union bpf_attr attr;
 
+       /* Check fields in attr */
        attr.prog_type = BPF_PROG_TYPE_KPROBE;
        attr.insn_cnt = 0;
        attr.insns = 0;
@@ -14,5 +28,9 @@ int main(void)
        attr.kern_version = 0;
 
        attr = attr;
-       return 0;
+       /*
+        * Test existence of __NR_bpf and BPF_PROG_LOAD.
+        * This call should fail if we run the testcase.
+        */
+       return syscall(__NR_bpf, BPF_PROG_LOAD, attr, sizeof(attr));
 }
index 5bdc6eab6852bd7159584629d2a9798fa03b2df2..1f91cc941b7c2b14243f6ffe4118a21a2ac39c83 100644 (file)
@@ -14,8 +14,8 @@
 #include "bpf.h"
 
 /*
- * When building perf, unistd.h is override. Define __NR_bpf is
- * required to be defined.
+ * When building perf, unistd.h is overrided. __NR_bpf is
+ * required to be defined explicitly.
  */
 #ifndef __NR_bpf
 # if defined(__i386__)