Re: [PATCH 14/18] mm/damon/core: remove attrs param from damon_update_region_access_rate()

[SJ Park <sj@kernel.org>]

On Mon, 29 Jun 2026 21:08:07 -0700 SJ Park <sj@kernel.org> wrote:

> damon_region_access_rate() is not using attrs parameter.  Remove it.

The function name on the commit message here is wrong.  It is
damon_update_region_access_rate(), as you can show from the diff below.
Andrew, could you please fix this up when you pick this patch?  Let me know if
you prefer sending a new revision.

> 
> Signed-off-by: SJ Park <sj@kernel.org>
> ---
>  include/linux/damon.h | 3 +--
>  mm/damon/core.c       | 4 +---
>  mm/damon/paddr.c      | 4 ++--
>  mm/damon/vaddr.c      | 6 +++---
>  4 files changed, 7 insertions(+), 10 deletions(-)
> 
> diff --git a/include/linux/damon.h b/include/linux/damon.h
> index 87c1ff479da82..02ed47c558cc2 100644
> --- a/include/linux/damon.h
> +++ b/include/linux/damon.h
> @@ -1016,8 +1016,7 @@ unsigned int damon_nr_accesses_mvsum(struct damon_region *r,
>  
>  int damon_set_regions(struct damon_target *t, struct damon_addr_range *ranges,
>  		unsigned int nr_ranges, unsigned long min_region_sz);
> -void damon_update_region_access_rate(struct damon_region *r, bool accessed,
> -		struct damon_attrs *attrs);
> +void damon_update_region_access_rate(struct damon_region *r, bool accessed);

Above diff shows the correct function name.


Thanks,
SJ

[...]