On Fri, Sep 27, 2024 at 1:45 PM Josef Bacik <josef@xxxxxxxxxxxxxx> wrote:
>
> We're just looking for pages in a mapping, use a folio and the folio
> lookup function directly instead of using the page helper.
>
> Reviewed-by: Joanne Koong <joannelkoong@xxxxxxxxx>
> Signed-off-by: Josef Bacik <josef@xxxxxxxxxxxxxx>
> ---
> fs/fuse/dev.c | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/fs/fuse/dev.c b/fs/fuse/dev.c
> index 1f64ae6d7a69..4c58113eb6a1 100644
> --- a/fs/fuse/dev.c
> +++ b/fs/fuse/dev.c
> @@ -1756,15 +1756,15 @@ static int fuse_retrieve(struct fuse_mount *fm, struct inode *inode,
> index = outarg->offset >> PAGE_SHIFT;
>
> while (num && ap->num_pages < num_pages) {
> - struct page *page;
> + struct folio *folio;
> unsigned int this_num;
>
> - page = find_get_page(mapping, index);
> - if (!page)
> + folio = __filemap_get_folio(mapping, index, 0, 0);
I think you can also just use "filemap_get_folio(mapping, index);" here
> + if (IS_ERR(folio))
> break;
>
> this_num = min_t(unsigned, num, PAGE_SIZE - offset);
> - ap->pages[ap->num_pages] = page;
> + ap->pages[ap->num_pages] = &folio->page;
> ap->descs[ap->num_pages].offset = offset;
> ap->descs[ap->num_pages].length = this_num;
> ap->num_pages++;
> --
> 2.43.0
>
