lkl: tests: minor fn parameter and kasan parsing changes

[ddiss]

The following changes since commit ac5cde6a24d2b7fa8613ca3bd51e6a5567fb5adc:

  Merge pull request #609 from ddiss/lklfuse_rst_manpage (2025-06-24 10:36:25 -0700)

are available in the Git repository at:

  https://github.com/ddiss/linux minor_test_changes

for you to fetch changes up to bd30a15647247495b9b472b2a05c23e007e1a32a:

  lkl: hijack: explicitly build with -std=gnu11 (2025-06-26 20:30:37 +1000)

----------------------------------------------------------------
David Disseldorp (4):
      lkl: tests: drop unused lkl_test.fn() parameters
      lkl: define LKL_CONFIG_KASAN_KUNIT_TEST and use for test
      lkl: tests: minor lkl_test_kasan changes
      lkl: hijack: explicitly build with -std=gnu11

 arch/lkl/Makefile           |  1 +
 tools/lkl/Makefile.autoconf |  2 --
 tools/lkl/lib/hijack/Build  |  3 +++
 tools/lkl/tests/boot.c      | 22 +++++++++++++---------
 tools/lkl/tests/test.c      |  2 +-
 tools/lkl/tests/test.h      |  8 +++-----
 6 files changed, 21 insertions(+), 17 deletions(-)

[ddiss]

regarding lkl: tests: drop unused lkl_test.fn() parameters: we can't satisfy -Wstrict-prototypes everywhere yet, as hijack HOST_CALL() uses a (*host_##name)() function pointer.
I'll add an extra hijack/Build change to set CFLAGS_hijack.o += -Wno-strict-prototypes for tracking this. The open build service uses -Wstrict-prototypes by default.

[tavip]

Nice cleaning up work @ddiss, thank you!

[ddiss]

regarding lkl: tests: drop unused lkl_test.fn() parameters: we can't satisfy -Wstrict-prototypes everywhere yet, as hijack HOST_CALL() uses a (*host_##name)() function pointer. I'll add an extra hijack/Build change to set CFLAGS_hijack.o += -Wno-strict-prototypes for tracking this. The open build service uses -Wstrict-prototypes by default.

The change in fn() vs call->fn(arg1, arg2, arg3) behavior is actually due to -std=gnu23/c23. -std=gnu11 doesn't complain so we can use that for now.

[ddiss]

v2:

• add patch to build hijack.c with -std=gnu11, fixing the build where gcc -std=gnu23 is default

checkpatch failure is due to inclusion of raw compiler output (lines exceeding 72 chars) in commit message. IMO it can be ignored.

[tavip]

Thanks @ddiss, sorry for the merge delay!