diff options
author | Eric Sandeen <sandeen@redhat.com> | 2009-09-16 14:45:10 -0400 |
---|---|---|
committer | Theodore Ts'o <tytso@mit.edu> | 2009-09-16 14:45:10 -0400 |
commit | fb0a387dcdcd21aab1b09ee7fd80b7c979bdbbfd (patch) | |
tree | dcb12c5fce8f7ccb8b183936ea71a29aba3f3846 /fs/ext4/ext4.h | |
parent | c40ce3c9ea97425a12d7e44031a98fe50add6fc1 (diff) |
ext4: limit block allocations for indirect-block files to < 2^32
Today, the ext4 allocator will happily allocate blocks past
2^32 for indirect-block files, which results in the block
numbers getting truncated, and corruption ensues.
This patch limits such allocations to < 2^32, and adds
BUG_ONs if we do get blocks larger than that.
This should address RH Bug 519471, ext4 bitmap allocator
must limit blocks to < 2^32
* ext4_find_goal() is modified to choose a goal < UINT_MAX,
so that our starting point is in an acceptable range.
* ext4_xattr_block_set() is modified such that the goal block
is < UINT_MAX, as above.
* ext4_mb_regular_allocator() is modified so that the group
search does not continue into groups which are too high
* ext4_mb_use_preallocated() has a check that we don't use
preallocated space which is too far out
* ext4_alloc_blocks() and ext4_xattr_block_set() add some BUG_ONs
No attempt has been made to limit inode locations to < 2^32,
so we may wind up with blocks far from their inodes. Doing
this much already will lead to some odd ENOSPC issues when the
"lower 32" gets full, and further restricting inodes could
make that even weirder.
For high inodes, choosing a goal of the original, % UINT_MAX,
may be a bit odd, but then we're in an odd situation anyway,
and I don't know of a better heuristic.
Signed-off-by: Eric Sandeen <sandeen@redhat.com>
Signed-off-by: "Theodore Ts'o" <tytso@mit.edu>
Diffstat (limited to 'fs/ext4/ext4.h')
-rw-r--r-- | fs/ext4/ext4.h | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/fs/ext4/ext4.h b/fs/ext4/ext4.h index 4dc64ed58d26..2a0f75d55fad 100644 --- a/fs/ext4/ext4.h +++ b/fs/ext4/ext4.h | |||
@@ -403,6 +403,9 @@ struct ext4_mount_options { | |||
403 | #endif | 403 | #endif |
404 | }; | 404 | }; |
405 | 405 | ||
406 | /* Max physical block we can addres w/o extents */ | ||
407 | #define EXT4_MAX_BLOCK_FILE_PHYS 0xFFFFFFFF | ||
408 | |||
406 | /* | 409 | /* |
407 | * Structure of an inode on the disk | 410 | * Structure of an inode on the disk |
408 | */ | 411 | */ |
@@ -857,6 +860,7 @@ struct ext4_sb_info { | |||
857 | unsigned long s_gdb_count; /* Number of group descriptor blocks */ | 860 | unsigned long s_gdb_count; /* Number of group descriptor blocks */ |
858 | unsigned long s_desc_per_block; /* Number of group descriptors per block */ | 861 | unsigned long s_desc_per_block; /* Number of group descriptors per block */ |
859 | ext4_group_t s_groups_count; /* Number of groups in the fs */ | 862 | ext4_group_t s_groups_count; /* Number of groups in the fs */ |
863 | ext4_group_t s_blockfile_groups;/* Groups acceptable for non-extent files */ | ||
860 | unsigned long s_overhead_last; /* Last calculated overhead */ | 864 | unsigned long s_overhead_last; /* Last calculated overhead */ |
861 | unsigned long s_blocks_last; /* Last seen block count */ | 865 | unsigned long s_blocks_last; /* Last seen block count */ |
862 | loff_t s_bitmap_maxbytes; /* max bytes for bitmap files */ | 866 | loff_t s_bitmap_maxbytes; /* max bytes for bitmap files */ |