diff options
author | Jan Kara <jack@suse.cz> | 2012-07-09 17:40:46 -0400 |
---|---|---|
committer | Jan Kara <jack@suse.cz> | 2012-07-09 17:40:46 -0400 |
commit | 44f4f729e7a143b08bd63c33cb78b3181d9f4716 (patch) | |
tree | 6a49af7fcbc83b7eec5b98cbb2ac614ed3191309 /fs/ext3 | |
parent | 349ecd6a3c0e4f97fa4dc6bd3917455ccc106d23 (diff) |
ext3: Check return value of blkdev_issue_flush()
blkdev_issue_flush() can fail. Make sure the error gets properly propagated.
Signed-off-by: Jan Kara <jack@suse.cz>
Diffstat (limited to 'fs/ext3')
-rw-r--r-- | fs/ext3/fsync.c | 9 |
1 files changed, 7 insertions, 2 deletions
diff --git a/fs/ext3/fsync.c b/fs/ext3/fsync.c index d4dff278cbd8..b31dbd4c46ad 100644 --- a/fs/ext3/fsync.c +++ b/fs/ext3/fsync.c | |||
@@ -92,8 +92,13 @@ int ext3_sync_file(struct file *file, loff_t start, loff_t end, int datasync) | |||
92 | * disk caches manually so that data really is on persistent | 92 | * disk caches manually so that data really is on persistent |
93 | * storage | 93 | * storage |
94 | */ | 94 | */ |
95 | if (needs_barrier) | 95 | if (needs_barrier) { |
96 | blkdev_issue_flush(inode->i_sb->s_bdev, GFP_KERNEL, NULL); | 96 | int err; |
97 | |||
98 | err = blkdev_issue_flush(inode->i_sb->s_bdev, GFP_KERNEL, NULL); | ||
99 | if (!ret) | ||
100 | ret = err; | ||
101 | } | ||
97 | out: | 102 | out: |
98 | trace_ext3_sync_file_exit(inode, ret); | 103 | trace_ext3_sync_file_exit(inode, ret); |
99 | return ret; | 104 | return ret; |