mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
synced 2026-08-31 10:31:33 -04:00
mm/damon/core: remove damon_region->nr_accesses_bp
No code touches damon_region->nr_accesses_bp field. Remove it. Link: https://lore.kernel.org/20260630040812.149729-19-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:
@@ -45,8 +45,6 @@ struct damon_size_range {
|
||||
* @ar: The address range of the region.
|
||||
* @sampling_addr: Address of the sample for the next access check.
|
||||
* @nr_accesses: Access frequency of this region.
|
||||
* @nr_accesses_bp: @nr_accesses in basis point (0.01%) that updated for
|
||||
* each sampling interval.
|
||||
* @probe_hits: Number of probe-positive region samples.
|
||||
* @list: List head for siblings.
|
||||
* @age: Age of this region.
|
||||
@@ -59,13 +57,6 @@ struct damon_size_range {
|
||||
* not be done with direct access but with the helper function,
|
||||
* damon_update_region_access_rate().
|
||||
*
|
||||
* @nr_accesses_bp is another representation of @nr_accesses in basis point
|
||||
* (1 in 10,000) that updated for every &damon_attrs->sample_interval in a
|
||||
* manner similar to moving sum. By the algorithm, this value becomes
|
||||
* @nr_accesses * 10000 for every &struct damon_attrs->aggr_interval. This can
|
||||
* be used when the aggregation interval is too huge and therefore cannot wait
|
||||
* for it before getting the access monitoring results.
|
||||
*
|
||||
* @age is initially zero, increased for each aggregation interval, and reset
|
||||
* to zero again if the access frequency is significantly changed. If two
|
||||
* regions are merged into a new region, both @nr_accesses and @age of the new
|
||||
@@ -75,7 +66,6 @@ struct damon_region {
|
||||
struct damon_addr_range ar;
|
||||
unsigned long sampling_addr;
|
||||
unsigned int nr_accesses;
|
||||
unsigned int nr_accesses_bp;
|
||||
unsigned char probe_hits[DAMON_MAX_PROBES];
|
||||
struct list_head list;
|
||||
|
||||
|
||||
@@ -3649,8 +3649,7 @@ static int kdamond_fn(void *data)
|
||||
* aggregation, and make aggregation
|
||||
* information reset for all regions. Then,
|
||||
* following kdamond_reset_aggregated() call
|
||||
* will make the region information invalid,
|
||||
* particularly for ->nr_accesses_bp.
|
||||
* will make the region information invalid.
|
||||
*
|
||||
* Reset ->next_aggregation_sis to avoid that.
|
||||
* It will anyway correctly updated after this
|
||||
|
||||
Reference in New Issue
Block a user