Introduce a new tracepoint for exposing the per-region per-probe
positive sample count via tracefs.

Signed-off-by: SeongJae Park <[email protected]>
---
 include/trace/events/damon.h | 38 ++++++++++++++++++++++++++++++++++++
 mm/damon/core.c              |  9 +++++++++
 2 files changed, 47 insertions(+)

diff --git a/include/trace/events/damon.h b/include/trace/events/damon.h
index 7e25f4469b81b..2b96a03876034 100644
--- a/include/trace/events/damon.h
+++ b/include/trace/events/damon.h
@@ -130,6 +130,44 @@ TRACE_EVENT(damon_monitor_intervals_tune,
        TP_printk("sample_us=%lu", __entry->sample_us)
 );
 
+TRACE_EVENT_CONDITION(damon_aggregated_v2,
+
+       TP_PROTO(unsigned int target_id, struct damon_region *r,
+               unsigned int nr_regions, unsigned int nr_probes),
+
+       TP_ARGS(target_id, r, nr_regions, nr_probes),
+
+       TP_CONDITION(nr_probes > 0),
+
+       TP_STRUCT__entry(
+               __field(unsigned long, target_id)
+               __field(unsigned long, start)
+               __field(unsigned long, end)
+               __field(unsigned int, nr_regions)
+               __field(unsigned int, nr_accesses)
+               __field(unsigned int, age)
+               __dynamic_array(unsigned char, probe_hits, nr_probes)
+       ),
+
+       TP_fast_assign(
+               __entry->target_id = target_id;
+               __entry->start = r->ar.start;
+               __entry->end = r->ar.end;
+               __entry->nr_regions = nr_regions;
+               __entry->nr_accesses = r->nr_accesses;
+               __entry->age = r->age;
+               memcpy(__get_dynamic_array(probe_hits), r->probe_hits,
+                       sizeof(*r->probe_hits) * nr_probes);
+       ),
+
+       TP_printk("target_id=%lu nr_regions=%u %lu-%lu: %u %u probe_hits=%s",
+                       __entry->target_id, __entry->nr_regions,
+                       __entry->start, __entry->end,
+                       __entry->nr_accesses, __entry->age,
+                       __print_hex(__get_dynamic_array(probe_hits),
+                               __get_dynamic_array_len(probe_hits)))
+);
+
 TRACE_EVENT(damon_aggregated,
 
        TP_PROTO(unsigned int target_id, struct damon_region *r,
diff --git a/mm/damon/core.c b/mm/damon/core.c
index fe6c789f2cecb..0ad1a0af06893 100644
--- a/mm/damon/core.c
+++ b/mm/damon/core.c
@@ -1905,6 +1905,13 @@ static void kdamond_reset_aggregated(struct damon_ctx *c)
 {
        struct damon_target *t;
        unsigned int ti = 0;    /* target's index */
+       unsigned int nr_probes = 0;
+       struct damon_probe *probe;
+
+       if (trace_damon_aggregated_v2_enabled()) {
+               damon_for_each_probe(probe, c)
+                       nr_probes++;
+       }
 
        damon_for_each_target(t, c) {
                struct damon_region *r;
@@ -1913,6 +1920,8 @@ static void kdamond_reset_aggregated(struct damon_ctx *c)
                        int i;
 
                        trace_damon_aggregated(ti, r, damon_nr_regions(t));
+                       trace_damon_aggregated_v2(ti, r, damon_nr_regions(t),
+                                       nr_probes);
                        damon_warn_fix_nr_accesses_corruption(r);
                        r->last_nr_accesses = r->nr_accesses;
                        r->nr_accesses = 0;
-- 
2.47.3

Reply via email to