lkml.org 
[lkml]   [2022]   [Feb]   [22]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH AUTOSEL 5.15 19/28] selftests/seccomp: Fix seccomp failure by adding missing headers
    Date
    From: Sherry Yang <sherry.yang@oracle.com>

    [ Upstream commit 21bffcb76ee2fbafc7d5946cef10abc9df5cfff7 ]

    seccomp_bpf failed on tests 47 global.user_notification_filter_empty
    and 48 global.user_notification_filter_empty_threaded when it's
    tested on updated kernel but with old kernel headers. Because old
    kernel headers don't have definition of macro __NR_clone3 which is
    required for these two tests. Since under selftests/, we can install
    headers once for all tests (the default INSTALL_HDR_PATH is
    usr/include), fix it by adding usr/include to the list of directories
    to be searched. Use "-isystem" to indicate it's a system directory as
    the real kernel headers directories are.

    Signed-off-by: Sherry Yang <sherry.yang@oracle.com>
    Tested-by: Sherry Yang <sherry.yang@oracle.com>
    Reviewed-by: Kees Cook <keescook@chromium.org>
    Signed-off-by: Shuah Khan <skhan@linuxfoundation.org>
    Signed-off-by: Sasha Levin <sashal@kernel.org>
    ---
    tools/testing/selftests/seccomp/Makefile | 2 +-
    1 file changed, 1 insertion(+), 1 deletion(-)

    diff --git a/tools/testing/selftests/seccomp/Makefile b/tools/testing/selftests/seccomp/Makefile
    index 0ebfe8b0e147f..585f7a0c10cbe 100644
    --- a/tools/testing/selftests/seccomp/Makefile
    +++ b/tools/testing/selftests/seccomp/Makefile
    @@ -1,5 +1,5 @@
    # SPDX-License-Identifier: GPL-2.0
    -CFLAGS += -Wl,-no-as-needed -Wall
    +CFLAGS += -Wl,-no-as-needed -Wall -isystem ../../../../usr/include/
    LDFLAGS += -lpthread

    TEST_GEN_PROGS := seccomp_bpf seccomp_benchmark
    --
    2.34.1
    \
     
     \ /
      Last update: 2022-02-23 03:32    [W:7.341 / U:0.152 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site