Re: [Qemu-devel] [PATCH] linux-user: don't warn on missing capget/capset

[Andreas Färber]

Am 17.03.2014 12:55, schrieb address@hidden:
> From: Riku Voipio <address@hidden>
> 
> some people get numerous unimplemented capget/capset warnings. Since qemu
> linux-user is not secure to begin with, just skip the system call
> warning for now. Proper capset/capget to be added in Qemu 2.1

"QEMU". Drop "qemu" above?

> 
> Signed-off-by: Riku Voipio <address@hidden>

Since Peter specifically mentioned to you that this is what the SUSE
tree does, it would be only fair to attribute this change to Alex as
patch author in some way, be it Signed-off-by, From or Suggested-by:

https://github.com/openSUSE/qemu/commit/b0817614daa8594bcc10dff88b384027d00deb1f

Note that we only seemed to skip the warning for capget.

Regards,
Andreas

> ---
>  linux-user/syscall.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/linux-user/syscall.c b/linux-user/syscall.c
> index e404a32..4bfa3db 100644
> --- a/linux-user/syscall.c
> +++ b/linux-user/syscall.c
> @@ -7677,9 +7677,9 @@ abi_long do_syscall(void *cpu_env, int num, abi_long 
> arg1,
>          unlock_user(p, arg1, ret);
>          break;
>      case TARGET_NR_capget:
> -        goto unimplemented;
> +        goto unimplemented_nowarn;
>      case TARGET_NR_capset:
> -        goto unimplemented;
> +        goto unimplemented_nowarn;
>      case TARGET_NR_sigaltstack:
>  #if defined(TARGET_I386) || defined(TARGET_ARM) || defined(TARGET_MIPS) || \
>      defined(TARGET_SPARC) || defined(TARGET_PPC) || defined(TARGET_ALPHA) || 
> \
> 


-- 
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg