Re: [RFC PATCH v2] mm/damon/core: filter out damos address filter-split failed region

[sashiko-bot@kernel.org]

> When an address type DAMOS filter is configured, DAMOS tries to split
> the region for the filter range, and apply DAMOS action to the filter-in
> region.  The split operation (damon_split_region_at()) could fail from
> allocation failure under extreme memory pressure.  In the case, DAMOS
> action could be applied more or less than expected, ignoring the address
> filter.
> 
> The issue only rarely happens under extreme memory pressure.  No crash
> or leak happens.  Occasionally not applying the action to a few regions
> shouldn't be a real problem.  But applying the action to a region that
> the user has excluded via address filter could be a surprise depending
> on the use case.  It is only theoretical.  No such real case is found
> yet.
> 
> Filter out the entire region if the split is failed.  Update
> damos_filter_match() signature to receive the error from split.  Also
> update kunit tests for the updated signature.
> 
> The issue was discovered [1] by sashiko.
> 
> [1] https://lore.kernel.org/20260801203121.117449-1-sj@kernel.org
> 
> Fixes: ab9bda001b68 ("mm/damon/core: introduce address range type damos filter")
> Cc: <stable@vger.kernel.org> # 6.6.x
> Signed-off-by: SJ Park <sj@kernel.org>

Sashiko has reviewed this patch and found no issues.

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20261019133856.86835-1-sj@kernel.org?part=1