[Cluster-devel] [PATCH 12/13] fuse: drop redundant arguments to fuse_perform_write

2023-05-19 Thread Christoph Hellwig
pos is always equal to iocb->ki_pos, and mapping is always equal to
iocb->ki_filp->f_mapping.

Signed-off-by: Christoph Hellwig 
---
 fs/fuse/file.c | 11 +--
 1 file changed, 5 insertions(+), 6 deletions(-)

diff --git a/fs/fuse/file.c b/fs/fuse/file.c
index fd2f27f2144750..5f7b58798f99fc 100644
--- a/fs/fuse/file.c
+++ b/fs/fuse/file.c
@@ -1280,13 +1280,13 @@ static inline unsigned int fuse_wr_pages(loff_t pos, 
size_t len,
 max_pages);
 }
 
-static ssize_t fuse_perform_write(struct kiocb *iocb,
- struct address_space *mapping,
- struct iov_iter *ii, loff_t pos)
+static ssize_t fuse_perform_write(struct kiocb *iocb, struct iov_iter *ii)
 {
+   struct address_space *mapping = iocb->ki_filp->f_mapping;
struct inode *inode = mapping->host;
struct fuse_conn *fc = get_fuse_conn(inode);
struct fuse_inode *fi = get_fuse_inode(inode);
+   loff_t pos = iocb->ki_pos;
int err = 0;
ssize_t res = 0;
 
@@ -1385,8 +1385,7 @@ static ssize_t fuse_cache_write_iter(struct kiocb *iocb, 
struct iov_iter *from)
if (written < 0 || !iov_iter_count(from))
goto out;
 
-   written_buffered = fuse_perform_write(iocb, mapping, from,
- iocb->ki_pos);
+   written_buffered = fuse_perform_write(iocb, from);
if (written_buffered < 0) {
err = written_buffered;
goto out;
@@ -1406,7 +1405,7 @@ static ssize_t fuse_cache_write_iter(struct kiocb *iocb, 
struct iov_iter *from)
written += written_buffered;
iocb->ki_pos += written_buffered;
} else {
-   written = fuse_perform_write(iocb, mapping, from, iocb->ki_pos);
+   written = fuse_perform_write(iocb, from);
}
 out:
current->backing_dev_info = NULL;
-- 
2.39.2



Re: [Cluster-devel] [PATCH 12/13] fuse: drop redundant arguments to fuse_perform_write

2023-05-22 Thread Damien Le Moal
On 5/19/23 18:35, Christoph Hellwig wrote:
> pos is always equal to iocb->ki_pos, and mapping is always equal to
> iocb->ki_filp->f_mapping.
> 
> Signed-off-by: Christoph Hellwig 

Reviewed-by: Damien Le Moal 

-- 
Damien Le Moal
Western Digital Research