Re: [PATCH v4 06/15] ftrace: Use an opaque type for functions not callable from C

From: Sami Tolvanen
Date: Wed Oct 06 2021 - 12:31:26 EST


On Wed, Oct 6, 2021 at 6:02 AM Steven Rostedt <rostedt@xxxxxxxxxxx> wrote:
>
> On Tue, 5 Oct 2021 20:29:45 -0700
> Josh Poimboeuf <jpoimboe@xxxxxxxxxx> wrote:
>
> Thanks for Cc'ing me, as I should have been Cc'd on the original patch.

Sorry about that. I'll make sure you're cc'd on the next version.

> > On Thu, Sep 30, 2021 at 11:05:22AM -0700, Sami Tolvanen wrote:
> > > With CONFIG_CFI_CLANG, the compiler changes function references to point
> > > to the CFI jump table. As ftrace_call, ftrace_regs_call, and mcount_call
> > > are not called from C, use DECLARE_ASM_FUNC_SYMBOL to declare them.
>
> "not called from C" is a bit confusing.

Any thoughts on how to make this less confusing?

Sami