* [PATCH v2 18/19] mm: damon: Use trace_call__##name() at guarded tracepoint call sites [not found] <20260323160052.17528-1-vineeth@bitbyteword.org> @ 2026-03-23 16:00 ` Vineeth Pillai (Google) 2026-03-24 0:25 ` SeongJae Park 0 siblings, 1 reply; 2+ messages in thread From: Vineeth Pillai (Google) @ 2026-03-23 16:00 UTC (permalink / raw) Cc: Vineeth Pillai (Google), Steven Rostedt, Peter Zijlstra, SeongJae Park, Andrew Morton, damon, linux-mm, linux-kernel, linux-trace-kernel Replace trace_damos_stat_after_apply_interval() with trace_call__damos_stat_after_apply_interval() at a site already guarded by an early return when !trace_damos_stat_after_apply_interval_enabled(), avoiding a redundant static_branch_unlikely() re-evaluation inside the tracepoint. Suggested-by: Steven Rostedt <rostedt@goodmis.org> Suggested-by: Peter Zijlstra <peterz@infradead.org> Signed-off-by: Vineeth Pillai (Google) <vineeth@bitbyteword.org> --- mm/damon/core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/mm/damon/core.c b/mm/damon/core.c index adfc52fee9dc2..b1cc4f44f90a2 100644 --- a/mm/damon/core.c +++ b/mm/damon/core.c @@ -2342,7 +2342,7 @@ static void damos_trace_stat(struct damon_ctx *c, struct damos *s) break; sidx++; } - trace_damos_stat_after_apply_interval(cidx, sidx, &s->stat); + trace_call__damos_stat_after_apply_interval(cidx, sidx, &s->stat); } static void kdamond_apply_schemes(struct damon_ctx *c) -- 2.53.0 ^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH v2 18/19] mm: damon: Use trace_call__##name() at guarded tracepoint call sites 2026-03-23 16:00 ` [PATCH v2 18/19] mm: damon: Use trace_call__##name() at guarded tracepoint call sites Vineeth Pillai (Google) @ 2026-03-24 0:25 ` SeongJae Park 0 siblings, 0 replies; 2+ messages in thread From: SeongJae Park @ 2026-03-24 0:25 UTC (permalink / raw) To: Vineeth Pillai (Google) Cc: SeongJae Park, Steven Rostedt, Peter Zijlstra, Andrew Morton, damon, linux-mm, linux-kernel, linux-trace-kernel On Mon, 23 Mar 2026 12:00:37 -0400 "Vineeth Pillai (Google)" <vineeth@bitbyteword.org> wrote: > Replace trace_damos_stat_after_apply_interval() with > trace_call__damos_stat_after_apply_interval() at a site already guarded > by an early return when !trace_damos_stat_after_apply_interval_enabled(), > avoiding a redundant static_branch_unlikely() re-evaluation inside the > tracepoint. > > Suggested-by: Steven Rostedt <rostedt@goodmis.org> > Suggested-by: Peter Zijlstra <peterz@infradead.org> > Signed-off-by: Vineeth Pillai (Google) <vineeth@bitbyteword.org> Reviewed-by: SeongJae Park <sj@kernel.org> Thanks, SJ [...] ^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2026-03-24 0:25 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <20260323160052.17528-1-vineeth@bitbyteword.org>
2026-03-23 16:00 ` [PATCH v2 18/19] mm: damon: Use trace_call__##name() at guarded tracepoint call sites Vineeth Pillai (Google)
2026-03-24 0:25 ` SeongJae Park
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox