lkml.org 
[lkml]   [2023]   [Mar]   [15]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: [RFC PATCH 1/3] filemap: convert page_endio to folio_endio
From
On 3/15/23 13:32, Pankaj Raghav wrote:
> page_endio() already works on folios by converting a page in to a folio as
> the first step. Convert page_endio to folio_endio by taking a folio as the
> first parameter.
>
> Instead of doing a page to folio conversion in the page_endio()
> function, the consumers of this API do this conversion and call the
> folio_endio() function in this patch.
> The following patches will convert the consumers of this API to use native
> folio functions to pass to folio_endio().
>
> Signed-off-by: Pankaj Raghav <p.raghav@samsung.com>
> ---
> drivers/block/zram/zram_drv.c | 2 +-
> fs/mpage.c | 2 +-
> fs/orangefs/inode.c | 2 +-
> include/linux/pagemap.h | 2 +-
> mm/filemap.c | 8 +++-----
> 5 files changed, 7 insertions(+), 9 deletions(-)
>
> diff --git a/drivers/block/zram/zram_drv.c b/drivers/block/zram/zram_drv.c
> index aa490da3cef2..f441251c9138 100644
> --- a/drivers/block/zram/zram_drv.c
> +++ b/drivers/block/zram/zram_drv.c
> @@ -610,7 +610,7 @@ static void zram_page_end_io(struct bio *bio)
> {
> struct page *page = bio_first_page_all(bio);
>
> - page_endio(page, op_is_write(bio_op(bio)),
> + folio_endio(page_folio(page), op_is_write(bio_op(bio)),
> blk_status_to_errno(bio->bi_status));
> bio_put(bio);
> }
> diff --git a/fs/mpage.c b/fs/mpage.c
> index 22b9de5ddd68..40e86e839e77 100644
> --- a/fs/mpage.c
> +++ b/fs/mpage.c
> @@ -50,7 +50,7 @@ static void mpage_end_io(struct bio *bio)
>
> bio_for_each_segment_all(bv, bio, iter_all) {
> struct page *page = bv->bv_page;
> - page_endio(page, bio_op(bio),
> + folio_endio(page_folio(page), bio_op(bio),
> blk_status_to_errno(bio->bi_status));
> }
>
Can't this be converted to use 'bio_for_each_folio_all()' instead of
bio_for_each_segment_all()?

Cheers,

Hannes

\
 
 \ /
  Last update: 2023-03-27 01:01    [W:0.132 / U:1.124 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site