[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <oggzqu4j23ihzsi7qfwiluy5w3nwubgbyhqu2a3hdtta7cyhno@smlzq7xmrflq>
Date: Wed, 2 Jul 2025 16:00:51 +0200
From: Jan Kara <jack@...e.cz>
To: Zhang Yi <yi.zhang@...weicloud.com>
Cc: linux-ext4@...r.kernel.org, linux-fsdevel@...r.kernel.org,
linux-kernel@...r.kernel.org, tytso@....edu, adilger.kernel@...ger.ca, jack@...e.cz,
ojaswin@...ux.ibm.com, sashal@...nel.org, yi.zhang@...wei.com, libaokun1@...wei.com,
yukuai3@...wei.com, yangerkun@...wei.com
Subject: Re: [PATCH v3 01/10] ext4: process folios writeback in bytes
On Tue 01-07-25 21:06:26, Zhang Yi wrote:
> From: Zhang Yi <yi.zhang@...wei.com>
>
> Since ext4 supports large folios, processing writebacks in pages is no
> longer appropriate, it can be modified to process writebacks in bytes.
>
> Suggested-by: Jan Kara <jack@...e.cz>
> Signed-off-by: Zhang Yi <yi.zhang@...wei.com>
Just one small issue. With that fixed feel free to add:
Reviewed-by: Jan Kara <jack@...e.cz>
> @@ -2786,18 +2788,18 @@ static int ext4_do_writepages(struct mpage_da_data *mpd)
> writeback_index = mapping->writeback_index;
> if (writeback_index)
> cycled = 0;
> - mpd->first_page = writeback_index;
> - mpd->last_page = -1;
> + mpd->start_pos = writeback_index << PAGE_SHIFT;
> + mpd->end_pos = -1;
Careful here. Previously last_page was unsigned long so -1 was fine but now
loff_t is signed. So we should rather store LLONG_MAX here.
Honza
--
Jan Kara <jack@...e.com>
SUSE Labs, CR
Powered by blists - more mailing lists