Эх сурвалжийг харах

rip out previous armeb old-stat abi compat code since arm linux opinion is that the old stat.h abi was for little endian only ... people should use this patch to fix their armeb kernels: http://www.arm.linux.org.uk/developer/patches/viewpatch.php?id=863/1

Mike Frysinger 20 жил өмнө
parent
commit
e8a4b87a04

+ 2 - 16
libc/sysdeps/linux/arm/bits/kernel_stat.h

@@ -39,14 +39,7 @@ struct kernel_stat {
 /* see the notes in common/xstatconv.c about why we have these
  * funky funk unions here ... i blame the schools */
 struct kernel_stat64 {
-#if defined(__ARMEB__)
-	union {
-	unsigned short     old_abi;
-	unsigned long long new_abi;
-	} st_dev;
-#else
 	unsigned long long st_dev;
-#endif
 	unsigned char      __pad0[4];
 
 #define _HAVE_STAT64___ST_INO
@@ -56,21 +49,14 @@ struct kernel_stat64 {
 	unsigned long      st_uid;
 	unsigned long      st_gid;
 
-#if defined(__ARMEB__)
-	union {
-	unsigned short     old_abi;
-	unsigned long long new_abi;
-	} st_rdev;
-#else
 	unsigned long long st_rdev;
-#endif
 	unsigned char      __pad3[4];
 
 	long long          st_size;
 	unsigned long      st_blksize;
 #if defined(__ARMEB__)
-	unsigned long      __pad_st_blocks; /* future possible st_blocks high bits */
-	unsigned long      st_blocks;       /* Number 512-byte blocks allocated. */
+	unsigned long      __pad4;     /* future possible st_blocks high bits */
+	unsigned long      st_blocks;  /* Number 512-byte blocks allocated. */
 #else
 	unsigned long      st_blocks;  /* Number 512-byte blocks allocated. */
 	unsigned long      __pad4;     /* future possible st_blocks high bits */

+ 9 - 73
libc/sysdeps/linux/common/xstatconv.c

@@ -49,102 +49,38 @@ void __xstat_conv(struct kernel_stat *kbuf, struct stat *buf)
     buf->st_mtime = kbuf->st_mtime;
     buf->st_ctime = kbuf->st_ctime;
 #ifdef STAT_HAVE_NSEC
-	buf->st_atime_nsec = kbuf->st_atime_nsec;
-	buf->st_mtime_nsec = kbuf->st_mtime_nsec;
-	buf->st_ctime_nsec = kbuf->st_ctime_nsec;
+	buf->st_atime_nsec = kbuf->st_atimensec;
+	buf->st_mtime_nsec = kbuf->st_mtimensec;
+	buf->st_ctime_nsec = kbuf->st_ctimensec;
 #endif
 }
 
 #if defined(__UCLIBC_HAS_LFS__)
 
-/* OK, this is ugly, but not much we can do about it.
- *
- * The issue at hand is that the kernel introduced an ABI change between 
- * 2.4 and 2.6 for big endian machines on some architectures in stat64 with 
- * st_dev and st_rdev fields.   Older kernels used 'short' but newer kernels 
- * use a 'long'.  As if that wasn't fun enough, the location of st_blocks and 
- * its padding have swapped.  So, when applicable, we need to detect and shift 
- * bits around and hope for the best.  The kernel at least helps us out a bit 
- * because it will be sure to zero out all padding fields.
- *
- * Unimportant Note About Mis-detections:
- * - if user is running a 2.4 system with the old ABI and they stat a file on 
- *   a system which major 0 / minor 0.  We can ignore this though because the 
- *   kernel has major 0 reserved for non mountable devices (wh00t!)
- * - if user is running a 2.6 system with the new ABI and they stat a file on 
- *   a device whose st_dev fills up the lower 6 bytes into the upper 2 bytes 
- *   which overlap.  Again we should be able to safely ignore this because 
- *   that means it'd be a pretty big ass (read: uncommon) major/minor combo :).
-
- Old [2.4]:
-struct stat64 {
-    unsigned short  st_dev;
-    unsigned char   __pad0[10];
-    ...
-    unsigned long   st_blocks;
-    unsigned long   __pad4;
-    ...
- New [2.6]:
-struct stat64 {
-    unsigned long long  st_dev;
-    unsigned char   __pad0[4];
-    ...
-    unsigned long   __pad4;
-    unsigned long   st_blocks;
-    ...
-
- * sizeof(unsigned short) = 2
- * sizeof(unsigned long long) = 8
- */
-#include <endian.h>
-#include <sys/utsname.h>
-#if BYTE_ORDER == BIG_ENDIAN && \
-    (defined(__ARMEB__)) /* || defined(__sh__) || defined(__sparc__))*/
-# define NEED_ABI_CHECK 1
-#else
-# define NEED_ABI_CHECK 0
-#endif
-
 void __xstat64_conv(struct kernel_stat64 *kbuf, struct stat64 *buf)
 {
     /* Convert to current kernel version of `struct stat64'.  */
-
-#if NEED_ABI_CHECK
-	if (!kbuf->st_dev.old_abi) {
-		/* new 2.6 ABI */
-		buf->st_dev = kbuf->st_dev.new_abi;
-		buf->st_rdev = kbuf->st_rdev.new_abi;
-		buf->st_blocks = kbuf->st_blocks;
-	} else {
-		/* old 2.4 ABI */
-		buf->st_dev = kbuf->st_dev.old_abi;
-		buf->st_rdev = kbuf->st_dev.old_abi;
-		buf->st_blocks = kbuf->__pad_st_blocks;
-	}
-#else
     buf->st_dev = kbuf->st_dev;
     buf->st_ino = kbuf->st_ino;
-    buf->st_rdev = kbuf->st_rdev;
-#endif
-
 #ifdef _HAVE_STAT64___ST_INO
     buf->__st_ino = kbuf->__st_ino;
 #endif
-
     buf->st_mode = kbuf->st_mode;
     buf->st_nlink = kbuf->st_nlink;
     buf->st_uid = kbuf->st_uid;
     buf->st_gid = kbuf->st_gid;
+    buf->st_rdev = kbuf->st_rdev;
     buf->st_size = kbuf->st_size;
     buf->st_blksize = kbuf->st_blksize;
+    buf->st_blocks = kbuf->st_blocks;
     buf->st_atime = kbuf->st_atime;
     buf->st_mtime = kbuf->st_mtime;
     buf->st_ctime = kbuf->st_ctime;
-
 #ifdef STAT_HAVE_NSEC
-	buf->st_atime_nsec = kbuf->st_atime_nsec;
-	buf->st_mtime_nsec = kbuf->st_mtime_nsec;
-	buf->st_ctime_nsec = kbuf->st_ctime_nsec;
+	buf->st_atime_nsec = kbuf->st_atimensec;
+	buf->st_mtime_nsec = kbuf->st_mtimensec;
+	buf->st_ctime_nsec = kbuf->st_ctimensec;
 #endif
 }
+
 #endif