Re: [PATCH v3 3/9] build: Check that mlockall() exists

[Thomas Huth]

On 03/07/2020 16.56, Peter Maydell wrote:
> From: David CARLIER <devnexen@gmail.com>
> 
> Instead of assuming that all POSIX platforms provide mlockall(),
> test for it in configure. If the host doesn't provide this platform
> then os_mlock() will fail -ENOSYS, as it does already on Windows.
> 
> This is necessary for Haiku, which does not have mlockall().
> 
> Signed-off-by: David Carlier <devnexen@gmail.com>
> [PMM: Expanded commit message]
> Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
> Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
> ---
>  configure  | 15 +++++++++++++++
>  os-posix.c |  4 ++++
>  2 files changed, 19 insertions(+)
[...]
> diff --git a/os-posix.c b/os-posix.c
> index 3cd52e1e700..e02b566940c 100644
> --- a/os-posix.c
> +++ b/os-posix.c
> @@ -337,6 +337,7 @@ bool is_daemonized(void)
>  
>  int os_mlock(void)
>  {
> +#if defined CONFIG_MLOCKALL

Also missing the parentheses?

 Thomas