Re: [tip: x86/asm] x86/asm/ftrace: Mark function_hook as function
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
- Subject: Re: [tip: x86/asm] x86/asm/ftrace: Mark function_hook as function
- From: Steven Rostedt <rostedt@xxxxxxxxxxx>
- Date: Fri, 18 Oct 2019 16:31:25 -0400
- Cc: tip-bot2 for Jiri Slaby <tip-bot2@xxxxxxxxxxxxx>, linux-kernel@xxxxxxxxxxxxxxx, linux-tip-commits@xxxxxxxxxxxxxxx, Jiri Slaby <jslaby@xxxxxxx>, "H. Peter Anvin" <hpa@xxxxxxxxx>, Ingo Molnar <mingo@xxxxxxxxxx>, Josh Poimboeuf <jpoimboe@xxxxxxxxxx>, linux-arch@xxxxxxxxxxxxxxx, Masami Hiramatsu <mhiramat@xxxxxxxxxx>, Peter Zijlstra <peterz@xxxxxxxxxxxxx>, Thomas Gleixner <tglx@xxxxxxxxxxxxx>, x86-ml <x86@xxxxxxxxxx>, Ingo Molnar <mingo@xxxxxxxxxx>, Borislav Petkov <bp@xxxxxxxxx>
- In-reply-to: <20191018194856.GC20368@zn.tnic>
- References: <20191011115108.12392-22-jslaby@suse.cz> <157141622788.29376.4016565749507481510.tip-bot2@tip-bot2> <20191018124800.0a7006bb@gandalf.local.home> <20191018124956.764ac42e@gandalf.local.home> <20191018171354.GB20368@zn.tnic> <20191018133735.77e90e36@gandalf.local.home> <20191018194856.GC20368@zn.tnic>
On Fri, 18 Oct 2019 21:48:56 +0200
Borislav Petkov <bp@xxxxxxx> wrote:
> > The #define was because we use to support mcount or __fentry__, now we
> > just support __fentry__, and function_hook describes it better ;-)
>
> Well sorry but gcc documentation talks about __fentry__. I'd keep the
> same name to avoid confusion and explain above it what it is. Much
> better.
Still looks ugly ;-)
>
> > Heh, I guess we could, which would probably be quite a long comment as
> > it's the key behind ftrace itself.
>
> Well, you can explain with a couple of sentences what it is and
> then point at the bigger document explaining ftrace. Provided, Mr.
> Rostedt, you'll stop doing talks and finally sit down and write that
> documentation!
I do the talks hoping someone else will finally sit down and write the
documentation!
-- Steve
[Index of Archives]
[Linux Stable Commits]
[Linux Stable Kernel]
[Linux Kernel]
[Linux USB Devel]
[Linux Video &Media]
[Linux Audio Users]
[Yosemite News]
[Linux SCSI]