diff options
author | Jan Kara <jack@suse.cz> | 2010-03-29 07:55:39 -0400 |
---|---|---|
committer | Jan Kara <jack@suse.cz> | 2010-05-21 13:30:37 -0400 |
commit | 46891532370e862d6bddedef9e6ca22a59a51fa4 (patch) | |
tree | 0e4d6128f7f4a1f67d6c3ede3504100ec5d5ee7a | |
parent | 8cef107a1d890ff76c85d665083ff3562be01d32 (diff) |
ext2: Avoid loading bitmaps for full groups during block allocation
There is no point in loading bitmap for groups which are completely full.
This causes noticeable performance problems (and memory pressure) on small
systems with large full filesystem
(http://marc.info/?l=linux-ext4&m=126843108314310&w=2).
Port of the same ext3 patch.
Signed-off-by: Jan Kara <jack@suse.cz>
-rw-r--r-- | fs/ext2/balloc.c | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/fs/ext2/balloc.c b/fs/ext2/balloc.c index 3cf038c055d7..e8766a396776 100644 --- a/fs/ext2/balloc.c +++ b/fs/ext2/balloc.c | |||
@@ -1332,6 +1332,12 @@ retry_alloc: | |||
1332 | 1332 | ||
1333 | free_blocks = le16_to_cpu(gdp->bg_free_blocks_count); | 1333 | free_blocks = le16_to_cpu(gdp->bg_free_blocks_count); |
1334 | /* | 1334 | /* |
1335 | * skip this group (and avoid loading bitmap) if there | ||
1336 | * are no free blocks | ||
1337 | */ | ||
1338 | if (!free_blocks) | ||
1339 | continue; | ||
1340 | /* | ||
1335 | * skip this group if the number of | 1341 | * skip this group if the number of |
1336 | * free blocks is less than half of the reservation | 1342 | * free blocks is less than half of the reservation |
1337 | * window size. | 1343 | * window size. |