Messages in this thread | | | From | SeongJae Park <> | Subject | Re: [PATCH] mm/damon/core: iterate the regions list from current point in damon_set_regions() | Date | Tue, 6 Sep 2022 18:20:06 +0000 |
| |
Hi Kaixu,
On Tue, 6 Sep 2022 23:18:47 +0800 xiakaixu1987@gmail.com wrote:
> From: Kaixu Xia <kaixuxia@tencent.com> > > We iterate the whole regions list every time to get the first/last regions > intersecting with the specific range in damon_set_regions(), in order to > add new region or resize existing regions to fit in the specific range. > Actually, it is unnecessary to iterate the new added regions and the > front regions that have been checked. Just iterate the regions list from > the current point using list_for_each_entry_from() every time to improve > performance. > The kunit tests passed: > [PASSED] damon_test_apply_three_regions1 > [PASSED] damon_test_apply_three_regions2 > [PASSED] damon_test_apply_three_regions3 > [PASSED] damon_test_apply_three_regions4 > > Signed-off-by: Kaixu Xia <kaixuxia@tencent.com>
Reviewed-by: SeongJae Park <sj@kernel.org>
Thanks, SJ
[...]
| |