[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] linux-user: fix problems with inotify syscalls
From: |
Riku Voipio |
Subject: |
Re: [Qemu-devel] [PATCH] linux-user: fix problems with inotify syscalls |
Date: |
Tue, 31 Mar 2009 00:15:23 +0300 |
User-agent: |
Mutt/1.5.18 (2008-05-17) |
On Sat, Mar 28, 2009 at 01:49:46PM -0700, Nathan Froyd wrote:
> The sys_inotify* calls are defined if the target supports them and the
> host supports the necessary syscalls. But the syscalls are handled if
> the target supports them. This situation leads to compilation failures
> when the host doesn't support the necessary syscalls, as the linker will
> complain about undefined functions.
Looks ok to me, this is how the *at syscalls are #ifdeffed as well.
Acked-By: Riku Voipio <address@hidden>
> Fix this state of affairs by making the handling conditions the same as
> the call definition conditions.
> Signed-off-by: Nathan Froyd <address@hidden>
> ---
> linux-user/syscall.c | 6 +++---
> 1 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/linux-user/syscall.c b/linux-user/syscall.c
> index 226ee6c..8608171 100644
> --- a/linux-user/syscall.c
> +++ b/linux-user/syscall.c
> @@ -6109,19 +6109,19 @@ abi_long do_syscall(void *cpu_env, int num, abi_long
> arg1,
> ret = do_futex(arg1, arg2, arg3, arg4, arg5, arg6);
> break;
> #endif
> -#ifdef TARGET_NR_inotify_init
> +#if defined(TARGET_NR_inotify_init) && defined(__NR_inotify_init)
> case TARGET_NR_inotify_init:
> ret = get_errno(sys_inotify_init());
> break;
> #endif
> -#ifdef TARGET_NR_inotify_add_watch
> +#if defined(TARGET_NR_inotify_add_watch) && defined(__NR_inotify_add_watch)
> case TARGET_NR_inotify_add_watch:
> p = lock_user_string(arg2);
> ret = get_errno(sys_inotify_add_watch(arg1, path(p), arg3));
> unlock_user(p, arg2, 0);
> break;
> #endif
> -#ifdef TARGET_NR_inotify_rm_watch
> +#if defined(TARGET_NR_inotify_rm_watch) && defined(__NR_inotify_rm_watch)
> case TARGET_NR_inotify_rm_watch:
> ret = get_errno(sys_inotify_rm_watch(arg1, arg2));
> break;
> --
> 1.6.0.5
>
>
--
"rm -rf" only sounds scary if you don't have backups