linux-kernel - Re: [BUGFIX PATCH] selftests/x86: Check the availablity of sys/syscall.h

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-Id: <20191205172829.66ec55907ac31303bbede593@kernel.org>
Date:   Thu, 5 Dec 2019 17:28:29 +0900
From:   Masami Hiramatsu <mhiramat@...nel.org>
To:     Masami Hiramatsu <mhiramat@...nel.org>
Cc:     Shuah Khan <shuah@...nel.org>, linux-kselftest@...r.kernel.org,
        linux-kernel@...r.kernel.org, jaswinder.singh@...aro.org
Subject: Re: [BUGFIX PATCH] selftests/x86: Check the availablity of
 sys/syscall.h

Hi Shuah,

Could you pick this if there si no issue?

Thank you,

On Mon, 25 Nov 2019 20:03:44 +0900
Masami Hiramatsu <mhiramat@...nel.org> wrote:

> Since single_step_syscall.c depends on sys/syscall.h and
> its include, asm/unistd.h, we should check the availability
> of those headers.
> Without this fix, if gcc-multilib is not installed but
> libc6-dev-i386 is installed, kselftest tries to build 32bit
> binary and failed with following error message.
> 
> In file included from single_step_syscall.c:18:
> /usr/include/sys/syscall.h:24:10: fatal error: asm/unistd.h: No such file or directory
>  #include <asm/unistd.h>
>           ^~~~~~~~~~~~~~
> compilation terminated.
> 
> Signed-off-by: Masami Hiramatsu <mhiramat@...nel.org>
> ---
>  .../testing/selftests/x86/trivial_32bit_program.c  |    1 +
>  .../testing/selftests/x86/trivial_64bit_program.c  |    1 +
>  2 files changed, 2 insertions(+)
> 
> diff --git a/tools/testing/selftests/x86/trivial_32bit_program.c b/tools/testing/selftests/x86/trivial_32bit_program.c
> index aa1f58c2f71c..6b455eda24f7 100644
> --- a/tools/testing/selftests/x86/trivial_32bit_program.c
> +++ b/tools/testing/selftests/x86/trivial_32bit_program.c
> @@ -8,6 +8,7 @@
>  # error wrong architecture
>  #endif
>  
> +#include <sys/syscall.h>
>  #include <stdio.h>
>  
>  int main()
> diff --git a/tools/testing/selftests/x86/trivial_64bit_program.c b/tools/testing/selftests/x86/trivial_64bit_program.c
> index 39f4b84fbf15..07ae86df18ff 100644
> --- a/tools/testing/selftests/x86/trivial_64bit_program.c
> +++ b/tools/testing/selftests/x86/trivial_64bit_program.c
> @@ -8,6 +8,7 @@
>  # error wrong architecture
>  #endif
>  
> +#include <sys/syscall.h>
>  #include <stdio.h>
>  
>  int main()
> 


-- 
Masami Hiramatsu <mhiramat@...nel.org>