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

From: Steve Muckle
Date: Fri Apr 15 2022 - 17:44:50 EST


+Joe Fradley who is also looking at KUnit with Android.

On 4/15/22 14:43, Steve Muckle wrote:
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