[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20251124164905.55cd25bb@gandalf.local.home>
Date: Mon, 24 Nov 2025 16:49:05 -0500
From: Steven Rostedt <rostedt@...dmis.org>
To: Donglin Peng <dolinux.peng@...il.com>
Cc: linux-trace-kernel@...r.kernel.org, linux-kernel@...r.kernel.org,
Donglin Peng <pengdonglin@...omi.com>, Sven Schnelle <svens@...ux.ibm.com>,
Masami Hiramatsu <mhiramat@...nel.org>
Subject: Re: [PATCH v7] function_graph: Enable funcgraph-args and
funcgraph-retaddr to work simultaneously
On Mon, 24 Nov 2025 16:47:01 -0500
Steven Rostedt <rostedt@...dmis.org> wrote:
> On Fri, 14 Nov 2025 10:55:22 +0800
> Donglin Peng <dolinux.peng@...il.com> wrote:
>
> > --- a/kernel/trace/trace_entries.h
> > +++ b/kernel/trace/trace_entries.h
> > @@ -95,14 +95,14 @@ FTRACE_ENTRY_PACKED(fgraph_retaddr_entry, fgraph_retaddr_ent_entry,
> > TRACE_GRAPH_RETADDR_ENT,
> >
> > F_STRUCT(
> > - __field_struct( struct fgraph_retaddr_ent, graph_ent )
> > + __field_struct( struct ftrace_graph_ent, graph_ent )
> > __field_packed( unsigned long, graph_ent, func )
> > __field_packed( unsigned int, graph_ent, depth )
> > - __field_packed( unsigned long, graph_ent, retaddr )
>
> You can't delete the retaddr without breaking user space.
>
> Please keep that here.
>
Also, please rebase on top of:
git://git.kernel.org/pub/scm/linux/kernel/git/trace/linux-trace.git trace/for-next
Thanks!
-- Steve
Powered by blists - more mailing lists