On Thursday 02 January 2014 05:16:14 Baruch Siach wrote:
> Commit ee84b8b400 (linux: posix_fadvise: use new SYSCALL_ALIGN_64BIT)
> removed posix_fadvise implementation for xtensa, since xtensa does not
> define __NR_fadvise64. Redefine __NR_fadvise64 as __NR_fadvise64_64 to
> restore posix_fadvise.
> 
> --- a/libc/sysdeps/linux/common/posix_fadvise.c
> +++ b/libc/sysdeps/linux/common/posix_fadvise.c
> @@ -10,6 +10,10 @@
> 
>  #include <sys/syscall.h>
> 
> +#if !defined(__NR_fadvise64) && defined(__NR_fadvise64_64)
> +#define __NR_fadvise64       __NR_fadvise64_64
> +#endif
> +
>  #ifdef __NR_fadvise64
>  # include <fcntl.h>
>  # include <endian.h>

NAK: fairly certain this doesn't actually work.  the 64_64 version expects a 
64bit argument (for 32bit arches like xtensa, that means 2 32bits), but this 
file only passes up on 32bit.
-mike

Attachment: signature.asc
Description: This is a digitally signed message part.

_______________________________________________
uClibc mailing list
[email protected]
http://lists.busybox.net/mailman/listinfo/uclibc

Reply via email to