Re: [PATCH 1/9] LoongArch/ftrace: Add basic support

From: Steven Rostedt
Date: Fri Aug 19 2022 - 13:35:44 EST


On Fri, 19 Aug 2022 17:29:29 +0800
Jinyang He <hejinyang@xxxxxxxxxxx> wrote:

> It seems this patch adds non-dynamic ftrace, this code should not
> appear here.
> BTW is it really necessary for non-dynamic ftrace? I do not use it
> directly and frequently, IMHO, non-dynamic can be completely
>
> replaced dynamic?

Note, I keep the non dynamic ftrace around for debugging purposes.

But sure, it's pretty useless. It's also good for bringing ftrace to a new
architecture (like this patch is doing), as it is easier to implement than
dynamic ftrace, and getting the non-dynamic working is usually the first
step in getting dynamic ftrace working.

But it's really up to the arch maintainers to keep it or not.

-- Steve