99 lines
3.2 KiB
Diff
99 lines
3.2 KiB
Diff
|
From patchwork Thu Jan 9 09:35:45 2014
|
||
|
Content-Type: text/plain; charset="utf-8"
|
||
|
MIME-Version: 1.0
|
||
|
Content-Transfer-Encoding: 7bit
|
||
|
Subject: libc: posix_fadvise: Fix build breakage for !LFS
|
||
|
From: Vineet Gupta <Vineet.Gupta1@synopsys.com>
|
||
|
X-Patchwork-Id: 308533
|
||
|
Message-Id: <1389260145-8810-1-git-send-email-vgupta@synopsys.com>
|
||
|
To: <uclibc@uclibc.org>
|
||
|
Cc: Francois.Bedard@synopsys.com, Vineet Gupta <Vineet.Gupta1@synopsys.com>,
|
||
|
markos Chandras <markos.chandras@gmail.com>
|
||
|
Date: Thu, 9 Jan 2014 15:05:45 +0530
|
||
|
|
||
|
commit 00571b43df2e "libc: posix_fadvise: restore implementation for xtensa"
|
||
|
enabled posix_fadvise() for all arches (it was just not generated
|
||
|
before).
|
||
|
|
||
|
However this also unearthed an issue introduced by ee84b8b400
|
||
|
"linux: posix_fadvise: use new SYSCALL_ALIGN_64BIT" which is to
|
||
|
referencing LFS'ish code (off64_t) w/o proper checks which causes build
|
||
|
to break for !LFS.
|
||
|
|
||
|
Fix this by calling posix_fadvise64() only for LFS case and open-code
|
||
|
it's equivalent for !LFS.
|
||
|
|
||
|
Signed-off-by: Vineet Gupta <vgupta@synopsys.com>
|
||
|
Cc: Mike Frysinger <vapier@gentoo.org>
|
||
|
Cc: Baruch Siach <baruch@tkos.co.il>
|
||
|
Cc: Khem Raj <raj.khem@gmail.com>
|
||
|
Cc: markos Chandras <markos.chandras@gmail.com>
|
||
|
Cc: Bernhard Reutner-Fischer <rep.dot.nop@gmail.com>
|
||
|
|
||
|
---
|
||
|
|
||
|
Patch status: sent upstream (http://patchwork.ozlabs.org/patch/308533/)
|
||
|
|
||
|
libc/sysdeps/linux/common/posix_fadvise.c | 30 ++++++++++++++++++++++++------
|
||
|
1 file changed, 24 insertions(+), 6 deletions(-)
|
||
|
|
||
|
diff --git a/libc/sysdeps/linux/common/posix_fadvise.c b/libc/sysdeps/linux/common/posix_fadvise.c
|
||
|
index 25c294178e5e..14bbeeea13bc 100644
|
||
|
--- a/libc/sysdeps/linux/common/posix_fadvise.c
|
||
|
+++ b/libc/sysdeps/linux/common/posix_fadvise.c
|
||
|
@@ -22,17 +22,34 @@
|
||
|
# include <endian.h>
|
||
|
# include <bits/wordsize.h>
|
||
|
|
||
|
-# ifdef __NR_fadvise64_64
|
||
|
-int posix_fadvise64(int fd, off64_t offset, off64_t len, int advice);
|
||
|
-# endif
|
||
|
+# if defined(__NR_fadvise64_64) && defined(__UCLIBC_HAS_LFS__)
|
||
|
+#include <_lfs_64.h>
|
||
|
|
||
|
+int posix_fadvise64(int fd, off64_t offset, off64_t len, int advice);
|
||
|
int posix_fadvise(int fd, off_t offset, off_t len, int advice)
|
||
|
{
|
||
|
-# ifdef __NR_fadvise64_64
|
||
|
return posix_fadvise64(fd, offset, len, advice);
|
||
|
-# else
|
||
|
+}
|
||
|
+#else
|
||
|
+
|
||
|
+int posix_fadvise(int fd, off_t offset, off_t len, int advice)
|
||
|
+{
|
||
|
int ret;
|
||
|
INTERNAL_SYSCALL_DECL(err);
|
||
|
+
|
||
|
+# ifdef __NR_fadvise64_64
|
||
|
+# if __WORDSIZE == 64
|
||
|
+ ret = INTERNAL_SYSCALL(fadvise64_64, err, 4, fd, offset, len, advice);
|
||
|
+# else
|
||
|
+# if defined(__UCLIBC_SYSCALL_ALIGN_64BIT__) || defined(__arm__)
|
||
|
+ ret = INTERNAL_SYSCALL(fadvise64_64, err, 6, fd, advice,
|
||
|
+ OFF_HI_LO (offset), OFF_HI_LO (len));
|
||
|
+# else
|
||
|
+ ret = INTERNAL_SYSCALL(fadvise64_64, err, 6, fd,
|
||
|
+ OFF_HI_LO (offset), OFF_HI_LO (len), advice);
|
||
|
+# endif
|
||
|
+# endif
|
||
|
+# else /* __NR_fadvise64 */
|
||
|
# if __WORDSIZE == 64
|
||
|
ret = INTERNAL_SYSCALL(fadvise64, err, 4, fd, offset, len, advice);
|
||
|
# else
|
||
|
@@ -43,12 +60,13 @@ int posix_fadvise(int fd, off_t offset, off_t len, int advice)
|
||
|
# endif
|
||
|
OFF_HI_LO (offset), len, advice);
|
||
|
# endif
|
||
|
+# endif
|
||
|
if (INTERNAL_SYSCALL_ERROR_P (ret, err))
|
||
|
return INTERNAL_SYSCALL_ERRNO (ret, err);
|
||
|
return 0;
|
||
|
-# endif
|
||
|
}
|
||
|
# if defined __UCLIBC_HAS_LFS__ && (!defined __NR_fadvise64_64 || __WORDSIZE == 64)
|
||
|
strong_alias(posix_fadvise,posix_fadvise64)
|
||
|
# endif
|
||
|
#endif
|
||
|
+#endif
|