[PATCH] 2TB files: st_blocks is invalid when calling stat64
authorTakashi Sato <sho@tnes.nec.co.jp>
Sun, 26 Mar 2006 09:37:51 +0000 (01:37 -0800)
committerLinus Torvalds <torvalds@g5.osdl.org>
Sun, 26 Mar 2006 16:57:00 +0000 (08:57 -0800)
This patch series fixes the following problems on 32 bits architecture.

o stat64 returns the lower 32 bits of blocks, although userland st_blocks
  has 64 bits, because i_blocks has only 32 bits.  The ioctl with FIOQSIZE has
  the same problem.

o As Dave Kleikamp said, making >2TB file on JFS results in writing an
  invalid block number to disk inode.  The cause is the same as above too.

o In generic quota code dquot_transfer(), the file usage is calculated from
  i_blocks via inode_get_bytes().  If the file is over 2TB, the change of
  usage is less than expected.  The cause is the same as above too.

o As Trond Myklebust said, statfs64's entries related to blocks are invalid
  on statfs64 for a network filesystem which has more than 2^32-1 blocks with
  CONFIG_LBD disabled.  [PATCH 3/3]

We made patches to fix problems that occur when handling a large filesystem
and a large file.  It was discussed on the mails titled "stat64 for over 2TB
file returned invalid st_blocks".

Signed-off-by: Takashi Sato <sho@tnes.nec.co.jp>
Cc: Dave Kleikamp <shaggy@austin.ibm.com>
Cc: Jan Kara <jack@ucw.cz>
Cc: Trond Myklebust <trond.myklebust@fys.uio.no>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
include/asm-i386/stat.h
include/asm-m68k/stat.h
include/asm-sh/stat.h
include/linux/fs.h
include/linux/stat.h

index b464f8020ec422477c94d68c5649ed2666c63e34..67eae78323ba0d1d07a524371e70b511d2e2b6fa 100644 (file)
@@ -58,8 +58,7 @@ struct stat64 {
        long long       st_size;
        unsigned long   st_blksize;
 
-       unsigned long   st_blocks;      /* Number 512-byte blocks allocated. */
-       unsigned long   __pad4;         /* future possible st_blocks high bits */
+       unsigned long long      st_blocks;      /* Number 512-byte blocks allocated. */
 
        unsigned long   st_atime;
        unsigned long   st_atime_nsec;
index c4c402a45e2164c35f18774cb40fa286af0501aa..dd38bc2e9f989288681ea087c123c867189f8f10 100644 (file)
@@ -60,8 +60,7 @@ struct stat64 {
        long long       st_size;
        unsigned long   st_blksize;
 
-       unsigned long   __pad4;         /* future possible st_blocks high bits */
-       unsigned long   st_blocks;      /* Number 512-byte blocks allocated. */
+       unsigned long long      st_blocks;      /* Number 512-byte blocks allocated. */
 
        unsigned long   st_atime;
        unsigned long   st_atime_nsec;
index 914e3fcbbd37bdacfd33121be517b2fd05374cb8..6c41a60657f191cf3856946c91abb367e0c7cf23 100644 (file)
@@ -60,13 +60,7 @@ struct stat64 {
        long long       st_size;
        unsigned long   st_blksize;
 
-#if defined(__BIG_ENDIAN__)
-       unsigned long   __pad4;         /* Future possible st_blocks hi bits */
-       unsigned long   st_blocks;      /* Number 512-byte blocks allocated. */
-#else /* Must be little */
-       unsigned long   st_blocks;      /* Number 512-byte blocks allocated. */
-       unsigned long   __pad4;         /* Future possible st_blocks hi bits */
-#endif
+       unsigned long long      st_blocks;      /* Number 512-byte blocks allocated. */
 
        unsigned long   st_atime;
        unsigned long   st_atime_nsec;
index ab67181a5a55034b5bca512c3dd7fd2cba8d5add..64b0ca4f14e390e3e66b14f35a7aff51ef10d857 100644 (file)
@@ -490,7 +490,7 @@ struct inode {
        unsigned int            i_blkbits;
        unsigned long           i_blksize;
        unsigned long           i_version;
-       unsigned long           i_blocks;
+       sector_t                i_blocks;
        unsigned short          i_bytes;
        spinlock_t              i_lock; /* i_blocks, i_bytes, maybe i_size */
        struct mutex            i_mutex;
index 8ff2a122dfefc99a3b02431ef3af63a681016d82..8669291352dba326e952d0f2a02e5e2a552101b8 100644 (file)
@@ -69,7 +69,7 @@ struct kstat {
        struct timespec mtime;
        struct timespec ctime;
        unsigned long   blksize;
-       unsigned long   blocks;
+       unsigned long long      blocks;
 };
 
 #endif