Hi Kees, On 20 June 2017 at 00:21, Kees Cook <keescook@xxxxxxxxxxxx> wrote: > On Mon, Jun 19, 2017 at 6:44 AM, Sumit Semwal <sumit.semwal@xxxxxxxxxx> wrote: >> With older kernels, printf.sh and bitmap.sh fail because they can't find >> the respective test modules they are looking for. >> >> Add the skip portion on missing the respective test_XXX module. Error out >> with the same error code as prime_numbers.sh. >> >> v2: Per Shuah's review, search for the module rather than do modprobe. >> >> Signed-off-by: Sumit Semwal <sumit.semwal@xxxxxxxxxx> >> --- >> tools/testing/selftests/lib/bitmap.sh | 6 ++++++ >> tools/testing/selftests/lib/printf.sh | 6 ++++++ >> 2 files changed, 12 insertions(+) >> >> diff --git a/tools/testing/selftests/lib/bitmap.sh b/tools/testing/selftests/lib/bitmap.sh >> index 2da187b6ddad..b893046d511f 100755 >> --- a/tools/testing/selftests/lib/bitmap.sh >> +++ b/tools/testing/selftests/lib/bitmap.sh >> @@ -1,6 +1,12 @@ >> #!/bin/sh >> # Runs bitmap infrastructure tests using test_bitmap kernel module >> >> +if ! find /lib/modules/$(uname -r) -type f -name test_bitmap.ko | grep -q .; >> +then >> + echo "bitmap: test_bitmap.ko not found [SKIP]" >> + exit 77 >> +fi >> + >> if /sbin/modprobe -q test_bitmap; then >> /sbin/modprobe -q -r test_bitmap >> echo "bitmap: ok" >> diff --git a/tools/testing/selftests/lib/printf.sh b/tools/testing/selftests/lib/printf.sh >> index 4fdc70fe6980..804b7e75acc0 100755 >> --- a/tools/testing/selftests/lib/printf.sh >> +++ b/tools/testing/selftests/lib/printf.sh >> @@ -1,6 +1,12 @@ >> #!/bin/sh >> # Runs printf infrastructure using test_printf kernel module >> >> +if ! find /lib/modules/$(uname -r) -type f -name test_printf.ko | grep -q .; >> +then >> + echo "printf: test_printf.ko not found [SKIP]" >> + exit 77 >> +fi >> + >> if /sbin/modprobe -q test_printf; then >> /sbin/modprobe -q -r test_printf >> echo "printf: ok" > > This doesn't seem right to me. Can't these be built-ins too? > Yes, it can be. I'm running test right now to check presence of module with -q -n; that should take care of builtin as well. will send an updated patch soon. > -Kees > > -- > Kees Cook > Pixel Security Best, Sumit. -- To unsubscribe from this list: send the line "unsubscribe linux-kselftest" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html