diff options
author | Christoph Hellwig <hch@infradead.org> | 2010-08-18 05:29:15 -0400 |
---|---|---|
committer | Jens Axboe <jaxboe@fusionio.com> | 2010-09-10 06:35:39 -0400 |
commit | f8c131f5b6ffc899a70b30e541f367d47f89691c (patch) | |
tree | c1f4bc92dc382b80004b5b63223f16e67219986e /fs/nilfs2/super.c | |
parent | 7cd33ad23ec41d685902159be8b2c6552fab8bd0 (diff) |
nilfs2: replace barriers with explicit flush / FUA usage
Switch to the WRITE_FLUSH_FUA flag for log writes, remove the EOPNOTSUPP
detection for barriers and stop setting the barrier flag for discards.
tj: nilfs is now fixed to wait for discard completion. Updated this
patch accordingly and dropped warning about it.
Signed-off-by: Christoph Hellwig <hch@lst.de>
Acked-by: Ryusuke Konishi <konishi.ryusuke@lab.ntt.co.jp>
Signed-off-by: Tejun Heo <tj@kernel.org>
Signed-off-by: Jens Axboe <jaxboe@fusionio.com>
Diffstat (limited to 'fs/nilfs2/super.c')
-rw-r--r-- | fs/nilfs2/super.c | 10 |
1 files changed, 1 insertions, 9 deletions
diff --git a/fs/nilfs2/super.c b/fs/nilfs2/super.c index 922263393c76..faa5078ff751 100644 --- a/fs/nilfs2/super.c +++ b/fs/nilfs2/super.c | |||
@@ -178,17 +178,9 @@ static int nilfs_sync_super(struct nilfs_sb_info *sbi, int flag) | |||
178 | 178 | ||
179 | retry: | 179 | retry: |
180 | set_buffer_dirty(nilfs->ns_sbh[0]); | 180 | set_buffer_dirty(nilfs->ns_sbh[0]); |
181 | |||
182 | if (nilfs_test_opt(sbi, BARRIER)) { | 181 | if (nilfs_test_opt(sbi, BARRIER)) { |
183 | err = __sync_dirty_buffer(nilfs->ns_sbh[0], | 182 | err = __sync_dirty_buffer(nilfs->ns_sbh[0], |
184 | WRITE_SYNC | WRITE_BARRIER); | 183 | WRITE_SYNC | WRITE_FLUSH_FUA); |
185 | if (err == -EOPNOTSUPP) { | ||
186 | nilfs_warning(sbi->s_super, __func__, | ||
187 | "barrier-based sync failed. " | ||
188 | "disabling barriers\n"); | ||
189 | nilfs_clear_opt(sbi, BARRIER); | ||
190 | goto retry; | ||
191 | } | ||
192 | } else { | 184 | } else { |
193 | err = sync_dirty_buffer(nilfs->ns_sbh[0]); | 185 | err = sync_dirty_buffer(nilfs->ns_sbh[0]); |
194 | } | 186 | } |