From f5204a90ac6d591771de7aa04c50e90438409f68 Mon Sep 17 00:00:00 2001 From: "jln@chromium.org" Date: Tue, 25 Mar 2014 02:06:50 +0000 Subject: Make seccomp-bpf mandatory on ARM. Seccomp-bpf used to be optional on non Chrome OS ARM bots, since Ubuntu lacked kernel support. Now that Ubuntu has kernel support, make seccomp-bpf mandatory on ARM. BUG=243478 NOTRY=true Review URL: https://codereview.chromium.org/203493007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@259094 0039d316-1c4b-4281-b951-d872f2087c98 --- sandbox/linux/seccomp-bpf/bpf_tests.h | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) (limited to 'sandbox') diff --git a/sandbox/linux/seccomp-bpf/bpf_tests.h b/sandbox/linux/seccomp-bpf/bpf_tests.h index 7095c23..dd30b15 100644 --- a/sandbox/linux/seccomp-bpf/bpf_tests.h +++ b/sandbox/linux/seccomp-bpf/bpf_tests.h @@ -89,10 +89,9 @@ class BPFTests : public UnitTests { arg->test()(arg->aux_); } else { printf("This BPF test is not fully running in this configuration!\n"); - // Android, ARM and Valgrind are the three only configurations where we - // accept not having kernel BPF support. - // TODO(jln): remote ARM from this list when possible (crbug.com/243478). - if (!IsAndroid() && !IsRunningOnValgrind() && !IsArchitectureArm()) { + // Android and Valgrind are the only configurations where we accept not + // having kernel BPF support. + if (!IsAndroid() && !IsRunningOnValgrind()) { const bool seccomp_bpf_is_supported = false; BPF_ASSERT(seccomp_bpf_is_supported); } -- cgit v1.1