Re: [PATCH 5/6] kselftest/arm64: Fix check_ksm_options test

[Amit Kachhap]

On 10/26/20 5:42 PM, Vincenzo Frascino wrote:

The check_ksm_options test reports the error below because the test
plan is not declared correctly:

   # Planned tests != run tests (0 != 4)

Fix the test adding the correct test plan declaration.

This change makes sense and got missed earlier.
Acked by: Amit Daniel Kachhap <amit.kach...@arm.com>

Fixes: f981d8fa2646 ("kselftest/arm64: Verify KSM page merge for MTE pages")
Cc: Shuah Khan <sh...@kernel.org>
Cc: Catalin Marinas <catalin.mari...@arm.com>
Cc: Will Deacon <w...@kernel.org>
Cc: Gabor Kertesz <gabor.kert...@arm.com>
Cc: Amit Daniel Kachhap <amit.kach...@arm.com>
Signed-off-by: Vincenzo Frascino <vincenzo.frasc...@arm.com>
---
  tools/testing/selftests/arm64/mte/check_ksm_options.c | 4 ++++
  1 file changed, 4 insertions(+)

diff --git a/tools/testing/selftests/arm64/mte/check_ksm_options.c 
b/tools/testing/selftests/arm64/mte/check_ksm_options.c
index bc41ae630c86..3b23c4d61d38 100644
--- a/tools/testing/selftests/arm64/mte/check_ksm_options.c
+++ b/tools/testing/selftests/arm64/mte/check_ksm_options.c
@@ -140,6 +140,10 @@ int main(int argc, char *argv[])
        /* Register signal handlers */
        mte_register_signal(SIGBUS, mte_default_handler);
        mte_register_signal(SIGSEGV, mte_default_handler);
+
+       /* Set test plan */
+       ksft_set_plan(4);
+
        /* Enable KSM */
        mte_ksm_setup();