diff options
| author | Jaegeuk Kim <jaegeuk.kim@samsung.com> | 2014-03-03 11:28:40 +0900 |
|---|---|---|
| committer | Ziyan <jaraidaniel@gmail.com> | 2015-05-02 14:36:27 +0200 |
| commit | dadb1edcbcfd1b308504675582d77a0bc98631df (patch) | |
| tree | fe8fe0408065f490e3a4d59672060c2a796b2309 /fs | |
| parent | 8bd90b3a2146641572b9876ee6029dfa11efe123 (diff) | |
| download | kernel_samsung_tuna-dadb1edcbcfd1b308504675582d77a0bc98631df.zip kernel_samsung_tuna-dadb1edcbcfd1b308504675582d77a0bc98631df.tar.gz kernel_samsung_tuna-dadb1edcbcfd1b308504675582d77a0bc98631df.tar.bz2 | |
f2fs: fix to write node pages with WRITE_SYNC
This patch fixes performance regression of dbench reported by
Alex <hbx7d@yandex.com>.
This issue was revealed by Phoronix tests results:
http://www.phoronix.com/scan.php?page=article&item=linux_314_ssdfs&num=2
It turns out that we need to assign WRITE_SYNC to the node writes, if
fsync is triggered.
The performance numbers are like below, which is measured by Alex.
1. 355MB/s ext4
2. 225MB/s f2fs : WRITE for node writes
3. 525MB/s f2fs : WRITE_SYNC for node writes
Reported-And-Tested-by: Alex <hbx7d@yandex.com>.
Signed-off-by: Jaegeuk Kim <jaegeuk.kim@samsung.com>
Diffstat (limited to 'fs')
| -rw-r--r-- | fs/f2fs/file.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/f2fs/file.c b/fs/f2fs/file.c index facf03d..9dc83e3 100644 --- a/fs/f2fs/file.c +++ b/fs/f2fs/file.c @@ -117,7 +117,7 @@ int f2fs_sync_file(struct file *file, loff_t start, loff_t end, int datasync) int ret = 0; bool need_cp = false; struct writeback_control wbc = { - .sync_mode = WB_SYNC_NONE, + .sync_mode = WB_SYNC_ALL, .nr_to_write = LONG_MAX, .for_reclaim = 0, }; |