On Thu, Nov 13, 2025 at 01:02:18PM +0000, [email protected] wrote: > > diff --git a/include/linux/ftrace.h b/include/linux/ftrace.h > > index e23e6a859..ded3a306a 100644 > > --- a/include/linux/ftrace.h > > +++ b/include/linux/ftrace.h > > [ ... ] > > > @@ -568,6 +570,11 @@ static inline int modify_ftrace_direct_nolock(struct > > ftrace_ops *ops, unsigned l > > return -ENODEV; > > } > > > > +int update_ftrace_direct_add(struct ftrace_ops *ops, struct ftrace_hash > > *hash) > > +{ > > + return -ENODEV; > > +} > > + > > /* > > * This must be implemented by the architecture. > > * It is the way the ftrace direct_ops helper, when called > > Should this stub function have static inline qualifiers? All the other > stub functions in this section (ftrace_find_rec_direct, > register_ftrace_direct, unregister_ftrace_direct, modify_ftrace_direct, > and modify_ftrace_direct_nolock) use static inline. Without it, this > could cause multiple definition linker errors when the header is > included in multiple compilation units.
yep, also the other ones are missing it, will add.. thanks jirka > > > --- > AI reviewed your patch. Please fix the bug or email reply why it's not a bug. > See: https://github.com/kernel-patches/vmtest/blob/master/ci/claude/README.md > > CI run summary: https://github.com/kernel-patches/bpf/actions/runs/19332026793
