Skip to content

Commit

Permalink
[PATCH] 2TB files: st_blocks is invalid when calling stat64
Browse files Browse the repository at this point in the history
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>
  • Loading branch information
Takashi Sato authored and Linus Torvalds committed Mar 26, 2006
1 parent 93d2341 commit abcb6c9
Show file tree
Hide file tree
Showing 5 changed files with 5 additions and 13 deletions.
3 changes: 1 addition & 2 deletions include/asm-i386/stat.h
Original file line number Diff line number Diff line change
Expand Up @@ -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;
Expand Down
3 changes: 1 addition & 2 deletions include/asm-m68k/stat.h
Original file line number Diff line number Diff line change
Expand Up @@ -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;
Expand Down
8 changes: 1 addition & 7 deletions include/asm-sh/stat.h
Original file line number Diff line number Diff line change
Expand Up @@ -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;
Expand Down
2 changes: 1 addition & 1 deletion include/linux/fs.h
Original file line number Diff line number Diff line change
Expand Up @@ -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;
Expand Down
2 changes: 1 addition & 1 deletion include/linux/stat.h
Original file line number Diff line number Diff line change
Expand Up @@ -69,7 +69,7 @@ struct kstat {
struct timespec mtime;
struct timespec ctime;
unsigned long blksize;
unsigned long blocks;
unsigned long long blocks;
};

#endif
Expand Down

0 comments on commit abcb6c9

Please sign in to comment.