diff options
| author |   Al Viro <viro@zeniv.linux.org.uk> | 2016-10-10 13:26:27 -0400 |
|---|---|---|
| committer | Al Viro <viro@zeniv.linux.org.uk> | 2016-10-10 13:36:06 -0400 |
| commit | c3a690240423fc4eb8a0c3c7df025d13eadf140b (patch) | |
| tree | 20f4a32910427c2aa4ccf8e33633d6e7194628bf /fs/xfs/xfs_file.c | |
| parent | pipe: fix comment in pipe_buf_operations (diff) | |
| download | linux-dev-c3a690240423fc4eb8a0c3c7df025d13eadf140b.tar.xz linux-dev-c3a690240423fc4eb8a0c3c7df025d13eadf140b.zip | |
fix ITER_PIPE interaction with direct_IO
by making sure we call iov_iter_advance() on original
iov_iter even if direct_IO (done on its copy) has returned 0.
It's a no-op for old iov_iter flavours and does the right thing
(== truncation of the stuff we'd allocated, but not filled) in
ITER_PIPE case. Failures (e.g. -EIO) get caught and dealt with
by cleanup in generic_file_read_iter().
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Diffstat (limited to '')
| -rw-r--r-- | fs/xfs/xfs_file.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/xfs/xfs_file.c b/fs/xfs/xfs_file.c index 92f16cfb81e8..26cf153e3714 100644 --- a/fs/xfs/xfs_file.c +++ b/fs/xfs/xfs_file.c @@ -317,7 +317,7 @@ xfs_file_dio_aio_read( data = *to; ret = __blockdev_direct_IO(iocb, inode, target->bt_bdev, &data, xfs_get_blocks_direct, NULL, NULL, 0); - if (ret > 0) { + if (ret >= 0) { iocb->ki_pos += ret; iov_iter_advance(to, ret); } |