Re: [PATCH v3 02/10] fuse: convert fuse_send_write_pages to use folios

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Sep 27, 2024 at 1:46 PM Josef Bacik <josef@xxxxxxxxxxxxxx> wrote:
>
> Convert this to grab the folio from the fuse_args_pages and use the
> appropriate folio related functions.
>
> Reviewed-by: Matthew Wilcox (Oracle) <willy@xxxxxxxxxxxxx>
> Signed-off-by: Josef Bacik <josef@xxxxxxxxxxxxxx>

Reviewed-by: Joanne Koong <joannelkoong@xxxxxxxxx>

> ---
>  fs/fuse/file.c | 14 +++++++-------
>  1 file changed, 7 insertions(+), 7 deletions(-)
>
> diff --git a/fs/fuse/file.c b/fs/fuse/file.c
> index 132528cde745..17ac2de61cdb 100644
> --- a/fs/fuse/file.c
> +++ b/fs/fuse/file.c
> @@ -1168,23 +1168,23 @@ static ssize_t fuse_send_write_pages(struct fuse_io_args *ia,
>         offset = ap->descs[0].offset;
>         count = ia->write.out.size;
>         for (i = 0; i < ap->num_pages; i++) {
> -               struct page *page = ap->pages[i];
> +               struct folio *folio = page_folio(ap->pages[i]);
>
>                 if (err) {
> -                       ClearPageUptodate(page);
> +                       folio_clear_uptodate(folio);
>                 } else {
> -                       if (count >= PAGE_SIZE - offset)
> -                               count -= PAGE_SIZE - offset;
> +                       if (count >= folio_size(folio) - offset)
> +                               count -= folio_size(folio) - offset;
>                         else {
>                                 if (short_write)
> -                                       ClearPageUptodate(page);
> +                                       folio_clear_uptodate(folio);
>                                 count = 0;
>                         }
>                         offset = 0;
>                 }
>                 if (ia->write.page_locked && (i == ap->num_pages - 1))
> -                       unlock_page(page);
> -               put_page(page);
> +                       folio_unlock(folio);
> +               folio_put(folio);
>         }
>
>         return err;
> --
> 2.43.0
>
>





[Index of Archives]     [Linux Ext4 Filesystem]     [Union Filesystem]     [Filesystem Testing]     [Ceph Users]     [Ecryptfs]     [NTFS 3]     [AutoFS]     [Kernel Newbies]     [Share Photos]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux Cachefs]     [Reiser Filesystem]     [Linux RAID]     [NTFS 3]     [Samba]     [Device Mapper]     [CEPH Development]

  Powered by Linux