| Message ID | 20201026121248.2340-6-vincenzo.frascino@arm.com |
|---|---|
| State | Accepted |
| Commit | cbb268af05de9beedc6994f4d90628f451d996d7 |
| Headers | show |
| Series | [1/6] kselftest/arm64: Fix check_buffer_fill test | expand |
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.kachhap@arm.com> > > Fixes: f981d8fa2646 ("kselftest/arm64: Verify KSM page merge for MTE pages") > Cc: Shuah Khan <shuah@kernel.org> > Cc: Catalin Marinas <catalin.marinas@arm.com> > Cc: Will Deacon <will@kernel.org> > Cc: Gabor Kertesz <gabor.kertesz@arm.com> > Cc: Amit Daniel Kachhap <amit.kachhap@arm.com> > Signed-off-by: Vincenzo Frascino <vincenzo.frascino@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(); > >
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();
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. Fixes: f981d8fa2646 ("kselftest/arm64: Verify KSM page merge for MTE pages") Cc: Shuah Khan <shuah@kernel.org> Cc: Catalin Marinas <catalin.marinas@arm.com> Cc: Will Deacon <will@kernel.org> Cc: Gabor Kertesz <gabor.kertesz@arm.com> Cc: Amit Daniel Kachhap <amit.kachhap@arm.com> Signed-off-by: Vincenzo Frascino <vincenzo.frascino@arm.com> --- tools/testing/selftests/arm64/mte/check_ksm_options.c | 4 ++++ 1 file changed, 4 insertions(+)