]> git.ipfire.org Git - thirdparty/glibc.git/commitdiff
Remove __ASSUME_FCNTL64.
authorJoseph Myers <joseph@codesourcery.com>
Wed, 1 Aug 2012 21:27:58 +0000 (21:27 +0000)
committerJoseph Myers <joseph@codesourcery.com>
Wed, 1 Aug 2012 21:27:58 +0000 (21:27 +0000)
14 files changed:
ChangeLog
ports/ChangeLog.arm
ports/ChangeLog.hppa
ports/ChangeLog.m68k
ports/ChangeLog.mips
ports/ChangeLog.tile
ports/sysdeps/unix/sysv/linux/arm/kernel-features.h
ports/sysdeps/unix/sysv/linux/hppa/kernel-features.h
ports/sysdeps/unix/sysv/linux/m68k/kernel-features.h
ports/sysdeps/unix/sysv/linux/mips/kernel-features.h
ports/sysdeps/unix/sysv/linux/tile/kernel-features.h
sysdeps/unix/sysv/linux/i386/fcntl.c
sysdeps/unix/sysv/linux/i386/lockf64.c
sysdeps/unix/sysv/linux/kernel-features.h

index 6d7aefe43d3f4097026874afa6ceac5824ccf534..13c35303cd8270422914437d12fb4f0475820d5d 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,22 @@
 2012-08-01  Joseph Myers  <joseph@codesourcery.com>
 
+       * sysdeps/unix/sysv/linux/kernel-features.h (__ASSUME_FCNTL64):
+       Remove all definitions.
+       * sysdeps/unix/sysv/linux/i386/fcntl.c: Do not include
+       <kernel-features.h>.
+       [!__ASSUME_FCNTL64] (__have_no_fcntl64): Remove.
+       (miss_F_GETOWN_EX): Remove all definitions.
+       [NO_CANCELLATION && !__ASSUME_FCNTL64] (__fcntl_nocancel): Remove
+       macro definition.
+       [!__ASSUME_FCNTL64]: Remove conditional code.
+       [__ASSUME_FCNTL64]: Make code unconditional.
+       * sysdeps/unix/sysv/linux/i386/lockf64.c: Do not include
+       <kernel-features.h>.
+       [__NR_fcntl64 && !__ASSUME_FCNTL64] (__have_no_fcntl64): Remove.
+       (lockf64) [!__ASSUME_FCNTL64]: Remove conditional code.
+       (lockf64) [__NR_fcntl64]: Make code unconditional.
+       (lockf64) [__ASSUME_FCNTL64]: Likewise.
+
        * sysdeps/unix/sysv/linux/kernel-features.h
        (__ASSUME_VFORK_SYSCALL): Remove all definitions.
        * sysdeps/unix/sysv/linux/i386/vfork.S (__vfork) [__NR_vfork]:
index d7d9f4ce510cd3a18d4620b931f8478ab30ac6e9..58ad60d315307ff93da01365e7b991bde5db4916 100644 (file)
@@ -1,5 +1,8 @@
 2012-08-01  Joseph Myers  <joseph@codesourcery.com>
 
+       * sysdeps/unix/sysv/linux/arm/kernel-features.h
+       (__ASSUME_FCNTL64): Remove.
+
        * sysdeps/unix/sysv/linux/arm/kernel-features.h
        (__ASSUME_VFORK_SYSCALL): Remove.
 
index 83d298fe4a8df02cd658a7233fe8af997e35018d..bf54564dfb3d4d5185157dc09ba54e85f2f05d42 100644 (file)
@@ -1,3 +1,8 @@
+2012-08-01  Joseph Myers  <joseph@codesourcery.com>
+
+       * sysdeps/unix/sysv/linux/hppa/kernel-features.h
+       (__ASSUME_FCNTL64): Remove.
+
 2012-07-26  Andreas Jaeger  <aj@suse.de>
 
        * sysdeps/unix/sysv/linux/hppa/sys/epoll.h (EPOLLWAKEUP): Add new
index 993ba7fd283f2c3dff07d1592b28cddc87eed1e6..2e180a7f4b4af4093f23afc4b181355cdc4297bc 100644 (file)
@@ -1,5 +1,8 @@
 2012-08-01  Joseph Myers  <joseph@codesourcery.com>
 
+       * sysdeps/unix/sysv/linux/m68k/kernel-features.h
+       (__ASSUME_FCNTL64): Remove.
+
        * sysdeps/unix/sysv/linux/m68k/kernel-features.h
        (__ASSUME_VFORK_SYSCALL): Remove.
        * sysdeps/unix/sysv/linux/m68k/vfork.S (__vfork) [__NR_vfork]:
index d4978abd965afaddc7be7bee3116c0ba6f152ffc..8421a0df24c95252098e6bb493ee2af855f65353 100644 (file)
@@ -1,3 +1,8 @@
+2012-08-01  Joseph Myers  <joseph@codesourcery.com>
+
+       * sysdeps/unix/sysv/linux/mips/kernel-features.h
+       (__ASSUME_FCNTL64): Remove.
+
 2012-08-01  Steve Ellcey  <sellcey@mips.com>
 
        * sysdeps/mips/sys/asm.h [_MIPS_ISA != _MIPS_ISA_MIPS4 &&
index 9ef3f99ec342faed9bb9658f713d728c9747f749..471cf1207f9184eb57c7f5c2044e34ecb36a1090 100644 (file)
@@ -1,3 +1,8 @@
+2012-08-01  Joseph Myers  <joseph@codesourcery.com>
+
+       * sysdeps/unix/sysv/linux/tile/kernel-features.h
+       (__ASSUME_FCNTL64): Remove.
+
 2012-07-26  Andreas Jaeger  <aj@suse.de>
 
        * sysdeps/unix/sysv/linux/tile/sys/ptrace.h (__ptrace_eventcodes):
index 0ab5df094e2d89787f6e7d9e00f56a9898c4f383..745bca8bf65f3f8053371c3828a7195d1e80e924 100644 (file)
@@ -27,9 +27,6 @@
 #define __ASSUME_MMAP2_SYSCALL         1
 #define __ASSUME_STAT64_SYSCALL        1
 
-/* Arm got fcntl64 in 2.4.4.  */
-#define __ASSUME_FCNTL64               1
-
 /* The signal frame layout changed in 2.6.18.  */
 #if __LINUX_KERNEL_VERSION >= 132626
 # define __ASSUME_SIGFRAME_V2  1
index 8dcdfbfabfe621fecc2a02c735fc8d6b6224c1ce..13a3c010feacb8694d4145f873a88ef92046e8cb 100644 (file)
@@ -27,7 +27,6 @@
 #define __ASSUME_STAT64_SYSCALL                1
 #define __ASSUME_IPC64                 1
 #define __ASSUME_ST_INO_64_BIT         1
-#define __ASSUME_FCNTL64               1
 #define __ASSUME_GETDENTS64_SYSCALL    1
 
 /* PA-RISC 2.6.9 kernels had the first LWS CAS support */
index 05d06b2555fe29b8751887ffe33424907e67391b..ed62453ec3a23fd132eb7a1d4a53d9e6e0e5f8d4 100644 (file)
@@ -20,7 +20,6 @@
 /* These features were surely available with 2.4.12.  */
 #define __ASSUME_MMAP2_SYSCALL         1
 #define __ASSUME_STAT64_SYSCALL        1
-#define __ASSUME_FCNTL64               1
 
 /* Many syscalls were added in 2.6.10 for m68k.  */
 #if __LINUX_KERNEL_VERSION >= 132618
index 70f1fc1ed400e2a426cda80c50802c4ca87e65bd..996ba0aedf79cb9e14f5263687c4c824820173c0 100644 (file)
 /* MIPS platforms had IPC64 all along.  */
 #define __ASSUME_IPC64         1
 
-#if _MIPS_SIM == _ABIN32
-# define __ASSUME_FCNTL64              1
-#endif
-
 /* Support for the eventfd2 and signalfd4 syscalls was added in 2.6.27.  */
 #if __LINUX_KERNEL_VERSION >= 0x02061c
 # define __ASSUME_EVENTFD2     1
index 89929d2bc9393f130ba26326e974ae77251ce59c..766b4ad6cac2a2a35c7b834fd72ece76a7879bb2 100644 (file)
@@ -1,4 +1,4 @@
-/* Copyright (C) 2011 Free Software Foundation, Inc.
+/* Copyright (C) 2011-2012 Free Software Foundation, Inc.
    This file is part of the GNU C Library.
    Contributed by Chris Metcalf <cmetcalf@tilera.com>, 2011.
 
@@ -20,7 +20,6 @@
 /* TILE glibc support starts with 2.6.36, guaranteeing many kernel features. */
 #define __ASSUME_MMAP2_SYSCALL         1
 #define __ASSUME_STAT64_SYSCALL                1
-#define __ASSUME_FCNTL64               1
 #define __ASSUME_CLONE_THREAD_FLAGS    1
 #define __ASSUME_TGKILL                        1
 #define __ASSUME_UTIMES                        1
index c68a06b45ba3a41b5140e56a2f6ec12cd79797ac..a0f25449e06bdb56dd90c35b3483150fc25c6f68 100644 (file)
@@ -1,4 +1,4 @@
-/* Copyright (C) 2000,2002-2004,2006,2009,2010 Free Software Foundation, Inc.
+/* Copyright (C) 2000-2012 Free Software Foundation, Inc.
    This file is part of the GNU C Library.
 
    The GNU C Library is free software; you can redistribute it and/or
 #include <stdarg.h>
 
 #include <sys/syscall.h>
-#include <kernel-features.h>
 
-#ifndef __ASSUME_FCNTL64
-/* This variable is shared with all files that check for fcntl64.  */
-int __have_no_fcntl64;
-#endif
-
-#ifdef __ASSUME_F_GETOWN_EX
-# define miss_F_GETOWN_EX 0
-#elif !defined __ASSUME_FCNTL64
-static int miss_F_GETOWN_EX;
-#endif
-
-
-#if defined NO_CANCELLATION && !defined __ASSUME_FCNTL64
-# define __fcntl_nocancel  __libc_fcntl
-#endif
-
-#if !defined NO_CANCELLATION || !defined __ASSUME_FCNTL64
+#ifndef NO_CANCELLATION
 int
 __fcntl_nocancel (int fd, int cmd, ...)
 {
@@ -51,112 +34,11 @@ __fcntl_nocancel (int fd, int cmd, ...)
   arg = va_arg (ap, void *);
   va_end (ap);
 
-#ifndef __ASSUME_FCNTL64
-# ifdef __NR_fcntl64
-  if (! __have_no_fcntl64)
-    {
-      int result = INLINE_SYSCALL (fcntl64, 3, fd, cmd, arg);
-      if (result >= 0 || errno != ENOSYS)
-       return result;
-
-      __have_no_fcntl64 = 1;
-    }
-# endif
-  switch (cmd)
-    {
-    case F_GETLK64:
-      /* Convert arg from flock64 to flock and back.  */
-      {
-       struct flock fl;
-       struct flock64 *fl64 = arg;
-       int res;
-
-       fl.l_start = (off_t)fl64->l_start;
-       /* Check if we can represent the values with the smaller type.  */
-       if ((off64_t) fl.l_start != fl64->l_start)
-         {
-         eoverflow:
-           __set_errno (EOVERFLOW);
-           return -1;
-         }
-       fl.l_len = (off_t) fl64->l_len;
-       /* Check if we can represent the values with the smaller type.  */
-       if ((off64_t) fl.l_len != fl64->l_len)
-         goto eoverflow;
-
-       fl.l_type = fl64->l_type;
-       fl.l_whence = fl64->l_whence;
-       fl.l_pid = fl64->l_pid;
-
-       res = INLINE_SYSCALL (fcntl, 3, fd, F_GETLK, &fl);
-       if (res  != 0)
-         return res;
-       /* Everything ok, convert back.  */
-       fl64->l_type = fl.l_type;
-       fl64->l_whence = fl.l_whence;
-       fl64->l_start = fl.l_start;
-       fl64->l_len = fl.l_len;
-       fl64->l_pid = fl.l_pid;
-
-       return 0;
-      }
-    case F_SETLK64:
-    case F_SETLKW64:
-      /* Try to convert arg from flock64 to flock.  */
-      {
-       struct flock fl;
-       struct flock64 *fl64 = arg;
-
-       fl.l_start = (off_t) fl64->l_start;
-       /* Check if we can represent the values with the smaller type.  */
-       if ((off64_t) fl.l_start != fl64->l_start)
-         goto eoverflow;
-
-       fl.l_len = (off_t)fl64->l_len;
-       /* Check if we can represent the values with the smaller type.  */
-       if ((off64_t) fl.l_len != fl64->l_len)
-         {
-           __set_errno (EOVERFLOW);
-           return -1;
-         }
-       fl.l_type = fl64->l_type;
-       fl.l_whence = fl64->l_whence;
-       fl.l_pid = fl64->l_pid;
-       assert (F_SETLK - F_SETLKW == F_SETLK64 - F_SETLKW64);
-       return INLINE_SYSCALL (fcntl, 3, fd, cmd + F_SETLK - F_SETLK64, &fl);
-      }
-    case F_GETOWN:
-      if (! miss_F_GETOWN_EX)
-       {
-         INTERNAL_SYSCALL_DECL (err);
-         struct f_owner_ex fex;
-         int res = INTERNAL_SYSCALL (fcntl, err, 3, fd, F_GETOWN_EX, &fex);
-         if (!INTERNAL_SYSCALL_ERROR_P (res, err))
-           return fex.type == F_OWNER_GID ? -fex.pid : fex.pid;
-
-# ifndef __ASSUME_F_GETOWN_EX
-         if (INTERNAL_SYSCALL_ERRNO (res, err) == EINVAL)
-           miss_F_GETOWN_EX = 1;
-         else
-# endif
-           {
-             __set_errno (INTERNAL_SYSCALL_ERRNO (res, err));
-             return -1;
-           }
-       }
-      /* FALLTHROUGH */
-    default:
-      return INLINE_SYSCALL (fcntl, 3, fd, cmd, arg);
-    }
-  return -1;
-#else
   return INLINE_SYSCALL (fcntl64, 3, fd, cmd, arg);
-#endif  /* !__ASSUME_FCNTL64  */
 }
-#endif /* NO_CANCELLATION || !__ASSUME_FCNTL64 */
+#endif /* NO_CANCELLATION */
 
 
-#ifndef __fcntl_nocancel
 int
 __libc_fcntl (int fd, int cmd, ...)
 {
@@ -167,27 +49,17 @@ __libc_fcntl (int fd, int cmd, ...)
   arg = va_arg (ap, void *);
   va_end (ap);
 
-# ifdef __ASSUME_FCNTL64
   if (SINGLE_THREAD_P || (cmd != F_SETLKW && cmd != F_SETLKW64))
     return INLINE_SYSCALL (fcntl64, 3, fd, cmd, arg);
 
   int oldtype = LIBC_CANCEL_ASYNC ();
 
   int result = INLINE_SYSCALL (fcntl64, 3, fd, cmd, arg);
-# else
-  if (SINGLE_THREAD_P || (cmd != F_SETLKW && cmd != F_SETLKW64))
-    return __fcntl_nocancel (fd, cmd, arg);
-
-  int oldtype = LIBC_CANCEL_ASYNC ();
-
-  int result = __fcntl_nocancel (fd, cmd, arg);
-# endif
 
   LIBC_CANCEL_RESET (oldtype);
 
   return result;
 }
-#endif
 libc_hidden_def (__libc_fcntl)
 
 weak_alias (__libc_fcntl, __fcntl)
index 6e7603e529cd061fc1cc437935a78131b70b0a97..1caf9778e80e5afed78c4dfa98238bf20c3544e7 100644 (file)
@@ -1,5 +1,4 @@
-/* Copyright (C) 1994,1996,1997,1998,1999,2000,2003,2006
-       Free Software Foundation, Inc.
+/* Copyright (C) 1994-2012 Free Software Foundation, Inc.
    This file is part of the GNU C Library.
 
    The GNU C Library is free software; you can redistribute it and/or
 #include <string.h>
 #include <sysdep.h>
 
-#include <kernel-features.h>
-
 /* lockf is a simplified interface to fcntl's locking facilities.  */
 
-#ifdef __NR_fcntl64
-# if __ASSUME_FCNTL64 == 0
-/* This variable is shared with all files that check for fcntl64. The
-   declaration is in fcntl.c.  */
-extern int __have_no_fcntl64;
-# endif
-#endif
-
 int
 lockf64 (int fd, int cmd, off64_t len64)
 {
-#if __ASSUME_FCNTL64 == 0
-  struct flock fl;
-  off_t len = (off_t) len64;
-#endif
-#ifdef __NR_fcntl64
   struct flock64 fl64;
   int cmd64;
-#endif
-
-#if __ASSUME_FCNTL64 == 0
-  memset ((char *) &fl, '\0', sizeof (fl));
 
-  /* lockf is always relative to the current file position.  */
-  fl.l_whence = SEEK_CUR;
-  fl.l_start = 0;
-  fl.l_len = len;
-#endif
-#ifdef __NR_fcntl64
-# if __ASSUME_FCNTL64 == 0
-  if (!__have_no_fcntl64)
-    {
-# endif
-      memset ((char *) &fl64, '\0', sizeof (fl64));
-      fl64.l_whence = SEEK_CUR;
-      fl64.l_start = 0;
-      fl64.l_len = len64;
-# if __ASSUME_FCNTL64 == 0
-    }
-# endif
-#endif
+  memset ((char *) &fl64, '\0', sizeof (fl64));
+  fl64.l_whence = SEEK_CUR;
+  fl64.l_start = 0;
+  fl64.l_len = len64;
 
-#if __ASSUME_FCNTL64 == 0 && !defined __NR_fcntl64
-  if (len64 != (off64_t) len)
-    {
-      /* We can't represent the length.  */
-      __set_errno (EOVERFLOW);
-      return -1;
-    }
-#endif
   switch (cmd)
     {
     case F_TEST:
       /* Test the lock: return 0 if FD is unlocked or locked by this process;
         return -1, set errno to EACCES, if another process holds the lock.  */
-#if __ASSUME_FCNTL64 > 0
       fl64.l_type = F_RDLCK;
       if (INLINE_SYSCALL (fcntl64, 3, fd, F_GETLK64, &fl64) < 0)
         return -1;
@@ -90,104 +47,22 @@ lockf64 (int fd, int cmd, off64_t len64)
         return 0;
       __set_errno (EACCES);
       return -1;
-#else
-# ifdef __NR_fcntl64
-      if (!__have_no_fcntl64)
-       {
-         int res;
-
-         fl64.l_type = F_RDLCK;
-         res = INLINE_SYSCALL (fcntl64, 3, fd, F_GETLK64, &fl64);
-         /* If errno == ENOSYS try the 32bit interface if len64 can
-             be represented with 32 bits.  */
-
-         if (res == 0)
-           {
-             if (fl64.l_type == F_UNLCK || fl64.l_pid == __getpid ())
-               return 0;
-             __set_errno (EACCES);
-             return -1;
-           }
-         else if (errno == ENOSYS)
-           __have_no_fcntl64 = 1;
-         else
-           /* res < 0 && errno != ENOSYS.  */
-           return -1;
-         if (len64 != (off64_t) len)
-           {
-             /* We can't represent the length.  */
-             __set_errno (EOVERFLOW);
-             return -1;
-           }
-       }
-# endif
-      fl.l_type = F_RDLCK;
-      if (__fcntl (fd, F_GETLK, &fl) < 0)
-       return -1;
-      if (fl.l_type == F_UNLCK || fl.l_pid == __getpid ())
-       return 0;
-      __set_errno (EACCES);
-      return -1;
-#endif
     case F_ULOCK:
-#if __ASSUME_FCNTL64 == 0
-      fl.l_type = F_UNLCK;
-      cmd = F_SETLK;
-#endif
-#ifdef __NR_fcntl64
       fl64.l_type = F_UNLCK;
       cmd64 = F_SETLK64;
-#endif
       break;
     case F_LOCK:
-#if __ASSUME_FCNTL64 == 0
-      fl.l_type = F_WRLCK;
-      cmd = F_SETLKW;
-#endif
-#ifdef __NR_fcntl64
       fl64.l_type = F_WRLCK;
       cmd64 = F_SETLKW64;
-#endif
       break;
     case F_TLOCK:
-#if __ASSUME_FCNTL64 == 0
-      fl.l_type = F_WRLCK;
-      cmd = F_SETLK;
-#endif
-#ifdef __NR_fcntl64
       fl64.l_type = F_WRLCK;
       cmd64 = F_SETLK64;
-#endif
       break;
 
     default:
       __set_errno (EINVAL);
       return -1;
     }
-#if __ASSUME_FCNTL64 > 0
   return INLINE_SYSCALL (fcntl64, 3, fd, cmd64, &fl64);
-#else
-# ifdef __NR_fcntl64
-
-  if (!__have_no_fcntl64)
-    {
-      int res = INLINE_SYSCALL (fcntl64, 3, fd, cmd64, &fl64);
-
-      /* If errno == ENOSYS try the 32bit interface if len64 can
-        be represented with 32 bits.  */
-      if (res == 0 || errno != ENOSYS)
-       return res;
-
-      __have_no_fcntl64 = 1;
-
-      if (len64 != (off64_t) len)
-       {
-         /* We can't represent the length.  */
-         __set_errno (EOVERFLOW);
-         return -1;
-       }
-    }
-# endif
-  return __fcntl (fd, cmd, &fl);
-#endif
 }
index 7357165618d598c30aecf5c7c161645b9b0b11c5..e97539ba85ec9cb537ce74b525f4e6d15487a913 100644 (file)
 # define __ASSUME_ST_INO_64_BIT                1
 #endif
 
-/* To support locking of large files a new fcntl() syscall was introduced
-   in 2.4.0-test7.  */
-#if defined __i386__ || defined __sparc__
-# define __ASSUME_FCNTL64              1
-#endif
-
-/* Arm got fcntl64 in 2.4.4, PowerPC and SH have it also in 2.4.4 (I
-   don't know when it got introduced).  But PowerPC64 does not support
-   separate FCNTL64 call, FCNTL is already 64-bit */
-#if (defined __powerpc__ || defined __sh__) \
-    && !defined __powerpc64__
-# define __ASSUME_FCNTL64              1
-#endif
-
 /* The getdents64 syscall was introduced in 2.4.0-test7 (but later for
    MIPS n32).  */
 #define __ASSUME_GETDENTS64_SYSCALL    1