[PATCH] riscv: ftrace: Fix the logic issue in DYNAMIC_FTRACE selection

Steven Rostedt rostedt at goodmis.org
Thu Jul 3 08:40:18 PDT 2025


On Thu, 3 Jul 2025 15:00:02 +0200
Alexandre Ghiti <alex at ghiti.fr> wrote:


> 
> We could support static ftrace, but I don't think we should, so I agree 
> with this patch. In fact I had just prepared a patch for this here 
> https://github.com/linux-riscv/linux/pull/556/commits/0481092a5bec3818658981c11f629e06e66382b3 
> which is a bit more complete since I have removed some dead code.
> 
> Let's see what other people think about supporting static ftrace, I have 
> added Steven in cc if he has an opinion.

Yes, please only support the dynamic ftrace. The static is there only
to help archs to get ftrace up and running. Once dynamic is supported,
static should not be used.

Hmm, maybe I should just remove the prompt for DYNAMIC_FTRACE.

That is, once it is supported by an architecture, it should be the only
thing used.

-- Steve



More information about the linux-riscv mailing list