On Mon, 2026-04-13 at 14:26 -0300, Marcos Paulo de Souza wrote: > A new version of the patchset, with fewer patches now. Please take a > look! > > Original cover-letter: > These patches don't really change how the patches are run, just skip > some tests on kernels that don't support a feature (like kprobe and > livepatched living together) or when a livepatch sysfs attribute is > missing. > > The last patch slightly adjusts check_result function to skip dmesg > messages on SLE kernels when a livepatch is removed. > > These patches are based on printk/for-next branch. > > Please review! Thanks! > > Signed-off-by: Marcos Paulo de Souza <[email protected]>
I saw some checks made by sashiko, and besides they not being so horrible, I believe that I should fix me an send a v3 to address them. But still, feel free to take a look :) https://sashiko.dev/#/patchset/20260413-lp-tests-old-fixes-v2-0-367c7cb5006f%40suse.com > --- > Changes in v2: > - Patch descriptions were changed to remove "test-X", since it was > polluting the commit subjects (Miroslav Benes) > - Patch 8 was dropped since it was checking for a message from an > out-of-tree patch. (Petr Mladek) > - Patch 3 was dropped as should be treated as expected failure for > older kernels. (Petr Mladek) > - Patch 2 was changed to use y/n instead of 1/0, since it's more > natural to use it. > - Patch 1 was changed to handle ppc and loongson, and error out if > dealing with a different architecture that sets > CONFIG_ARCH_HAS_SYSCALL_WRAPPER and haven't changed the test to > include the proper wrapper prefix. > - Patch 4 was changed to invert the return of the bash function to > return 1 in failure, like > a normal bash function (Joe Lawrence) > - Patches 5, 6 an 7 were changed to not split the tests, but to only > run the tests > when the attribute were present (Miroslav Benes) > - Link to v1: > https://patch.msgid.link/[email protected] > > --- > Marcos Paulo de Souza (6): > selftests: livepatch: Check for ARCH_HAS_SYSCALL_WRAPPER config > selftests: livepatch: Replace true/false module parameter by > y/n > selftests: livepatch: Introduce does_sysfs_exists function > selftests: livepatch: Check if patched sysfs attribute exists > selftests: livepatch: Check if replace sysfs attribute exists > selftests: livepatch: Check if stack_order sysfs attribute > exists > > tools/testing/selftests/livepatch/functions.sh | 10 ++ > tools/testing/selftests/livepatch/test-kprobe.sh | 8 +- > tools/testing/selftests/livepatch/test-sysfs.sh | 120 > ++++++++++++--------- > .../livepatch/test_modules/test_klp_syscall.c | 17 ++- > 4 files changed, 99 insertions(+), 56 deletions(-) > --- > base-commit: 712c0756828becbfc629ff8d8b82deff5d1115e4 > change-id: 20260309-lp-tests-old-fixes-f955abc8ec27 > > Best regards, > -- > Marcos Paulo de Souza <[email protected]>
