diff mbox

[5/7] selftests: disable seccomp for arm64

Message ID 1439559818-21666-6-git-send-email-bamvor.zhangjian@linaro.org
State New
Headers show

Commit Message

Bamvor Zhang Jian Aug. 14, 2015, 1:43 p.m. UTC
Currently, seccomp need the __NR_poll which is not supported
by arm64(There is only __NR_ppoll). I am not sure we should
skip this test testcase or update the seccomp without __NR_poll.

Signed-off-by: Bamvor Jian Zhang <bamvor.zhangjian@linaro.org>
---
 tools/testing/selftests/seccomp/Makefile | 6 ++++++
 1 file changed, 6 insertions(+)

Comments

Bamvor Zhang Jian Aug. 25, 2015, 12:51 p.m. UTC | #1
Hi, Michael

On 08/24/2015 11:48 AM, Michael Ellerman wrote:
> On Fri, 2015-08-14 at 21:43 +0800, Bamvor Jian Zhang wrote:
>> Currently, seccomp need the __NR_poll which is not supported
>> by arm64(There is only __NR_ppoll). I am not sure we should
>> skip this test testcase or update the seccomp without __NR_poll.
> 
> You should fix or skip the test that needs __NR_poll, not skip the entire test
> suite.
> 
> You should also CC the author of the tests, they might be able to help you.
Thanks you suggestion. I am trying to do this.

regards

bamvor
> 
> cheers
> 
> 
> 
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/
diff mbox

Patch

diff --git a/tools/testing/selftests/seccomp/Makefile b/tools/testing/selftests/seccomp/Makefile
index 8401e87..ea2793a 100644
--- a/tools/testing/selftests/seccomp/Makefile
+++ b/tools/testing/selftests/seccomp/Makefile
@@ -1,4 +1,10 @@ 
+
+uname_M := $(shell uname -m 2>/dev/null || echo not)
+ARCH ?= $(shell echo $(uname_M) | sed -e s/aarch64.*/arm64/)
+
+ifneq ($(ARCH),arm64)
 TEST_PROGS := seccomp_bpf
+endif
 CFLAGS += -Wl,-no-as-needed -Wall
 LDFLAGS += -lpthread