diff options
| author | Chao Yu <chao2.yu@samsung.com> | 2014-02-28 03:12:05 +0100 |
|---|---|---|
| committer | Jaegeuk Kim <jaegeuk.kim@samsung.com> | 2014-02-28 05:09:08 +0100 |
| commit | 9cf3c3898a274ca637b88ad01b0830550ee2d318 (patch) | |
| tree | b0cdb47c4dde8ed7a3e5463e0bc3e8bc10979355 /fs/f2fs/checkpoint.c | |
| parent | f2fs: use existing macro to clean up some codes (diff) | |
| download | linux-9cf3c3898a274ca637b88ad01b0830550ee2d318.tar.xz linux-9cf3c3898a274ca637b88ad01b0830550ee2d318.zip | |
f2fs: fix dirty page accounting when redirty
We should de-account dirty counters for page when redirty in ->writepage().
Wu Fengguang described in 'commit 971767caf632190f77a40b4011c19948232eed75':
"writeback: fix dirtied pages accounting on redirty
De-account the accumulative dirty counters on page redirty.
Page redirties (very common in ext4) will introduce mismatch between
counters (a) and (b)
a) NR_DIRTIED, BDI_DIRTIED, tsk->nr_dirtied
b) NR_WRITTEN, BDI_WRITTEN
This will introduce systematic errors in balanced_rate and result in
dirty page position errors (ie. the dirty pages are no longer balanced
around the global/bdi setpoints)."
Signed-off-by: Chao Yu <chao2.yu@samsung.com>
Signed-off-by: Jaegeuk Kim <jaegeuk.kim@samsung.com>
Diffstat (limited to 'fs/f2fs/checkpoint.c')
| -rw-r--r-- | fs/f2fs/checkpoint.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/fs/f2fs/checkpoint.c b/fs/f2fs/checkpoint.c index c8516ee24126..f069249011b2 100644 --- a/fs/f2fs/checkpoint.c +++ b/fs/f2fs/checkpoint.c @@ -178,6 +178,7 @@ no_write: redirty_out: dec_page_count(sbi, F2FS_DIRTY_META); wbc->pages_skipped++; + account_page_redirty(page); set_page_dirty(page); return AOP_WRITEPAGE_ACTIVATE; } |