Re: [PATCH v2 00/11] mm/damon: support ARM32 with LPAE

[SeongJae Park]

On Fri, 22 Aug 2025 17:34:08 +0800 Quanmin Yan <yanquanmin1@xxxxxxxxxx> wrote:

> Previously, DAMON's physical address space monitoring only supported
> memory ranges below 4GB on LPAE-enabled systems. This was due to
> the use of 'unsigned long' in 'struct damon_addr_range', which is
> 32-bit on ARM32 even with LPAE enabled[1].
> 
> To add DAMON support for ARM32 with LPAE enabled, a new core layer
> parameter called 'addr_unit' was introduced[2]. Operations set layer
> can translate a core layer address to the real address by multiplying
> the parameter value to the core layer address. Support of the parameter
> is up to each operations layer implementation, though. For example,
> operations set implementations for virtual address space can simply
> ignore the parameter. Add the support on paddr, which is the DAMON
> operations set implementation for the physical address space, as we have
> a clear use case for that.
> 
> [1]https://lore.kernel.org/all/20250408075553.959388-1-zuoze1@xxxxxxxxxx/
> [2]https://lore.kernel.org/all/20250416042551.158131-1-sj@xxxxxxxxxx/
> 
> Changes from v1
> (https://lore.kernel.org/all/20250821105159.2503894-1-yanquanmin1@xxxxxxxxxx/)
> - Set dst->min_sz_region as src->min_sz_region in damon_commit_ctx().
> - Fixed the incorrect 'min_region' in the commit message.

Thank you for continuing this work, Quanmin!  Now the entire series looks good
to me :)


Thanks,
SJ

[...]