consistent treatment of EFAULT on O_DIRECT read/write
authorAl Viro <viro@zeniv.linux.org.uk>
Tue, 4 Oct 2016 00:38:55 +0000 (20:38 -0400)
committerAl Viro <viro@zeniv.linux.org.uk>
Tue, 4 Oct 2016 00:38:55 +0000 (20:38 -0400)
Make local filesystems treat a fault as shortened IO,
returning -EFAULT only if nothing had been transferred.
That's how everything else (NFS, FUSE, ceph, Lustre)
behaves.

Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
fs/direct-io.c

index 7c3ce73cb6170ee820aec560ece2fafc8b48b83c..fb9aa16a77272818e39294b21506927076da27d7 100644 (file)
@@ -246,6 +246,9 @@ static ssize_t dio_complete(struct dio *dio, ssize_t ret, bool is_async)
                if ((dio->op == REQ_OP_READ) &&
                    ((offset + transferred) > dio->i_size))
                        transferred = dio->i_size - offset;
+               /* ignore EFAULT if some IO has been done */
+               if (unlikely(ret == -EFAULT) && transferred)
+                       ret = 0;
        }
 
        if (ret == 0)