fuse: cleanup fuse_direct_io()

Fix the following sparse warnings:

fs/fuse/file.c:1216:43: warning: cast removes address space of expression
fs/fuse/file.c:1216:43: warning: incorrect type in initializer (different address spaces)
fs/fuse/file.c:1216:43:    expected void [noderef] <asn:1>*iov_base
fs/fuse/file.c:1216:43:    got void *<noident>
fs/fuse/file.c:1241:43: warning: cast removes address space of expression
fs/fuse/file.c:1241:43: warning: incorrect type in initializer (different address spaces)
fs/fuse/file.c:1241:43:    expected void [noderef] <asn:1>*iov_base
fs/fuse/file.c:1241:43:    got void *<noident>
fs/fuse/file.c:1267:43: warning: cast removes address space of expression
fs/fuse/file.c:1267:43: warning: incorrect type in initializer (different address spaces)
fs/fuse/file.c:1267:43:    expected void [noderef] <asn:1>*iov_base
fs/fuse/file.c:1267:43:    got void *<noident>

Signed-off-by: Miklos Szeredi <mszeredi@suse.cz>

diff --git a/fs/fuse/cuse.c b/fs/fuse/cuse.c
index fb9dcfdf040bc76f8a276292d4de014c673df7ba..6f96a8def14766be9fa173e26f12583dc3f04697 100644 (file)
--- a/fs/fuse/cuse.c
+++ b/fs/fuse/cuse.c
@@ -91,19 +91,22 @@ static ssize_t cuse_read(struct file *file, char __user *buf, size_t count,
                         loff_t *ppos)
 {
        loff_t pos = 0;
+       struct iovec iov = { .iov_base = buf, .iov_len = count };
 
-       return fuse_direct_io(file, buf, count, &pos, 0);
+       return fuse_direct_io(file, &iov, 1, count, &pos, 0);
 }
 
 static ssize_t cuse_write(struct file *file, const char __user *buf,
                          size_t count, loff_t *ppos)
 {
        loff_t pos = 0;
+       struct iovec iov = { .iov_base = (void __user *)buf, .iov_len = count };
+
        /*
         * No locking or generic_write_checks(), the server is
         * responsible for locking and sanity checks.
         */
-       return fuse_direct_io(file, buf, count, &pos, 1);
+       return fuse_direct_io(file, &iov, 1, count, &pos, 1);
 }
 
 static int cuse_open(struct inode *inode, struct file *file)

diff --git a/fs/fuse/file.c b/fs/fuse/file.c
index 68e10d43bd3f355bdd88489545109d19344aa6e9..28bc9c672196331e02b49783de494f0cec24fe08 100644 (file)
--- a/fs/fuse/file.c
+++ b/fs/fuse/file.c
@@ -1148,9 +1148,9 @@ static inline int fuse_iter_npages(const struct iov_iter *ii_p)
        return min(npages, FUSE_MAX_PAGES_PER_REQ);
 }
 
-static ssize_t __fuse_direct_io(struct file *file, const struct iovec *iov,
-                               unsigned long nr_segs, size_t count,
-                               loff_t *ppos, int write)
+ssize_t fuse_direct_io(struct file *file, const struct iovec *iov,
+                      unsigned long nr_segs, size_t count, loff_t *ppos,
+                      int write)
 {
        struct fuse_file *ff = file->private_data;
        struct fuse_conn *fc = ff->fc;
@@ -1209,13 +1209,6 @@ static ssize_t __fuse_direct_io(struct file *file, const struct iovec *iov,
 
        return res;
 }
-
-ssize_t fuse_direct_io(struct file *file, const char __user *buf,
-                      size_t count, loff_t *ppos, int write)
-{
-       struct iovec iov = { .iov_base = (void *)buf, .iov_len = count };
-       return __fuse_direct_io(file, &iov, 1, count, ppos, write);
-}
 EXPORT_SYMBOL_GPL(fuse_direct_io);
 
 static ssize_t __fuse_direct_read(struct file *file, const struct iovec *iov,
@@ -1227,8 +1220,8 @@ static ssize_t __fuse_direct_read(struct file *file, const struct iovec *iov,
        if (is_bad_inode(inode))
                return -EIO;
 
-       res = __fuse_direct_io(file, iov, nr_segs, iov_length(iov, nr_segs),
-                              ppos, 0);
+       res = fuse_direct_io(file, iov, nr_segs, iov_length(iov, nr_segs),
+                            ppos, 0);
 
        fuse_invalidate_attr(inode);
 
@@ -1238,7 +1231,7 @@ static ssize_t __fuse_direct_read(struct file *file, const struct iovec *iov,
 static ssize_t fuse_direct_read(struct file *file, char __user *buf,
                                     size_t count, loff_t *ppos)
 {
-       struct iovec iov = { .iov_base = (void *)buf, .iov_len = count };
+       struct iovec iov = { .iov_base = buf, .iov_len = count };
        return __fuse_direct_read(file, &iov, 1, ppos);
 }
 
@@ -1251,7 +1244,7 @@ static ssize_t __fuse_direct_write(struct file *file, const struct iovec *iov,
 
        res = generic_write_checks(file, ppos, &count, 0);
        if (!res) {
-               res = __fuse_direct_io(file, iov, nr_segs, count, ppos, 1);
+               res = fuse_direct_io(file, iov, nr_segs, count, ppos, 1);
                if (res > 0)
                        fuse_write_update_size(inode, *ppos);
        }
@@ -1264,7 +1257,7 @@ static ssize_t __fuse_direct_write(struct file *file, const struct iovec *iov,
 static ssize_t fuse_direct_write(struct file *file, const char __user *buf,
                                 size_t count, loff_t *ppos)
 {
-       struct iovec iov = { .iov_base = (void *)buf, .iov_len = count };
+       struct iovec iov = { .iov_base = (void __user *)buf, .iov_len = count };
        struct inode *inode = file->f_path.dentry->d_inode;
        ssize_t res;
 

diff --git a/fs/fuse/fuse_i.h b/fs/fuse/fuse_i.h
index 70cef60afe0ec7fd5923b457f146b942c3323428..13befcd29c5b5eaac5978fbe3e98b6a4dc25bd25 100644 (file)
--- a/fs/fuse/fuse_i.h
+++ b/fs/fuse/fuse_i.h
@@ -811,8 +811,9 @@ int fuse_reverse_inval_entry(struct super_block *sb, u64 parent_nodeid,
 
 int fuse_do_open(struct fuse_conn *fc, u64 nodeid, struct file *file,
                 bool isdir);
-ssize_t fuse_direct_io(struct file *file, const char __user *buf,
-                      size_t count, loff_t *ppos, int write);
+ssize_t fuse_direct_io(struct file *file, const struct iovec *iov,
+                      unsigned long nr_segs, size_t count, loff_t *ppos,
+                      int write);
 long fuse_do_ioctl(struct file *file, unsigned int cmd, unsigned long arg,
                   unsigned int flags);
 long fuse_ioctl_common(struct file *file, unsigned int cmd,