Received-SPF: pass (google.com: domain of linux-kernel+bounces-106428-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) client-ip=139.178.88.99;
Date: Mon, 18 Mar 2024 18:12:29 +0100
From: Jan Kara <jack@suse.cz>
To: Christian Brauner <brauner@kernel.org>
Cc: viro@zeniv.linux.org.uk, jack@suse.cz,
	Kemeng Shi <shikemeng@huaweicloud.com>, tim.c.chen@linux.intel.com,
	linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH v2 0/6] Fixes and cleanups to fs-writeback
Message-ID: <20240318171229.ftdwkh3a45r4y6j7@quack3>
References: <20240228091958.288260-1-shikemeng@huaweicloud.com>
Precedence: bulk
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20240228091958.288260-1-shikemeng@huaweicloud.com>

On Wed 28-02-24 17:19:52, Kemeng Shi wrote:
> v1->v2:
> -Filter non-expired in requeue_inode in patch "fs/writeback: avoid to
> writeback non-expired inode in kupdate writeback"
> -Wrap the comment at 80 columns in patch "fs/writeback: only calculate
> dirtied_before when b_io is empty"
> -Abandon patch "fs/writeback: remove unneeded check in
> writeback_single_inode"
> -Collect RVB from Jan and Tim

Christian, the series looks good to me. Please pick it up once your tree
settles after the merge window. Thanks!

								Honza

> 
> Kemeng Shi (6):
>   fs/writeback: avoid to writeback non-expired inode in kupdate
>     writeback
>   fs/writeback: bail out if there is no more inodes for IO and queued
>     once
>   fs/writeback: remove unused parameter wb of finish_writeback_work
>   fs/writeback: only calculate dirtied_before when b_io is empty
>   fs/writeback: correct comment of __wakeup_flusher_threads_bdi
>   fs/writeback: remove unnecessary return in writeback_inodes_sb
> 
>  fs/fs-writeback.c | 57 +++++++++++++++++++++++++++--------------------
>  1 file changed, 33 insertions(+), 24 deletions(-)
> 
> -- 
> 2.30.0
> 
-- 
Jan Kara <jack@suse.com>
SUSE Labs, CR