From:   SeongJae Park <sj@kernel.org>
To:     Kefeng Wang <wangkefeng.wang@huawei.com>
Cc:     SeongJae Park <sj@kernel.org>,
        Andrew Morton <akpm@linux-foundation.org>, linux-mm@kvack.org,
        linux-kernel@vger.kernel.org, damon@lists.linux.dev
Subject: Re: [PATCH v3 0/3] mm/damon/paddr: minor code improvement
Date:   Wed,  8 Mar 2023 18:17:17 +0000
Message-Id: <20230308181717.46186-1-sj@kernel.org>
In-Reply-To: <20230308083311.120951-1-wangkefeng.wang@huawei.com>
References: 
MIME-Version: 1.0
Content-Transfer-Encoding: 8bit
Precedence: bulk

Hi Kefeng,

On Wed, 8 Mar 2023 16:33:08 +0800 Kefeng Wang <wangkefeng.wang@huawei.com> wrote:

> Unify folio_put() to make code more clear, and also fix minor issue in
> damon_pa_young().
> 
> v3: 
> - as SJ split the bugfix from v2, and it has merged[1], only do code
>   improvement
> - fix a missing folio_sz update in damon_pa_young()
> - add RB from SJ
> - rebase on next-20230308
> 
> [1] https://lkml.kernel.org/r/20230304193949.296391-1-sj@kernel.org
> 
> v2: 
> - use goto lable in damon_pa_pageout() and refact too more functions
> 
> Kefeng Wang (3):
>   mm/damon/paddr: minor refactor of damon_pa_pageout()
>   mm/damon/paddr: minor refactor of
>     damon_pa_mark_accessed_or_deactivate()
>   mm/damon/paddr: fix missing folio_sz update in damon_pa_young()
> 
>  mm/damon/paddr.c | 26 ++++++++++----------------
>  1 file changed, 10 insertions(+), 16 deletions(-)

For the series,

Reviewed-by: SeongJae Park <sj@kernel.org>

Thank you for your efforts on this, Kefeng!


Thanks,
SJ

> 
> -- 
> 2.35.3
>