diff options
| author | Andreas Gruenbacher <agruenba@redhat.com> | 2021-08-02 13:44:20 +0200 |
|---|---|---|
| committer | Andreas Gruenbacher <agruenba@redhat.com> | 2021-10-18 16:33:03 +0200 |
| commit | bb523b406c849eef8f265a07cd7f320f1f177743 (patch) | |
| tree | 049910999167cc360d3fe794e1c9944fd9b50285 /fs/btrfs | |
| parent | 0c8eb2884a42d992c7726539328b7d3568f22143 (diff) | |
| download | linux-stable-bb523b406c849eef8f265a07cd7f320f1f177743.tar.gz linux-stable-bb523b406c849eef8f265a07cd7f320f1f177743.tar.bz2 linux-stable-bb523b406c849eef8f265a07cd7f320f1f177743.zip | |
gup: Turn fault_in_pages_{readable,writeable} into fault_in_{readable,writeable}
Turn fault_in_pages_{readable,writeable} into versions that return the
number of bytes not faulted in, similar to copy_to_user, instead of
returning a non-zero value when any of the requested pages couldn't be
faulted in. This supports the existing users that require all pages to
be faulted in as well as new users that are happy if any pages can be
faulted in.
Rename the functions to fault_in_{readable,writeable} to make sure
this change doesn't silently break things.
Neither of these functions is entirely trivial and it doesn't seem
useful to inline them, so move them to mm/gup.c.
Signed-off-by: Andreas Gruenbacher <agruenba@redhat.com>
Diffstat (limited to 'fs/btrfs')
| -rw-r--r-- | fs/btrfs/ioctl.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/fs/btrfs/ioctl.c b/fs/btrfs/ioctl.c index cc61813213d8..c0739f0af634 100644 --- a/fs/btrfs/ioctl.c +++ b/fs/btrfs/ioctl.c @@ -2261,9 +2261,8 @@ static noinline int search_ioctl(struct inode *inode, key.offset = sk->min_offset; while (1) { - ret = fault_in_pages_writeable(ubuf + sk_offset, - *buf_size - sk_offset); - if (ret) + ret = -EFAULT; + if (fault_in_writeable(ubuf + sk_offset, *buf_size - sk_offset)) break; ret = btrfs_search_forward(root, &key, path, sk->min_transid); |