diff options
| author | David Chinner <dgc@sgi.com> | 2006-03-22 02:47:40 +0100 |
|---|---|---|
| committer | Nathan Scott <nathans@sgi.com> | 2006-03-22 02:47:40 +0100 |
| commit | 2ddee844eef48bf9240ebdfd6c5ffc4333c7d639 (patch) | |
| tree | 16cc4c8b13d31b94f247b54d726cb3d191ce5f6d /fs/xfs | |
| parent | [XFS] Fixup naming inconsistencies found by Pekka Enberg and one from Jan (diff) | |
| download | linux-2ddee844eef48bf9240ebdfd6c5ffc4333c7d639.tar.xz linux-2ddee844eef48bf9240ebdfd6c5ffc4333c7d639.zip | |
[XFS] Check that a page has dirty buffers before finding it acceptable for
rewrite clustering. This prevents writing excessive amounts of clean data
when doing random rewrites of a cached file.
SGI-PV: 951193
SGI-Modid: xfs-linux-melb:xfs-kern:25531a
Signed-off-by: David Chinner <dgc@sgi.com>
Signed-off-by: Nathan Scott <nathans@sgi.com>
Diffstat (limited to 'fs/xfs')
| -rw-r--r-- | fs/xfs/linux-2.6/xfs_aops.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/xfs/linux-2.6/xfs_aops.c b/fs/xfs/linux-2.6/xfs_aops.c index 965757cf15e9..97fc056130eb 100644 --- a/fs/xfs/linux-2.6/xfs_aops.c +++ b/fs/xfs/linux-2.6/xfs_aops.c @@ -647,7 +647,7 @@ xfs_is_delayed_page( acceptable = (type == IOMAP_UNWRITTEN); else if (buffer_delay(bh)) acceptable = (type == IOMAP_DELAY); - else if (buffer_mapped(bh)) + else if (buffer_dirty(bh) && buffer_mapped(bh)) acceptable = (type == 0); else break; |