diff options
| author |   Eric Sandeen <sandeen@redhat.com> | 2009-09-17 11:55:58 -0400 |
|---|---|---|
| committer |   Theodore Ts'o <tytso@mit.edu> | 2009-09-17 11:55:58 -0400 |
| commit | 0a80e9867db154966b2a771042e10452ac110e1e (patch) | |
| tree | 51f43830b3c0dd733b0d446399eb3ea2b6bc84ed /fs/ext4/move_extent.c | |
| parent | ext4: Fix the alloc on close after a truncate hueristic (diff) | |
| download | linux-dev-0a80e9867db154966b2a771042e10452ac110e1e.tar.xz linux-dev-0a80e9867db154966b2a771042e10452ac110e1e.zip | |
ext4: replace MAX_DEFRAG_SIZE with EXT_MAX_BLOCK
There's no reason to redefine the maximum allowable offset
in an extent-based file just for defrag;
EXT_MAX_BLOCK already does this.
Signed-off-by: Eric Sandeen <sandeen@redhat.com>
Signed-off-by: "Theodore Ts'o" <tytso@mit.edu>
Diffstat (limited to 'fs/ext4/move_extent.c')
| -rw-r--r-- | fs/ext4/move_extent.c | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/fs/ext4/move_extent.c b/fs/ext4/move_extent.c index 1f027b1ec430..c07a2915e40b 100644 --- a/fs/ext4/move_extent.c +++ b/fs/ext4/move_extent.c @@ -1040,12 +1040,12 @@ mext_check_arguments(struct inode *orig_inode, return -EINVAL; } - if ((orig_start > MAX_DEFRAG_SIZE) || - (donor_start > MAX_DEFRAG_SIZE) || - (*len > MAX_DEFRAG_SIZE) || - (orig_start + *len > MAX_DEFRAG_SIZE)) { - ext4_debug("ext4 move extent: Can't handle over [%lu] blocks " - "[ino:orig %lu, donor %lu]\n", MAX_DEFRAG_SIZE, + if ((orig_start > EXT_MAX_BLOCK) || + (donor_start > EXT_MAX_BLOCK) || + (*len > EXT_MAX_BLOCK) || + (orig_start + *len > EXT_MAX_BLOCK)) { + ext4_debug("ext4 move extent: Can't handle over [%u] blocks " + "[ino:orig %lu, donor %lu]\n", EXT_MAX_BLOCK, orig_inode->i_ino, donor_inode->i_ino); return -EINVAL; } |