Commit 26c87fb7 authored by Christoph Hellwig's avatar Christoph Hellwig Committed by Al Viro

fs: remove do_compat_readv_writev

opencode it in both callers to simplify the call stack a bit.
Signed-off-by: default avatarChristoph Hellwig <hch@lst.de>
Signed-off-by: default avatarAl Viro <viro@zeniv.linux.org.uk>
parent 251b42a1
...@@ -1142,32 +1142,13 @@ SYSCALL_DEFINE6(pwritev2, unsigned long, fd, const struct iovec __user *, vec, ...@@ -1142,32 +1142,13 @@ SYSCALL_DEFINE6(pwritev2, unsigned long, fd, const struct iovec __user *, vec,
} }
#ifdef CONFIG_COMPAT #ifdef CONFIG_COMPAT
static ssize_t compat_do_readv_writev(int type, struct file *file,
const struct compat_iovec __user *uvector,
unsigned long nr_segs, loff_t *pos,
int flags)
{
struct iovec iovstack[UIO_FASTIOV];
struct iovec *iov = iovstack;
struct iov_iter iter;
ssize_t ret;
ret = compat_import_iovec(type, uvector, nr_segs,
UIO_FASTIOV, &iov, &iter);
if (ret < 0)
return ret;
ret = __do_readv_writev(type, file, &iter, pos, flags);
kfree(iov);
return ret;
}
static size_t compat_readv(struct file *file, static size_t compat_readv(struct file *file,
const struct compat_iovec __user *vec, const struct compat_iovec __user *vec,
unsigned long vlen, loff_t *pos, int flags) unsigned long vlen, loff_t *pos, int flags)
{ {
struct iovec iovstack[UIO_FASTIOV];
struct iovec *iov = iovstack;
struct iov_iter iter;
ssize_t ret = -EBADF; ssize_t ret = -EBADF;
if (!(file->f_mode & FMODE_READ)) if (!(file->f_mode & FMODE_READ))
...@@ -1177,8 +1158,11 @@ static size_t compat_readv(struct file *file, ...@@ -1177,8 +1158,11 @@ static size_t compat_readv(struct file *file,
if (!(file->f_mode & FMODE_CAN_READ)) if (!(file->f_mode & FMODE_CAN_READ))
goto out; goto out;
ret = compat_do_readv_writev(READ, file, vec, vlen, pos, flags); ret = compat_import_iovec(READ, vec, vlen, UIO_FASTIOV, &iov, &iter);
if (ret < 0)
goto out;
ret = __do_readv_writev(READ, file, &iter, pos, flags);
kfree(iov);
out: out:
if (ret > 0) if (ret > 0)
add_rchar(current, ret); add_rchar(current, ret);
...@@ -1275,6 +1259,9 @@ static size_t compat_writev(struct file *file, ...@@ -1275,6 +1259,9 @@ static size_t compat_writev(struct file *file,
const struct compat_iovec __user *vec, const struct compat_iovec __user *vec,
unsigned long vlen, loff_t *pos, int flags) unsigned long vlen, loff_t *pos, int flags)
{ {
struct iovec iovstack[UIO_FASTIOV];
struct iovec *iov = iovstack;
struct iov_iter iter;
ssize_t ret = -EBADF; ssize_t ret = -EBADF;
if (!(file->f_mode & FMODE_WRITE)) if (!(file->f_mode & FMODE_WRITE))
...@@ -1284,8 +1271,11 @@ static size_t compat_writev(struct file *file, ...@@ -1284,8 +1271,11 @@ static size_t compat_writev(struct file *file,
if (!(file->f_mode & FMODE_CAN_WRITE)) if (!(file->f_mode & FMODE_CAN_WRITE))
goto out; goto out;
ret = compat_do_readv_writev(WRITE, file, vec, vlen, pos, flags); ret = compat_import_iovec(WRITE, vec, vlen, UIO_FASTIOV, &iov, &iter);
if (ret < 0)
goto out;
ret = __do_readv_writev(WRITE, file, &iter, pos, flags);
kfree(iov);
out: out:
if (ret > 0) if (ret > 0)
add_wchar(current, ret); add_wchar(current, ret);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment