diff options
author | Jan Kara <jack@suse.cz> | 2013-04-09 18:39:26 +0200 |
---|---|---|
committer | Theodore Ts'o <tytso@mit.edu> | 2013-04-09 18:39:26 +0200 |
commit | f45a5ef91bef7e02149a216ed6dc3fcdd8b38268 (patch) | |
tree | f42e693444d3f2456a7c0a002cffdc1f31f6eeea | |
parent | ext4: speed-up releasing blocks on commit (diff) | |
download | linux-f45a5ef91bef7e02149a216ed6dc3fcdd8b38268.tar.xz linux-f45a5ef91bef7e02149a216ed6dc3fcdd8b38268.zip |
ext4: improve credit estimate for EXT4_SINGLEDATA_TRANS_BLOCKS
Estimate of 27 credits for allocation of a block in extent based inode
is unnecessarily high. We can easily argue 20 is enough.
Signed-off-by: Jan Kara <jack@suse.cz>
Signed-off-by: "Theodore Ts'o" <tytso@mit.edu>
-rw-r--r-- | fs/ext4/ext4_jbd2.h | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/fs/ext4/ext4_jbd2.h b/fs/ext4/ext4_jbd2.h index aeed0bac693b..c8c6885406db 100644 --- a/fs/ext4/ext4_jbd2.h +++ b/fs/ext4/ext4_jbd2.h @@ -29,11 +29,13 @@ * block to complete the transaction. * * For extents-enabled fs we may have to allocate and modify up to - * 5 levels of tree + root which are stored in the inode. */ + * 5 levels of tree, data block (for each of these we need bitmap + group + * summaries), root which is stored in the inode, sb + */ #define EXT4_SINGLEDATA_TRANS_BLOCKS(sb) \ (EXT4_HAS_INCOMPAT_FEATURE(sb, EXT4_FEATURE_INCOMPAT_EXTENTS) \ - ? 27U : 8U) + ? 20U : 8U) /* Extended attribute operations touch at most two data buffers, * two bitmap buffers, and two group summaries, in addition to the inode |