aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDavid Gibson <david@gibson.dropbear.id.au>2005-11-22 00:32:24 -0500
committerLinus Torvalds <torvalds@g5.osdl.org>2005-11-22 12:13:43 -0500
commit74a8a65c526187fe636a6a2abcb7d9ebc5c753ab (patch)
tree97900dc68c67eb0d9e2010e34a0c00e27c1ffc1d
parent86e07ce71a8aad5074f7316f9b297d2137630283 (diff)
[PATCH] Fix hugetlbfs_statfs() reporting of block limits
Currently, if a hugetlbfs is mounted without limits (the default), statfs() will return -1 for max/free/used blocks. This does not appear to be in line with normal convention: simple_statfs() and shmem_statfs() both return 0 in similar cases. Worse, it confuses the translation logic in put_compat_statfs(), causing it to return -EOVERFLOW on such a mount. This patch alters hugetlbfs_statfs() to return 0 for max/free/used blocks on a mount without limits. Note that we need the test in the patch below, rather than just using 0 in the sbinfo structure, because the -1 marked in the free blocks field is used internally to tell the Signed-off-by: David Gibson <david@gibson.dropbear.id.au> Signed-off-by: Andrew Morton <akpm@osdl.org> Signed-off-by: Linus Torvalds <torvalds@osdl.org>
-rw-r--r--fs/hugetlbfs/inode.c12
1 files changed, 8 insertions, 4 deletions
diff --git a/fs/hugetlbfs/inode.c b/fs/hugetlbfs/inode.c
index 64983ab55586..8c1cef3bb677 100644
--- a/fs/hugetlbfs/inode.c
+++ b/fs/hugetlbfs/inode.c
@@ -512,10 +512,14 @@ static int hugetlbfs_statfs(struct super_block *sb, struct kstatfs *buf)
512 buf->f_bsize = HPAGE_SIZE; 512 buf->f_bsize = HPAGE_SIZE;
513 if (sbinfo) { 513 if (sbinfo) {
514 spin_lock(&sbinfo->stat_lock); 514 spin_lock(&sbinfo->stat_lock);
515 buf->f_blocks = sbinfo->max_blocks; 515 /* If no limits set, just report 0 for max/free/used
516 buf->f_bavail = buf->f_bfree = sbinfo->free_blocks; 516 * blocks, like simple_statfs() */
517 buf->f_files = sbinfo->max_inodes; 517 if (sbinfo->max_blocks >= 0) {
518 buf->f_ffree = sbinfo->free_inodes; 518 buf->f_blocks = sbinfo->max_blocks;
519 buf->f_bavail = buf->f_bfree = sbinfo->free_blocks;
520 buf->f_files = sbinfo->max_inodes;
521 buf->f_ffree = sbinfo->free_inodes;
522 }
519 spin_unlock(&sbinfo->stat_lock); 523 spin_unlock(&sbinfo->stat_lock);
520 } 524 }
521 buf->f_namelen = NAME_MAX; 525 buf->f_namelen = NAME_MAX;