summaryrefslogtreecommitdiffstats
path: root/fs/ceph/addr.c
diff options
context:
space:
mode:
authorNeilBrown <neilb@suse.com>2016-09-01 22:26:23 +0800
committerIlya Dryomov <idryomov@gmail.com>2016-10-03 16:13:49 +0200
commit5d7eb1a3225309b294275385ece36fee256d5c46 (patch)
tree58a8af4b49861403ee16e5ccac9d1bdb868c99b4 /fs/ceph/addr.c
parent1afe478569ba7414dde8a874dda9c1ea621c0c63 (diff)
downloadlinux-5d7eb1a3225309b294275385ece36fee256d5c46.tar.bz2
ceph: ignore error from invalidate_inode_pages2_range() in direct write
This call can fail if there are dirty pages. The preceding call to filemap_write_and_wait_range() will normally remove dirty pages, but as inode_lock() is not held over calls to ceph_direct_read_write(), it could race with non-direct writes and pages could be dirtied immediately after filemap_write_and_wait_range() returns If there are dirty pages, they will be removed by the subsequent call to truncate_inode_pages_range(), so having them here is not a problem. If the 'ret' value is left holding an error, then in the async IO case (aio_req is not NULL) the loop that would normally call ceph_osdc_start_request() will see the error in 'ret' and abort all requests. This doesn't seem like correct behaviour. So use separate 'ret2' instead of overloading 'ret'. Signed-off-by: NeilBrown <neilb@suse.com> Reviewed-by: Jeff Layton <jlayton@redhat.com> Reviewed-by: Yan, Zheng <zyan@redhat.com>
Diffstat (limited to 'fs/ceph/addr.c')
0 files changed, 0 insertions, 0 deletions