Re: [RFC PATCH 0/2] kunit: Support redirecting function calls

[Steve Muckle]

On 4/4/22 13:13, Brendan Higgins wrote:
> On Fri, Mar 18, 2022 at 9:22 AM Steven Rostedt <rostedt@xxxxxxxxxxx> wrote:
> > On Fri, 18 Mar 2022 10:13:12 +0800
> > David Gow <davidgow@xxxxxxxxxx> wrote:
> >
> > > Does either (or both) of these features sound useful, and is this
> > > sort-of API the right model? (Personally, I think there's a reasonable
> > > scope for both.) Is anything obviously missing or wrong? Do the names,
> > > descriptions etc. make any sense?
> >
> > Obviously I'm biased toward the ftrace solution ;-)
>
> Personally, I like providing both - as long as we can keep the
> interface the same.
>
> Ftrace is less visually invasive, but it is also less flexible in
> capabilities, and requires substantial work to support on new
> architectures.

The general feature looks useful to me. I'm not sure the ftrace based 
API is worth it given it is only offering a visual improvement and has 
some drawbacks compared to the other implementation (won't work with 
inline functions, dependencies on other features). Livepatch is absent 
on arm64 which mostly rules it out for my purposes (Android Generic 
Kernel Image testing).

cheers,
Steve