mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
synced 2026-08-30 16:53:20 -04:00
mm/damon/core: remove attrs param from damon_update_region_access_rate()
damon_update_region_access_rate() is not using attrs parameter. Remove it. Link: https://lore.kernel.org/20260630040812.149729-15-sj@kernel.org Signed-off-by: SJ Park <sj@kernel.org> Cc: Brendan Higgins <brendan.higgins@linux.dev> Cc: "Masami Hiramatsu (Google)" <mhiramat@kernel.org> Cc: Mathieu Desnoyers <mathieu.desnoyers@efficios.com> Cc: Shuah Khan <shuah@kernel.org> Cc: Steven Rostedt <rostedt@goodmis.org> Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
This commit is contained in:
@@ -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);
|
||||
|
||||
struct damos_filter *damos_new_filter(enum damos_filter_type type,
|
||||
bool matching, bool allow);
|
||||
|
||||
@@ -3833,15 +3833,13 @@ static unsigned int damon_moving_sum(unsigned int mvsum, unsigned int nomvsum,
|
||||
* damon_update_region_access_rate() - Update the access rate of a region.
|
||||
* @r: The DAMON region to update for its access check result.
|
||||
* @accessed: Whether the region has accessed during last sampling interval.
|
||||
* @attrs: The damon_attrs of the DAMON context.
|
||||
*
|
||||
* Update the access rate of a region with the region's last sampling interval
|
||||
* access check result.
|
||||
*
|
||||
* Usually this will be called by &damon_operations->check_accesses callback.
|
||||
*/
|
||||
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)
|
||||
{
|
||||
if (accessed)
|
||||
r->nr_accesses++;
|
||||
|
||||
@@ -91,12 +91,12 @@ static void __damon_pa_check_access(struct damon_region *r,
|
||||
/* If the region is in the last checked page, reuse the result */
|
||||
if (ALIGN_DOWN(last_addr, last_folio_sz) ==
|
||||
ALIGN_DOWN(sampling_addr, last_folio_sz)) {
|
||||
damon_update_region_access_rate(r, last_accessed, attrs);
|
||||
damon_update_region_access_rate(r, last_accessed);
|
||||
return;
|
||||
}
|
||||
|
||||
last_accessed = damon_pa_young(sampling_addr, &last_folio_sz);
|
||||
damon_update_region_access_rate(r, last_accessed, attrs);
|
||||
damon_update_region_access_rate(r, last_accessed);
|
||||
|
||||
last_addr = sampling_addr;
|
||||
}
|
||||
|
||||
@@ -501,19 +501,19 @@ static void __damon_va_check_access(struct mm_struct *mm,
|
||||
static bool last_accessed;
|
||||
|
||||
if (!mm) {
|
||||
damon_update_region_access_rate(r, false, attrs);
|
||||
damon_update_region_access_rate(r, false);
|
||||
return;
|
||||
}
|
||||
|
||||
/* If the region is in the last checked page, reuse the result */
|
||||
if (same_target && (ALIGN_DOWN(last_addr, last_folio_sz) ==
|
||||
ALIGN_DOWN(r->sampling_addr, last_folio_sz))) {
|
||||
damon_update_region_access_rate(r, last_accessed, attrs);
|
||||
damon_update_region_access_rate(r, last_accessed);
|
||||
return;
|
||||
}
|
||||
|
||||
last_accessed = damon_va_young(mm, r->sampling_addr, &last_folio_sz);
|
||||
damon_update_region_access_rate(r, last_accessed, attrs);
|
||||
damon_update_region_access_rate(r, last_accessed);
|
||||
|
||||
last_addr = r->sampling_addr;
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user