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

[Hannes Reinecke]

On 5/24/23 08:38, 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 
---
  fs/fuse/file.c | 11 +--
  1 file changed, 5 insertions(+), 6 deletions(-)


Reviewed-by: Hannes Reinecke 

Cheers,

Hannes
--
Dr. Hannes ReineckeKernel Storage Architect
h...@suse.de  +49 911 74053 688
SUSE Software Solutions GmbH, Maxfeldstr. 5, 90409 Nürnberg
HRB 36809 (AG Nürnberg), Geschäftsführer: Ivo Totev, Andrew
Myers, Andrew McDonald, Martje Boudien Moerman

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

[Miklos Szeredi]

On Wed, 24 May 2023 at 08:38, 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 

Acked-by: Miklos Szeredi 

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

[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 
Reviewed-by: Damien Le Moal 
---
 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 90d587a7bdf813..bf48aae49daf56 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;
 
@@ -1382,8 +1382,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;
@@ -1403,7 +1402,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:
inode_unlock(inode);
-- 
2.39.2