lkml.org 
[lkml]   [2023]   [Nov]   [5]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [RFC PATCH 24/32] x86/ftrace: Enable HAVE_FUNCTION_GRAPH_FREGS
    On Sun, Nov 05, 2023 at 02:11:30PM -0500, Steven Rostedt wrote:
    > On Sun, 5 Nov 2023 18:25:36 +0100
    > Peter Zijlstra <peterz@infradead.org> wrote:
    >
    > > On Mon, Nov 06, 2023 at 01:11:21AM +0900, Masami Hiramatsu (Google) wrote:
    > > > From: Masami Hiramatsu (Google) <mhiramat@kernel.org>
    > > >
    > > > Support HAVE_FUNCTION_GRAPH_FREGS on x86-64, which saves ftrace_regs
    > > > on the stack in ftrace_graph return trampoline so that the callbacks
    > > > can access registers via ftrace_regs APIs.
    > >
    > > What is ftrace_regs ? If I look at arch/x86/include/asm/ftrace.h it's a
    > > pointless wrapper around pt_regs.
    > >
    > > Can we please remove the pointless wrappery and call it what it is?
    >
    > A while back ago when I introduced FTRACE_WITH_ARGS, it would have all
    > ftrace callbacks get a pt_regs, but it would be partially filled for
    > those that did not specify the "REGS" flag when registering the
    > callback. You and Thomas complained that it would be a bug to return
    > pt_regs that was not full because something might read the non filled
    > registers and think they were valid.
    >
    > To solve this, I came up with ftrace_regs to only hold the registers
    > that were required for function parameters (including the stack
    > pointer). You could then call arch_ftrace_get_regs(ftrace_regs) and if
    > this "wrapper" had all valid pt_regs registers, then it would return
    > the pt_regs, otherwise it would return NULL, and you would need to use
    > the ftrace_regs accessor calls to get the function registers. You and
    > Thomas agreed with this.

    Changelog nor code made it clear this was partial anything. So this is
    still the partial thing?

    Can we then pretty clear clarify all that, and make it clear which regs
    are in there? Because when I do 'vim -t ftrace_regs' it just gets me a
    seemingly pointless wrapper struct, no elucidating comments nothingses.

    > You even Acked the patch:
    >
    > commit 02a474ca266a47ea8f4d5a11f4ffa120f83730ad
    > Author: Steven Rostedt (VMware) <rostedt@goodmis.org>
    > Date: Tue Oct 27 10:55:55 2020 -0400

    You expect me to remember things from 3 years ago?

    \
     
     \ /
      Last update: 2023-11-20 13:48    [W:2.890 / U:0.092 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site